summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid S. Miller <davem@sunset.davemloft.net>2006-01-17 02:54:21 -0800
committerDavid S. Miller <davem@sunset.davemloft.net>2006-01-17 02:54:21 -0800
commit8243126c5e29030bf1a3fb75187a513966dcba62 (patch)
tree39750d44770efcdac150f041f71b7272c8da20f9
parentf09484ff87f677056ce631aa3d8e486861501b51 (diff)
downloadop-kernel-dev-8243126c5e29030bf1a3fb75187a513966dcba62.zip
op-kernel-dev-8243126c5e29030bf1a3fb75187a513966dcba62.tar.gz
[NET]: Make second arg to skb_reserved() signed.
Some subsystems, such as PPP, can send negative values here. It just happened to work correctly on 32-bit with an unsigned value, but on 64-bit this explodes. Figured out by Paul Mackerras based upon several PPP crash reports. Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--include/linux/skbuff.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h
index e5fd66c..ad7cc22 100644
--- a/include/linux/skbuff.h
+++ b/include/linux/skbuff.h
@@ -926,7 +926,7 @@ static inline int skb_tailroom(const struct sk_buff *skb)
* Increase the headroom of an empty &sk_buff by reducing the tail
* room. This is only allowed for an empty buffer.
*/
-static inline void skb_reserve(struct sk_buff *skb, unsigned int len)
+static inline void skb_reserve(struct sk_buff *skb, int len)
{
skb->data += len;
skb->tail += len;
OpenPOWER on IntegriCloud