11 Jul
2018
11 Jul
'18
8:13 p.m.
Ondrej Zajicek <santiago@crfreenet.org> writes:
On Tue, Jul 10, 2018 at 11:56:40PM +0200, Toke Høiland-Jørgensen wrote:
Signed-off-by: Toke Høiland-Jørgensen <toke@toke.dk>
Hi
I think that the current position is correct and the patch is not - the follow-up code ('The end of the common TLV header') checks for full 2-byte TLV header, while BABEL_TLV_PAD1 is just 1-byte padding.
Yeah, you're right; sorry for the noise. Guess I was seeing things after looking at too many bounds check constructs last night :) -Toke