Ondrej Zajicek <santi...@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 <t...@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

Reply via email to