Wireshark · Wireshark-dev: Re: [Wireshark-dev] [PATCH] babel: fix infinite loop with TLVs of length 0.

[Juliusz Chroboczek <jch@xxxxxxx>]

[Resent with fixed CC.]

> do you intend to push the patch set to our Gerrit as explained in
> https://www.wireshark.org/docs/wsdg_html_chunked/ChSrcContribute.html ?
> Our workflow is not using mail based patches.

We tried, but failed.  "git review" returned a 500 error.  Slightly later,
authentication failed (I was using the password autogenerated in the
Gerrit user interface).

> Presumably we should also remove the test on sublen == 0 I added to fix the
> infinite loop (as you stated this was valid).

Right, we missed that.

> Moreover in case of MESSAGE_SUB_PAD1, is really beg variable only
> incremented by 1 and not 2?

That's right, PAD1 is just a lone byte of value 0, not a real TLV.

  https://tools.ietf.org/html/draft-ietf-babel-rfc6126bis-15#section-4.7.1

> (you fetched a sublen field also and highlighted 2 bytes for the
> hf_babel_subtlv field).

Right again.

I'll fix the code, see again if I can get Gerrit to work, then submit
a new version.

Thanks for your help,

-- Juliusz