aboutsummaryrefslogtreecommitdiffstats
path: root/epan/dissectors/packet-acn.c
AgeCommit message (Collapse)AuthorFilesLines
2015-01-20acn: col_...() shouldn't be called under 'if (tree)'Bill Meier1-34/+28
Change-Id: I4d7d1826391b0ac12ef6ad184e7c79d279db5e28 Also: remove some dead initializers. Reviewed-on: https://code.wireshark.org/review/6688 Reviewed-by: Bill Meier <wmeier@newsguy.com>
2015-01-08guid_to_ep_str -> guid_to_strMichael Mann1-3/+3
guid_to_str now uses wmem allocation. Change-Id: I8e48d1a720942fbefbaa6227ae0929cb9f856359 Reviewed-on: https://code.wireshark.org/review/6391 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2015-01-07Use tvb_address_to_str as an optimizationMichael Mann1-12/+3
Change-Id: Ic79d8bdc58b84087531b298b3b27273c1afccd4f Reviewed-on: https://code.wireshark.org/review/6357 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-12-22First batch of unincludes. Last attempt...Martin Mathieson1-1/+0
Change-Id: I3681462aeb98ca62ed3ec5eb226b2553317391a9 Reviewed-on: https://code.wireshark.org/review/5997 Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com> Reviewed-by: Martin Mathieson <martin.r.mathieson@googlemail.com>
2014-12-21Cleanup use of #includes in non-generated epan/dissector/*.cBill Meier1-2/+0
Specifically: - Set packet.h to be the first wireshark #include after config.h and "system" #includes. packet.h added as an #include in some cases when missing. - Remove some #includes included (directly/indirectly) in packet.h. E.g., glib.h. (Done only for those files including packet.h). - As needed, move "system" #includes to be after config.h and before wireshark #includes. - Rework various #include file specifications for consistency. - Misc. Change-Id: Ifaa1a14b50b69fbad38ea4838a49dfe595c54c95 Reviewed-on: https://code.wireshark.org/review/5923 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Bill Meier <wmeier@newsguy.com>
2014-09-11Eliminate proto_tree_add_text from some dissectors.Michael Mann1-24/+64
Change-Id: Ie41ac30f8edaeee75b23717ef1d5147d804a65b3 Reviewed-on: https://code.wireshark.org/review/4074 Reviewed-by: Michael Mann <mmann78@netscape.net> Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-09-08Convert some deprecated tvb_length callsEvan Huus1-2/+2
Change-Id: I6337f8e02a41106a363b9bf767bc4b3697442d41 Reviewed-on: https://code.wireshark.org/review/4027 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-06-22convert to proto_tree_add_subtree[_format]Michael Mann1-17/+13
Change-Id: I358bfaa4e5d40cd01b766f614f8bd0dbaf611dd0 Reviewed-on: https://code.wireshark.org/review/2508 Reviewed-by: Evan Huus <eapache@gmail.com> Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-06-20Rename dissector_add_handle() to dissector_add_for_decode_as().Guy Harris1-1/+1
Hopefully that name makes it clear what the routiner's purpose is, and will encourage people to use it rather than using dissector_add_uint() with a bogus integer value. Change-Id: Ic5be456d0ad40b176aab01712ab7b13aed5de2a8 Reviewed-on: https://code.wireshark.org/review/2483 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-06-19Revert "Fixup: tvb_* -> tvb_captured"Michael Mann1-2/+2
https://www.wireshark.org/lists/wireshark-dev/201406/msg00131.html This reverts commit 246fe2ca4c67d8c98caa84e2f57694f6322e2f96. Change-Id: Ib24bae0198c13a84bd7f731bf4af921212109a8f Reviewed-on: https://code.wireshark.org/review/2430 Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-06-18Fixup: tvb_* -> tvb_capturedDario Lombardo1-2/+2
Change-Id: I9209c1271967405c34c1b6fa43e1726a4d3a5a3f Reviewed-on: https://code.wireshark.org/review/2377 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-04-06Convert ep_address_to_str -> address_to_str(wmem_packet_scope, ...) for all ↵Michael Mann1-3/+3
dissectors Change-Id: I7489e2fb3a1f2630ca17b0a5fe1aa873992f1061 Reviewed-on: https://code.wireshark.org/review/975 Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-03-04Remove all $Id$ from top of fileAlexis La Goutte1-2/+0
(Using sed : sed -i '/^ \* \$Id\$/,+1 d') Fix manually some typo (in export_object_dicom.c and crc16-plain.c) Change-Id: I4c1ae68d1c4afeace8cb195b53c715cf9e1227a8 Reviewed-on: https://code.wireshark.org/review/497 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2013-12-21Add missing includesJakub Zawadzki1-0/+1
svn path=/trunk/; revision=54332
2013-12-19Rename more to_str functions to have ep_ in the name if they return ephemeralEvan Huus1-3/+3
buffers. Remove two unused functions. svn path=/trunk/; revision=54250
2013-11-21Constify a character array;Bill Meier1-2/+16
Fix a typo; Add editor modelines. svn path=/trunk/; revision=53480
2013-03-12- [-Wmissing-prototypes]Anders Broman1-0/+4
- explicit casts. svn path=/trunk/; revision=48265
2013-02-26it's ==> its & its ==> it's as needed.Bill Meier1-1/+1
svn path=/trunk/; revision=47891
2013-02-16From Alyssa Milburn:Jaap Keuter1-2/+2
This patch adds a check for a zero count to the existing sanity check code. From me: In addition drop superfluous sanity check. svn path=/trunk/; revision=47692
2013-01-31Comment out unused hf[] entries & etc.Bill Meier1-3/+9
(found by checkhf) svn path=/trunk/; revision=47389
2012-12-26OK, *this* isn't a possibly-infinite loop.Guy Harris1-3/+3
svn path=/trunk/; revision=46755
2012-12-26Widen some loop variables, in the hopes that it'll squelch some loopGuy Harris1-3/+3
warnings. svn path=/trunk/; revision=46754
2012-09-20We always HAVE_CONFIG_H so don't bother checking whether we have it or not.Jeff Morriss1-2/+0
svn path=/trunk/; revision=45017
2012-09-10Initial commit to support yet another method of passing data between dissectors.Jakub Zawadzki1-1/+1
Add new parameter 'data' to heur_dissector_t and new_dissector_t, for now it's always NULL svn path=/trunk/; revision=44860
2012-06-28Update Free Software Foundation address.Jakub Zawadzki1-1/+1
(COPYING will be updated in next commit) svn path=/trunk/; revision=43536
2012-05-19Fix some "warning: Value stored to '...' is never read" clang scan-build ↵Bill Meier1-2/+2
warnings. Also; Do some whitespace cleanup in a few cases. svn path=/trunk/; revision=42715
2012-05-16all:Bill Meier1-548/+685
Merge .h files into .c files since .h files unused elsewhere; Do whitespace, indentation & formatting cleanup. packet-acn.c: Fix 2 minor bugs wherein subtree not displayed in packet-details because tree variable used in proto_tree_add_text() always NULL. svn path=/trunk/; revision=42647
2012-03-02Fix some Dead Store (Dead assignement/Dead increment) Warning found by ClangAlexis La Goutte1-20/+20
svn path=/trunk/; revision=41302
2011-10-15Convert proto_tree_add_item() 'encoding' arg for field types FT_STRING, ↵Bill Meier1-3/+3
FT_STRINGZ, FT_UINT_STRING as follows: 1. If there's no character encoding (ENC_ASCII, ...) specified then use ENC_ASCII. 2. For all but FT_UINT_STRING, always use ENC_NA (replacing any existing True/1/FALSE/0 /ENC_BIG_ENDIAN/ENC_LITTLE_ENDIAN). svn path=/trunk/; revision=39426
2011-10-10 Convert 'encoding' parameter of certain proto_tree_add_item() calls in ↵Bill Meier1-1/+1
non-autogenerated epan/dissectors: Specifically: Replace FALSE|0 and TRUE|1 by ENC_BIG_ENDIAN|ENC_LITTLE_ENDIAN as the encoding parameter for proto_tree_add_item() calls which directly reference an item in hf[] which has a type of: FT_BOOLEAN FT_IPv4 FT_EUI64 FT_GUID FT_UINT_STRING Also: For type FT_ITv6 use ENC_NA. (This was missed in SVN #39260) svn path=/trunk/; revision=39328
2011-09-19Use ENC_NA and ENC_BIG_ENDIAN rather than FALSE in proto_tree_add_item()Guy Harris1-129/+128
calls and tvb_get_guid() calls. For strings, add ENC_UTF_8. svn path=/trunk/; revision=39044
2011-08-08Dissectors don't need to #include <gmodule.h>Bill Meier1-2/+1
Also: #include <stdlib.h> not req'd in some cases. svn path=/trunk/; revision=38409
2011-05-26From Michael Mann.Anders Broman1-2/+2
Part of "display filters with redundancies of PROTABBREV in them." The ones left outs should be fixed differently I think. Rename som ndps hf variables while at it. https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2794 svn path=/trunk/; revision=37406
2011-04-27From Yaniv Kaul: some 'set but not used' compilation fixes.Bill Meier1-14/+0
svn path=/trunk/; revision=36889
2011-04-17Removed an unneeded assignment, they are both always 0.Stig Bjørlykke1-2/+0
Found by clang. svn path=/trunk/; revision=36686
2010-05-10Indentation & whitespace cleanup (including: "4 space tabs" ==> spaces)Bill Meier1-502/+502
svn path=/trunk/; revision=32735
2010-05-06Follow up for revision 32670.Jaap Keuter1-8/+1
Straighten out is_acn() function. svn path=/trunk/; revision=32690
2010-05-05From Jim Schaettle:Anders Broman1-10/+99
Add Ratified DMX to ACN Dissector. https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4745 svn path=/trunk/; revision=32670
2010-04-06#include <string.h> not needed.Bill Meier1-1/+0
svn path=/trunk/; revision=32411
2010-03-03remove check_colAnders Broman1-18/+11
svn path=/trunk/; revision=32089
2010-01-03Fixed abbrev for PDU Length.Stig Bjørlykke1-28/+1
Removed unused hf entries. svn path=/trunk/; revision=31420
2009-09-30Rework is_acn to use tvb_memeql insted of doing a tvb_get_epemeral_string ↵Bill Meier1-13/+9
and a memcmp; Remove some unneeded calls to col_clear(). svn path=/trunk/; revision=30211
2009-09-06Rename address_to_str() to ep_address_to_str() because:Kovarththanan Rajaratnam1-4/+4
1) This indicates that the string has ephemeral lifetime 2) More consistent with its existing seasonal counterpart, se_address_to_str(). svn path=/trunk/; revision=29747
2009-08-09Don't guard col_set_str (COL_INFO/COL_PROTOCOL) with col_checkKovarththanan Rajaratnam1-3/+1
svn path=/trunk/; revision=29345
2009-06-18From Kovarththanan Rajaratnam via bug 3548:Stig Bjørlykke1-2/+2
(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-03-18Use g_strreverse() to reverse strings.Guy Harris1-16/+1
svn path=/trunk/; revision=27783
2008-11-23From Bill Florac (bug 3048):Stig Bjørlykke1-15/+16
Calling proto_tree_add_int_format() with a unsigned value can cause dissection failure. Correctly switched to proto_tree_add_uint_format(). svn path=/trunk/; revision=26827
2008-10-31Fix some "format not a string literal and no format arguments" warnings.Stig Bjørlykke1-5/+5
svn path=/trunk/; revision=26646
2008-10-31Thou shalt not "print" a string with a printf-like function by using itGuy Harris1-5/+5
as the format string; thou shalt, instead, use "%s" as the format string and use the string as the argument to "%s". Thanks and a tip of the Hatlo hat to the version of GCC that comes with Ubuntu 8.10 for catching this. svn path=/trunk/; revision=26640
2008-09-26Minor cleanup related to proto_register & proto_reg_handoffBill Meier1-8/+6
svn path=/trunk/; revision=26280
generated by cgit v1.2.3 (git 2.25.1) at 2024-09-01 05:22:56 +0000