Wireshark-bugs: [Wireshark-bugs] [Bug 2825] Patch: expanded dissection of the kademlia protocol

Date: Sun, 24 Aug 2008 08:41:11 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2825





--- Comment #6 from Stefan Monhof <stefan.monhof@xxxxxxxxxxxxxxx>  2008-08-24 08:41:10 PDT ---
(In reply to comment #2)
> (From update of attachment 2176 [details])
> here's something wrong... we are dereferencing a pointer in one line and
> testing for it in the following line.
> 
> Either for good, it cannot be NULL and so the test in the following line is
> useless, or for bad it can actually be NULL and crash when derreferenced.
> 
> 
> @@ -863,8 +921,8 @@
> 
>      tag_full_name = "UnknownTagName";
> 
> +    tagname_value = *(guint8*)tagname;
>      if ( tagname && string_length == 1 ) {
> -        guint8 tagname_value = *(guint8*)tagname;
>          /* lookup tagname */
>          tag_full_name = val_to_str( tagname_value, kademlia_tags,
> tag_full_name );
>      }
> 

Uh, you're right. I've had added some information there, but later I moved them
to another function. It seems I forgot to change it back correctly.
I'll add a fix ontop of the actual revision.


-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.