]> granicus.if.org Git - libnl/commit
lib/route/cls/u32.c: avoid overflowing an unsigned char
authorRasmus Villemoes <linux@rasmusvillemoes.dk>
Thu, 8 Jun 2017 08:59:24 +0000 (10:59 +0200)
committerRasmus Villemoes <linux@rasmusvillemoes.dk>
Thu, 8 Jun 2017 08:59:24 +0000 (10:59 +0200)
commitf0e3d127cbb45a9b03b6cf37b34e74433d8064bf
treee436d5639c38b9cc5d919271ee6394d00d359013
parent2fda8661fd55e747d44974054b73bb66153aeb59
lib/route/cls/u32.c: avoid overflowing an unsigned char

If rtnl_u32_add_key is called too many times, sel->nkeys will wrap from
255 to 0, effectively killing all the previous rtnl_u32_add_key
calls (while having an inaccessible chunk of memory sitting beyond the
accessible part of the ->keys array). Not sure NOMEM is the best error
code, but that's at least something users would already have to be
prepared to handle.
lib/route/cls/u32.c