[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wireshark-dev
Subject:    Re: [Wireshark-dev] Warn Dissector bug, protocol RADIUS
From:       Jeff Morriss <jeff.morriss.ws () gmail ! com>
Date:       2016-03-30 20:12:09
Message-ID: CAKkq+FYPq21wupKwRb2XP5dQn3cFqq7h_JxjEvuyCS8OvC9YuA () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


On Wed, Mar 30, 2016 at 2:05 PM, Jeff Morriss <jeff.morriss.ws@gmail.com>
wrote:

>
>
> On Wed, Mar 30, 2016 at 11:51 AM, Anders Broman <
> anders.broman@ericsson.com> wrote:
>
>> Hi,
>>
>> After the recent radius changes I get these console printouts for radius
>> packets….
>>
>>
> Hey, you blamin' me? ;-) ;-)
>

Well you should have been, even if you weren't. :-)


> C:\Development\wireshark>17:30:27          Warn Dissector bug, protocol
>> RADIUS,
>>
>> in packet 65: proto.c:2494: failed assertion "(guint)hfindex <
>> gpa_hfinfo.len" (
>>
>> Unregistered hf!)
>>
>
> Hmm, it looks like proto_register_prefix() isn't calling the "register the
> fields" function until *after* dissecting the packets (at least once).
> Weird.  I'll try to take a deeper look in a bit.
>

https://code.wireshark.org/review/14711

[Attachment #5 (text/html)]

<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar \
30, 2016 at 2:05 PM, Jeff Morriss <span dir="ltr">&lt;<a \
href="mailto:jeff.morriss.ws@gmail.com" \
target="_blank">jeff.morriss.ws@gmail.com</a>&gt;</span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div \
class="gmail_extra"><br><div class="gmail_quote"><span class="">On Wed, Mar 30, 2016 \
at 11:51 AM, Anders Broman <span dir="ltr">&lt;<a \
href="mailto:anders.broman@ericsson.com" \
target="_blank">anders.broman@ericsson.com</a>&gt;</span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex">





<div link="blue" vlink="purple" lang="EN-US">
<div>
<p class="MsoNormal">Hi,<u></u><u></u></p>
<p class="MsoNormal">After the recent radius changes I get these console printouts \
for radius packets….<u></u><u></u></p> <p \
class="MsoNormal"><u></u></p></div></div></blockquote><div><br></div></span><div>Hey, \
you blamin&#39; me? ;-) ;-) \
<br></div></div></div></div></blockquote><div><br></div><div>Well you should have \
been, even if you weren&#39;t. :-)<br></div><div>  </div><blockquote \
class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div \
class="gmail_quote"><div></div><span class=""><blockquote class="gmail_quote" \
style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex"><div link="blue" vlink="purple" lang="EN-US"><div> \
<p class="MsoNormal">C:\Development\wireshark&gt;17:30:27                   Warn \
Dissector bug, protocol RADIUS,</p> <p class="MsoNormal">in packet 65: proto.c:2494: \
failed assertion &quot;(guint)hfindex &lt; gpa_hfinfo.len&quot; (</p> <p \
class="MsoNormal"><span lang="SV">Unregistered \
hf!)</span></p></div></div></blockquote><div><br></div></span><div>Hmm, it looks like \
proto_register_prefix() isn&#39;t calling the &quot;register the fields&quot; \
function until *after* dissecting the packets (at least once).   Weird.   I&#39;ll \
try to take a deeper look in a bit.<br></div></div></div></div> \
</blockquote></div><br><a \
href="https://code.wireshark.org/review/14711">https://code.wireshark.org/review/14711</a><br><br></div></div>




___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@wireshark.org>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@wireshark.org?subject=unsubscribe

[prev in list] [next in list] [prev in thread] [next in thread]