aboutsummaryrefslogtreecommitdiffstats
path: root/epan/dissectors/packet-x11.c
AgeCommit message (Collapse)AuthorFilesLines
2005-08-20remove sprintf from pgm and x11Ronnie Sahlberg1-16/+21
svn path=/trunk/; revision=15450
2005-08-14ememify packet-x11Ronnie Sahlberg1-44/+5
plug a memleak for ...].strptr that is alloced by g_malloc() but never released. svn path=/trunk/; revision=15351
2005-08-12more gmemchunk->se_alloc() improvementsRonnie Sahlberg1-9/+2
svn path=/trunk/; revision=15319
2005-06-26Get rid of the private "my_match_strval()" routine in many dissectors;Guy Harris1-3/+3
add a "match_strval_idx()" routine that does the same thing, and have "match_strval()" call it. Make those routines, and "val_to_str()", return a "const" pointer. Update dissectors as necessary to squelch compiler warnings produced by that. Use "val_to_str()" rather than using "match_strval()" and then, if the result is null, substituting a specific string. Clean up some other "match_strval()"/"val_to_str()" usages. Add a null pointer check in the NDPS dissector's "attribute_value()" routine, as it's not clear that "global_attribute_name" won't be null at that point. Make some global variables in the AFS4INT dissector local. Make some routines not used outside the module they're in static. Make some tables "static const". Clean up white space. Fix Gerald's address in some files. svn path=/trunk/; revision=14786
2005-06-26packet-x11.c(1780) : warning C4090: 'function' : different 'const' qualifiersUlf Lamping1-2/+2
packet-x11.c(1780) : warning C4022: 'g_tree_insert' : pointer mismatch for actual parameter 3 packet-x11.c(2209) : warning C4090: 'initializing' : different 'const' qualifiers svn path=/trunk/; revision=14781
2005-06-10just to get things straight: a dissector should *never* do any g_assert() calls!Ulf Lamping1-2/+2
in a simple approach, I've replaced all g_assert() and g_assert_not_reached() calls by their exception throwing counterparts DISSECTOR_ASSERT() and DISSECTOR_ASSERT_NOT_REACHED() this will replace application crash by showing a dissector bug, which is the desired behaviour there were some g_assert calls in the protocol registering functions, which might not be acting as expected now, but to be able to simply search for g_assert in the future I've replaced that calls too one g_assert remained, the one when someone throws an unknown exception "into" packet_frame.c, but IMHO this one should remain. svn path=/trunk/; revision=14608
2005-03-24Keep a list of all the x11_conv_data_t structures, so that when we'reGuy Harris1-21/+33
freeing our data structures, we can go through all those structures and free up the hash tables associated with them, rather than just leaking them. svn path=/trunk/; revision=13891
2005-02-02From Jon Ringle:Anders Broman1-4/+4
1) Added a setup_frame parameter to conversation_t 2) Used the conversation_t next to maintain a list of conversations with the same src/dest tuple but different setup_frame number. 3) Changed the signature of find_conversation() and conversation_new() to pass in the frame number. 4) Adjusted packet-sdp to select RTP conversation if both m=audio and m=image are present, and T.38 conversation if only m=image is present. I expect that RTP/T.38 dissecting to be better, but I don't have a way to generate T.38 packets. svn path=/trunk/; revision=13243
2005-01-16The keycode_first and keycode_count arguments to "listOfKeysyms()" canGuy Harris1-4/+8
come directly from packet data, so don't abort if they define a sequence of keycodes that goes past 255 - just report errors for the keycodes past 255. Clean up indentation. svn path=/trunk/; revision=13059
2005-01-16Revert to the previous formatting of the item.Guy Harris1-7/+3
svn path=/trunk/; revision=13058
2005-01-16Display the keycode list for a modifier a bit differently (don't listGuy Harris1-12/+12
the modifier name for every keycode, just give the modifier name once and then show all the keycodes), and use "proto_item_append_text()" rather than building a string. Clean up white space. svn path=/trunk/; revision=13057
2004-12-25Again, some warnings removed.Ulf Lamping1-4/+4
svn path=/trunk/; revision=12834
2004-12-25a lot of warnings removed, most of them about pointer to int casts without ↵Ulf Lamping1-10/+10
using the appropriate GLib macros svn path=/trunk/; revision=12832
2004-09-27Move prefs.c and prefs.h into the epan subdirectory.Guy Harris1-1/+1
svn path=/trunk/; revision=12115
2004-09-17IP addresses are always big-endian.Guy Harris1-1/+2
svn path=/trunk/; revision=12027
2004-08-21Prevent the word "desegmentation" at the GUI, but use reassembling at that ↵Ulf Lamping1-2/+3
places, as we also use "defragmentation" and "reassembling". Use reassembling as the general term for such actions. I (hopefully) didn't changed any protocol fields or preference file names, but only the GUI labels appearing in the protocol display and the protocol preferences. Also added a note to the protocol preferences (where appropriate), that you have to enable "Allow subdissectors to reassemble TCP streams" at the corresponding protocol settings for TCP reassembling to take effect. If you encounter any mistakes I've made here, please let me know... svn path=/trunk/; revision=11784
2004-07-18Move dissectors to epan/dissectors directory.Gilbert Ramirez1-0/+5382
Also move ncp222.py, x11-fields, process-x11-fields.pl, make-reg-dotc, and make-reg-dotc.py. Adjust #include lines in files that include packet-*.h files. svn path=/trunk/; revision=11410