Wireshark · Wireshark-bugs: [Wireshark-bugs] [Bug 6335] New: Radius dissector, includes Radius attribute constants as Radius mes

[bugzilla-daemon@xxxxxxxxxxxxx]

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

           Summary: Radius dissector, includes Radius attribute constants
                    as Radius message types in static gboolean
                    is_radius(tvbuff_t *tvb) function
           Product: Wireshark
           Version: 1.6.1
          Platform: x86
        OS/Version: Windows XP
            Status: NEW
          Severity: Trivial
          Priority: Low
         Component: Wireshark
        AssignedTo: bugzilla-admin@xxxxxxxxxxxxx
        ReportedBy: ridvanozaydin@xxxxxxxxx


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

I was viewing the wireshark source code and in radius dissector (
packet-radius.h, packet-radius.c ) i noticed two radius attributes put into a
switch case statement, in "static gboolean is_radius(tvbuff_t *tvb)" function
along with  radius headers identifiers. 

case RADIUS_EAP_MESSAGE_CODE:
case RADIUS_MESSAGE_AUTHENTICATOR:

in rfc 2869, i saw these two defined as attributes.

"http://www.faqs.org/rfcs/rfc2869.html";
"Section 5 Attributes"

I definetely have no xp with the radius protocol, maybe i am completely wrong
but i thought this might worth letting someone know ...

Source code i downloaded belongs to :
wireshark 1.6.1
path to access files is:
wireshark-1.6.1\wireshark-1.6.1\epan\dissectors

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