Wireshark · Wireshark-bugs: [Wireshark-bugs] [Bug 8010] New: Wildcarded address not changed to not wildcarded in conversation.

[bugzilla-daemon@xxxxxxxxxxxxx]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8010

           Summary: Wildcarded address not changed to not wildcarded in
                    conversation.
           Product: Wireshark
           Version: SVN
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Low
         Component: Dissection engine (libwireshark)
        AssignedTo: bugzilla-admin@xxxxxxxxxxxxx
        ReportedBy: intkecsk@xxxxxxxxx


Created attachment 9559
  --> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=9559
Proposed correction for this error and some similar errors.

Build Information:
{Revision 46138 doesn't build successfully due to errors in capture-wpcap.c}

--
Revision 46138

It is said in "conversation.c"
"If search address B isn't wildcarded, and this is for a
connection-oriented protocol, set the second address for this
conversation to address B, as that's the address that matched the
wildcarded second address for this conversation.

(This assumes that, for all connection oriented protocols, the
endpoints of a connection have only one address each, i.e. you
don't get packets in a given direction coming from more than one
address, unless the CONVERSATION_TEMPLATE option is set.)"

This is not the case, because instead of

if (!(options & NO_ADDR_B) && ptype != PT_UDP)

it is written

if (!(conversation->options & NO_ADDR_B) && ptype != PT_UDP)
(line 884)
Latter condition is always false because NO_ADDR_B=NO_ADDR2=0x1, and
conversation is found in conversation_hashtable_no_addr2.

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