aboutsummaryrefslogtreecommitdiffstats
path: root/epan/dissectors/packet-radius.c
AgeCommit message (Collapse)AuthorFilesLines
2011-09-14Update based upon latest names "packet-type-codes" list from the IANA:Bill Meier1-122/+114
- Change some type-code names to match IANA list; - Handle additional type-codes as given in the IANA list; - Don't consider certain "attribute types" to be valid packet-type codes See Bug 6335: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6335 - Minor code cleanup. svn path=/trunk/; revision=38997
2011-06-03Add some notes about attributes with a special format.Guy Harris1-0/+17
svn path=/trunk/; revision=37529
2011-05-20Add #include <stdlib.h> to files which reference fcns declared in stdlib.h;Bill Meier1-0/+4
(In many cases I previously incorrectly removed the #include <stdlib.h>). svn path=/trunk/; revision=37334
2011-04-16Fix the warnings in Coverity CIDs 606-608.Gerald Combs1-1/+1
svn path=/trunk/; revision=36671
2011-04-04Change the signature of elem_fcn():ns to include pinfo.Anders Broman1-6/+6
svn path=/trunk/; revision=36443
2011-03-08Don't bother creating/finding conversations if we're encapsulated within anChris Maynard1-0/+10
error packet, such as an ICMP destination unreachable. Prior to this change, I had captures that showed such encapsulated requests with responses indicated in other frames, which obviously can't be true. svn path=/trunk/; revision=36169
2011-03-08Conversation tracking must be handled whether "tree" is NULL or not.Chris Maynard1-208/+205
svn path=/trunk/; revision=36168
2011-03-08Add support for the pre-rfc DAE port 1700Jörg Mayer1-0/+2
svn path=/trunk/; revision=36167
2011-01-21Introduce, and start using, tvb_eth_to_str().Jeff Morriss1-1/+1
Export tvb_*_to_str() and put them in alphabetical order. svn path=/trunk/; revision=35602
2011-01-08Use report_failure() to report problems reading the RADIUS dictionary,Guy Harris1-2/+2
so it shows up in the GUI in Wireshark. svn path=/trunk/; revision=35427
2010-12-20Rename the routines that handle dissector tables with unsigned integerGuy Harris1-7/+7
keys to have _uint in their names, to match the routines that handle dissector tables with string keys. (Using _port can confuse people into thinking they're intended solely for use with TCP/UDP/etc. ports when, in fact, they work better for things such as Ethernet types, where the binding of particular values to particular protocols are a lot stronger.) svn path=/trunk/; revision=35224
2010-10-10Define some fcns & vars as static ...Bill Meier1-23/+23
svn path=/trunk/; revision=34458
2010-09-17Use val_to_str_ext() & friends to access sminmpec_values[];Bill Meier1-2/+4
Also: packet-nhrp.c: #include sminmpec.h not req'd; svn path=/trunk/; revision=34143
2010-05-21Have abs_time_to_str() and abs_time_to_str_secs() take an additionalGuy Harris1-1/+1
argument indicating whether to include the time zone in the string. If we're constructing a display filter, don't include the time zone, otherwise do. Fixes bug 4756. svn path=/trunk/; revision=32913
2010-02-27Add a third date format, ABSOLUTE_TIME_DOY_UTC, to show UTC with theGuy Harris1-1/+1
date as YYYY/DDD, where DDD is a 1-origin day of year. Move the formats to a "time_fmt.h" file, included by the headers that use it. Have abs_time_to_str() and abs_time_secs_to_str() take the date format value, rather than a Boolean "show this as UTC" flag, as an argument. Document the ABSOLUTE_TIME_ formats a bit better. Use that format in the CCSDS and VCDU dissectors, rather than having those dissectors do the formatting themselves. svn path=/trunk/; revision=32034
2009-12-19For fields of type FT_ABSOLUTE_TIME, have the "display" value be one ofGuy Harris1-1/+1
ABSOLUTE_TIME_LOCAL or ABSOLUTE_TIME_UTC, indicating whether to display the date/time in local time or UTC. (int)ABSOLUTE_TIME_LOCAL == (int)BASE_NONE, so there's no source or binary compatiblity issue, although we might want to eliminate BASE_NONE at some point and have the BASE_ values used with integral types start at 0, so that you can't specify BASE_NONE for an integral field. svn path=/trunk/; revision=31319
2009-12-10Add an argument to abs_time_to_str() and abs_time_secs_to_str()Guy Harris1-1/+1
indicating whether the time should be shown as local time or UTC. For now, always pass FALSE, meaning "show as local time". Clean up some stuff in the SNMP dissector, use abs_time_secs_to_str() for times with one-second resolution, and update a comment in various macros in the WSP dissector, while we're at it. svn path=/trunk/; revision=31227
2009-12-10Fix Bug 4316: Crash when using custom RADIUS dictionary.Bill Meier1-0/+1
(The bug was caused by a variable not being initialized). See: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4316 svn path=/trunk/; revision=31221
2009-10-12Initialize radius_call, so it's null if we don't pass through the codeGuy Harris1-1/+1
path that sets it. svn path=/trunk/; revision=30524
2009-10-12Replace use of GMemChunks by se_alloc();Bill Meier1-575/+554
@include <stdlib.h> not req'd; Use consistent indentation. svn path=/trunk/; revision=30522
2009-08-26(FWIW) One step towards including stdio.h & stdlib.h only when req'd.Bill Meier1-0/+1
svn path=/trunk/; revision=29568
2009-08-26From Thierry Andry via bug 3941:Stig Bjørlykke1-1/+4
Added preference to set radius request TTL. svn path=/trunk/; revision=29561
2009-08-09Don't guard col_clear with col_checkKovarththanan Rajaratnam1-2/+1
svn path=/trunk/; revision=29344
2009-08-09Don't guard col_set_str (COL_PROTOCOL) with col_checkKovarththanan Rajaratnam1-2/+1
svn path=/trunk/; revision=29340
2009-07-30Remove a couple of unused variables.Gerald Combs1-6/+0
svn path=/trunk/; revision=29245
2009-07-07From Kovarththanan Rajaratnam via bug 3548:Stig Bjørlykke1-4/+4
Minor hf cleanup. svn path=/trunk/; revision=28968
2009-07-06Radius: Additional attribute handling:Bill Meier1-0/+10
- implement "ether" attribute; - parse "abinary" attribute [uses existing radius_abinary()] Also: add a comment in packet-radius.h about the meaning of the encrypt flag. svn path=/trunk/; revision=28952
2009-07-03From: Tobias Witek: Fix processing of radius 'vendor specific attributes'Bill Meier1-0/+5
In the RADIUS dissector, the function radius_register_avp_dissector() registers vendors that are not already present in the dictionary hash-table. As far as I can see, there are two problems with this: 1. The function does not set the number of type/length octets and the has_flags variable for that AVP, which is required to correctly decode AVP/VSA values 2. In some situations, the function is called _before_ radius_load_dictionary() is called (for example for the vendor 3GPP (ID: 10415)) Therefore, all vendor entries that are created by calling radius_register_avp_dissector() leave their type_octets and length_octets un-initialized, which causes incorrect decoding. [Result: Radius dissector displays messages such as: "Malformed Packet: RADIUS" and "Error/Malformed: Malformed Packet(Exception occurred)"] The attached patch fixes this problem by assuming that the dictionary knows the 'ground truth' about the type/length octet and the has_flags information and allows it to overwrite these values even for vendors that have already been loaded. Also: (from Bill Meier): set the type/length octet and the has_flags variables to default "standard" values (1,1,FALSE) in radius_register_avp_dissector(). Fixes Bug #3651 (and Bug #3635). https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3651 https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3635 svn path=/trunk/; revision=28937
2009-06-30packet-radius: Fix exception CLEANUP handling when malformed attribute ↵Bill Meier1-12/+15
list/pair seen; Fixes crash reported in Bug #3578. [https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3578] Essentially: CLEANUP_CALL_AND_POP wasn't being executed for certain error exits from dissect_attribute_pairs() thus leaving a CLEANUP entry on the exception stack. Also: vsa_buffer_table wasn't being destroyed if an exception occurred in dissect_attribute_pairs. svn path=/trunk/; revision=28891
2009-06-18From Kovarththanan Rajaratnam via bug 3548:Stig Bjørlykke1-20/+20
(1) Trailing/leading spaces are removed from 'name's/'blurb's (2) Duplicate 'blurb's are replaced with NULL (3) Empty ("") 'blurb's are replaced with NULL (4) BASE_NONE, NULL, 0x0 are used for 'display', 'strings' and 'bitmask' fields for FT_NONE, FT_BYTES, FT_IPv4, FT_IPv6, FT_ABSOLUTE_TIME, FT_RELATIVE_TIME, FT_PROTOCOL, FT_STRING and FT_STRINGZ field types (5) Only allow non-zero value for 'display' if 'bitmask' is non-zero svn path=/trunk/; revision=28770
2009-06-14Prevent duplicate header field registrations.Jaap Keuter1-5/+2
svn path=/trunk/; revision=28720
2009-06-09RFC3576 states, that some messages will use port 3799 insteadJörg Mayer1-0/+2
of the "standard" radius ports. Register radius to that port as well. svn path=/trunk/; revision=28672
2009-05-13Apply some of the patches from:Anders Broman1-4/+4
http://wiki.wireshark.org/Development/Optimization svn path=/trunk/; revision=28356
2009-04-02From Naoyoshi Ueda:Anders Broman1-9/+324
Radius dissector enhancement to support WiMAX vendor specific attributes. https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3176 svn path=/trunk/; revision=27937
2009-03-29More size_t casts.Anders Broman1-1/+1
svn path=/trunk/; revision=27887
2009-03-22P64 changes: Fix some cases for which size_t is not requiredBill Meier1-1/+1
svn path=/trunk/; revision=27822
2009-03-18From Bjørn Mork:Anders Broman1-0/+36
Decode ipv6prefix attributes in packet-radius. svn path=/trunk/; revision=27769
2008-12-29From Florian Lohoff:Anders Broman1-2/+77
Decode binary "Ascend-Data-Filter" vendor specific attribute in packet-radius https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2975 svn path=/trunk/; revision=27127
2008-12-29From Florian Lohoff:Anders Broman1-4/+5
packet-radius.c - avp specific dissectors broken https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2974 svn path=/trunk/; revision=27126
2008-09-30Minor cleanup related to proto_register, proto_reg_handoffBill Meier1-30/+29
svn path=/trunk/; revision=26307
2008-09-11Work around a problem with custom columns.Anders Broman1-2/+1
svn path=/trunk/; revision=26178
2008-08-05luis.ontanon@gmail.com => luis@ontanon.orgLuis Ontanon1-1/+1
svn path=/trunk/; revision=25937
2008-05-14Fix some of the Errors/warnings detected by checkapi.Bill Meier1-5/+9
svn path=/trunk/; revision=25290
2008-04-03http://www.wireshark.org/lists/wireshark-dev/200803/msg00308.htmlLuis Ontanon1-150/+132
proto.[hc] define new APIs to allow delayed registration of protocol fields, so that dissectors with "flexible" fields like xml, radius, diameter, snmp do not have to load their files at startup but can do so as late as possible. gtk/dfilter_expr_dlg.c : have the expression dialog registering all prefixes so that all fileds appear in the dialog tshark.c register all prefixes when called with -G epan/radius_dict.l epan/dissectors/packet-radius.c epan/dissectors/packet-radius.h refactor registration to delay dictionary loading as long as possible svn path=/trunk/; revision=24762
2008-03-03Fix for http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2257Jörg Mayer1-1/+1
svn path=/trunk/; revision=24533
2008-01-14This patch adds support for configuration profiles, which can be used toStig Bjørlykke1-1/+1
configure and use more than one set of preferences and configuration files. This can be found in the "Configuration Profiles..." menu item from the Edit menu, or by pressing Shift-Ctrl-A. It's also possible to start wireshark and tshark with a named profile by using the "-C ProfileName" option. A new status pane in the main window will show the current profile. The configuration files currently stored in the Profiles are: - Preferences - Capture Filters - Display Filters - Coloring Rules - Disabled Protocols - User Accessible Tables The recent data are by design not added to the profile. Planned future enhancements: - make a more convenient function to switch between profiles - add a "clone profile" button to copy an existing profile - make the profiles list active and accept return as OK - save users "Decode as" in the profile - make new, clone and deletion of profiles more secure - make some of the recent values available in the profile This patch also fixes: - setting default status pane sizes - a bug setting status pane for packets when not having main lower pane. svn path=/trunk/; revision=24089
2007-11-30use g_strconcat() which should be faster instead of g_strdup_printf()Tomas Kukosa1-5/+5
svn path=/trunk/; revision=23681
2007-09-30change dissect_radius() to become a new-style dissector that can reject Ronnie Sahlberg1-69/+123
packets that doesnt look like valid radius. verify that a packet is radius by checking that the command code is known and also that the length is between 20 and 4096 bytes move the tap data to be ep_allocated instead of a static global variable dont use tvb_memcpy() to read a structure off the wiredata use tvb_get_... to unmarshall the fields explicitely this fixes bug 1634 svn path=/trunk/; revision=23039
2007-07-13Fix bugAnders Broman1-26/+47
http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1640 change "Change-Filter-Request (43)" to "CoA-Request (43)" svn path=/trunk/; revision=22301
2007-05-29Use G_GINT64_MODIFIER, rather than the PRI[douxX]64 macros, for GLibGuy Harris1-1/+1
routines and routines using those routines. GLib might use different modifiers for 64-bit quantities than the platform's C library does. svn path=/trunk/; revision=21990