Wireshark-bugs: [Wireshark-bugs] [Bug 7282] New: RVALS() needs BASE_RANGE_STRING (?)

Date: Thu, 24 May 2012 00:05:16 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7282

           Summary: RVALS() needs BASE_RANGE_STRING (?)
           Product: Wireshark
           Version: SVN
          Platform: x86
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Low
         Component: Wireshark
        AssignedTo: bugzilla-admin@xxxxxxxxxxxxx
        ReportedBy: wireshark@xxxxxxxxx


Build Information:
Paste the COMPLETE build information from "Help->About Wireshark", "wireshark
-v", or "tshark -v".
--
Dear all,

recently, I played with a range_string and saw mysterious crashes when it was
used in an hf variable. It turned out that I set BASE_HEX and did not or it
with BASE_RANGE_STRING. My understanding from epan/protocol.h is that this is
generally required.

------
/* Following constants have to be ORed with a base_display_e when dissector
 * want to use specials MACROs (for the moment, only RVALS) for a
 * header_field_info */
#define BASE_RANGE_STRING 0x10
------

The attached patch corrects hf definitions in the code that use RVALS() but
have no BASE_RANGE_STRING. Please merge it if you agree that this should be
fixed.

Best regards,

   Martin

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