]> git.ipfire.org Git - thirdparty/iproute2.git/commit
ss: avoid passing negative numbers to malloc
authorAndreas Henriksson <andreas@fatal.se>
Wed, 13 Nov 2013 08:46:42 +0000 (09:46 +0100)
committerStephen Hemminger <stephen@networkplumber.org>
Sat, 23 Nov 2013 01:03:23 +0000 (17:03 -0800)
commitf26ef6ec09ea3b7a57916885ae74d9b3e5afadcc
tree5c1f8365b20eb0dcd6dc09a84e497e9df0a64379
parentbc7635a8b377dafd7b074661294ccd217a6aaf9b
ss: avoid passing negative numbers to malloc

Example:

$ ss state established \( sport = :4060  or sport = :4061 or sport = :4062  or sport = :4063 or sport = :4064  or sport = :4065 or sport = :4066  or sport = :4067 \)  > /dev/null
Aborted

In the example above ssfilter_bytecompile(...) will return (int)136.
char l1 = 136; means -120 which will result in a negative number
being passed to malloc at misc/ss.c:913.

Simply declare l1 and l2 as integers to avoid the char overflow.

This is one of the issues originally reported in http://bugs.debian.org/511720

Fix the same problem in other code paths as well (thanks to Eric Dumazet).

Reported-by: Andreas Schuldei <andreas@debian.org>
Signed-off-by: Andreas Henriksson <andreas@fatal.se>
Reviewed-by: Eric Dumazet <edumazet@google.com>
misc/ss.c