]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
ieee1394: fix broken signed char assumption.
authorLinus Torvalds <torvalds@g5.osdl.org>
Wed, 6 Jul 2005 20:05:50 +0000 (13:05 -0700)
committerLinus Torvalds <torvalds@g5.osdl.org>
Wed, 6 Jul 2005 20:05:50 +0000 (13:05 -0700)
"ack_code" is assigned (and tested against) negative numbers, but was
declared as "char".  Which only works if "char" is signed - which it
necessarily isn't.

So make that signedness assumption specific.

drivers/ieee1394/ieee1394_core.h

index 73bd8efd2b6c94081400539c0cf5508f409e42d0..0b31429d0a683cdae682f26017178ac7669a34e9 100644 (file)
@@ -38,8 +38,8 @@ struct hpsb_packet {
 
         /* These are core internal. */
         signed char tlabel;
-        char ack_code;
-        char tcode;
+       signed char ack_code;
+       unsigned char tcode;
 
         unsigned expect_response:1;
         unsigned no_waiter:1;