diff options
author | Konstantin Sharlaimov <konstantin.sharlaimov@gmail.com> | 2007-06-24 08:05:54 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2007-06-24 08:05:54 +0200 |
commit | 4b2a8fb3a7f7935f62a7bbdc851789fb7c2da032 (patch) | |
tree | 73481354594bdab8a9208619ac9dd3f1ce47f424 /drivers/net/ppp_generic.c | |
parent | [PPP]: Revert 606f585e363527da9feaed79465132c0c661fd9e (diff) | |
download | linux-4b2a8fb3a7f7935f62a7bbdc851789fb7c2da032.tar.xz linux-4b2a8fb3a7f7935f62a7bbdc851789fb7c2da032.zip |
[PPP]: Fix osize too small errors when decoding mppe.
The mppe_decompress() function required a buffer that is 1 byte too
small when receiving a message of mru size. This fixes buffer
allocation to prevent this from occurring.
Signed-off-by: Konstantin Sharlaimov <konstantin.sharlaimov@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ppp_generic.c')
-rw-r--r-- | drivers/net/ppp_generic.c | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/drivers/net/ppp_generic.c b/drivers/net/ppp_generic.c index 541168713f1f..3ef0092dc09c 100644 --- a/drivers/net/ppp_generic.c +++ b/drivers/net/ppp_generic.c @@ -1708,7 +1708,18 @@ ppp_decompress_frame(struct ppp *ppp, struct sk_buff *skb) goto err; if (proto == PPP_COMP) { - ns = dev_alloc_skb(ppp->mru + PPP_HDRLEN); + int obuff_size; + + switch(ppp->rcomp->compress_proto) { + case CI_MPPE: + obuff_size = ppp->mru + PPP_HDRLEN + 1; + break; + default: + obuff_size = ppp->mru + PPP_HDRLEN; + break; + } + + ns = dev_alloc_skb(obuff_size); if (ns == 0) { printk(KERN_ERR "ppp_decompress_frame: no memory\n"); goto err; |