Commit e2a4800e authored by Florian Westphal's avatar Florian Westphal Committed by David S. Miller

ppp: deflate: never return len larger than output buffer

When we've run out of space in the output buffer to store more data, we
will call zlib_deflate with a NULL output buffer until we've consumed
remaining input.

When this happens, olen contains the size the output buffer would have
consumed iff we'd have had enough room.

This can later cause skb_over_panic when ppp_generic skb_put()s
the returned length.
Reported-by: default avatarIain Douglas <centos@1n6.org.uk>
Signed-off-by: default avatarFlorian Westphal <fw@strlen.de>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent d445d63b
...@@ -246,7 +246,7 @@ static int z_compress(void *arg, unsigned char *rptr, unsigned char *obuf, ...@@ -246,7 +246,7 @@ static int z_compress(void *arg, unsigned char *rptr, unsigned char *obuf,
/* /*
* See if we managed to reduce the size of the packet. * See if we managed to reduce the size of the packet.
*/ */
if (olen < isize) { if (olen < isize && olen <= osize) {
state->stats.comp_bytes += olen; state->stats.comp_bytes += olen;
state->stats.comp_packets++; state->stats.comp_packets++;
} else { } else {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment