diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-05-26 03:40:11 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-05-26 03:40:11 -0400 |
| commit | ff937938e7781b2c1bffce0a5400af308e3946d5 (patch) | |
| tree | 70a5ccf25d967d93f4a47ab30d25b0b81e538cd5 | |
| parent | ed0f160ad674407adb3aba499444f71c83289c63 (diff) | |
sctp: dubious bitfields in sctp_transport
Sparse complains because these one-bit bitfields are signed.
include/net/sctp/structs.h:879:24: error: dubious one-bit signed bitfield
include/net/sctp/structs.h:889:31: error: dubious one-bit signed bitfield
include/net/sctp/structs.h:895:26: error: dubious one-bit signed bitfield
include/net/sctp/structs.h:898:31: error: dubious one-bit signed bitfield
include/net/sctp/structs.h:901:27: error: dubious one-bit signed bitfield
It doesn't cause a problem in the current code, but it would be better
to clean it up. This was introduced by c0058a35aacc7: "sctp: Save some
room in the sctp_transport by using bitfields".
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | include/net/sctp/structs.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/net/sctp/structs.h b/include/net/sctp/structs.h index 6173c619913a..4b860116e096 100644 --- a/include/net/sctp/structs.h +++ b/include/net/sctp/structs.h | |||
| @@ -876,7 +876,7 @@ struct sctp_transport { | |||
| 876 | 876 | ||
| 877 | /* Reference counting. */ | 877 | /* Reference counting. */ |
| 878 | atomic_t refcnt; | 878 | atomic_t refcnt; |
| 879 | int dead:1, | 879 | __u32 dead:1, |
| 880 | /* RTO-Pending : A flag used to track if one of the DATA | 880 | /* RTO-Pending : A flag used to track if one of the DATA |
| 881 | * chunks sent to this address is currently being | 881 | * chunks sent to this address is currently being |
| 882 | * used to compute a RTT. If this flag is 0, | 882 | * used to compute a RTT. If this flag is 0, |
