* remote.c (read_frame): Split readchar/checksum calculation into

two parts since evaluation order is undefined.
This commit is contained in:
Steve Chamberlain 1994-07-30 05:04:03 +00:00
parent 0c18c7370b
commit 205fc02b50
2 changed files with 7 additions and 2 deletions

View File

@ -1,3 +1,8 @@
Fri Jul 29 21:56:23 1994 Steve Chamberlain (sac@jonny.cygnus.com)
* remote.c (read_frame): Split readchar/checksum calculation into
two parts since evaluation order is undefined.
Fri Jul 29 13:46:08 1994 Fred Fish (fnf@cygnus.com)
From Kevin A. Buettner (kev@cujo.geg.mot.com).

View File

@ -1231,8 +1231,8 @@ read_frame (buf)
*bp = '\000';
pktcsum = fromhex (readchar (remote_timeout)) << 4
| fromhex (readchar (remote_timeout));
pktcsum = fromhex (readchar (remote_timeout)) << 4;
pktcsum |= fromhex (readchar (remote_timeout));
if (csum == pktcsum)
return 1;