diff options
author | Eyal Birger <eyal.birger@gmail.com> | 2015-03-01 07:58:29 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-03-02 00:19:30 -0500 |
commit | b4772ef879a8f7d8c56118c2ae5a296fcf6f81d2 (patch) | |
tree | b4498c776c28e5c87cdeebf17577d002bcd21d47 /net/can | |
parent | 2472d7613bd3bae40a7dc75539c0858d5b9e795a (diff) |
net: use common macro for assering skb->cb[] available size in protocol families
As part of an effort to move skb->dropcount to skb->cb[] use a common
macro in protocol families using skb->cb[] for ancillary data to
validate available room in skb->cb[].
Signed-off-by: Eyal Birger <eyal.birger@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/can')
-rw-r--r-- | net/can/bcm.c | 2 | ||||
-rw-r--r-- | net/can/raw.c | 6 |
2 files changed, 4 insertions, 4 deletions
diff --git a/net/can/bcm.c b/net/can/bcm.c index ee9ffd956552..d559f922326d 100644 --- a/net/can/bcm.c +++ b/net/can/bcm.c | |||
@@ -328,7 +328,7 @@ static void bcm_send_to_user(struct bcm_op *op, struct bcm_msg_head *head, | |||
328 | * containing the interface index. | 328 | * containing the interface index. |
329 | */ | 329 | */ |
330 | 330 | ||
331 | BUILD_BUG_ON(sizeof(skb->cb) < sizeof(struct sockaddr_can)); | 331 | sock_skb_cb_check_size(sizeof(struct sockaddr_can)); |
332 | addr = (struct sockaddr_can *)skb->cb; | 332 | addr = (struct sockaddr_can *)skb->cb; |
333 | memset(addr, 0, sizeof(*addr)); | 333 | memset(addr, 0, sizeof(*addr)); |
334 | addr->can_family = AF_CAN; | 334 | addr->can_family = AF_CAN; |
diff --git a/net/can/raw.c b/net/can/raw.c index 00c13ef23661..94601b7ff0a3 100644 --- a/net/can/raw.c +++ b/net/can/raw.c | |||
@@ -95,8 +95,8 @@ struct raw_sock { | |||
95 | */ | 95 | */ |
96 | static inline unsigned int *raw_flags(struct sk_buff *skb) | 96 | static inline unsigned int *raw_flags(struct sk_buff *skb) |
97 | { | 97 | { |
98 | BUILD_BUG_ON(sizeof(skb->cb) <= (sizeof(struct sockaddr_can) + | 98 | sock_skb_cb_check_size(sizeof(struct sockaddr_can) + |
99 | sizeof(unsigned int))); | 99 | sizeof(unsigned int)); |
100 | 100 | ||
101 | /* return pointer after struct sockaddr_can */ | 101 | /* return pointer after struct sockaddr_can */ |
102 | return (unsigned int *)(&((struct sockaddr_can *)skb->cb)[1]); | 102 | return (unsigned int *)(&((struct sockaddr_can *)skb->cb)[1]); |
@@ -135,7 +135,7 @@ static void raw_rcv(struct sk_buff *oskb, void *data) | |||
135 | * containing the interface index. | 135 | * containing the interface index. |
136 | */ | 136 | */ |
137 | 137 | ||
138 | BUILD_BUG_ON(sizeof(skb->cb) < sizeof(struct sockaddr_can)); | 138 | sock_skb_cb_check_size(sizeof(struct sockaddr_can)); |
139 | addr = (struct sockaddr_can *)skb->cb; | 139 | addr = (struct sockaddr_can *)skb->cb; |
140 | memset(addr, 0, sizeof(*addr)); | 140 | memset(addr, 0, sizeof(*addr)); |
141 | addr->can_family = AF_CAN; | 141 | addr->can_family = AF_CAN; |