Commit Graph

18 Commits

Author SHA1 Message Date
Alexis La Goutte dfec5ef599 Make test.sh from buildbot Happy
svn path=/trunk/; revision=51692
2013-09-03 06:31:53 +00:00
Michael Mann 5405b34469 Batch of filterable expert info.
svn path=/trunk/; revision=51688
2013-09-02 23:32:31 +00:00
Evan Huus 3398ae2aea Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8809
The LLRP Standard 1.0.1 defines the ProtocolID Parameter as 8 bit value (see
LLRP Standard 1.0.1 document, page 138, AccessSpecParameter) but Wireshark
treats it as 16 bit value and therefore doesn't recognize the
EPCGlobalClass1Gen2 protocol type and marks the whole packet afterwards as
invalid.

svn path=/trunk/; revision=49991
2013-06-17 22:22:17 +00:00
Jaap Keuter a62cc81d4b Now only if I can get the case right :)
svn path=/trunk/; revision=48686
2013-04-01 15:33:42 +00:00
Jaap Keuter 97136126bc Typo's. That is all.
svn path=/trunk/; revision=48685
2013-04-01 15:29:49 +00:00
Evan Huus 37600a157b Rename value string (and similar) functions to use a consistent pattern. This
was done using textual search+replace, not anything syntax-aware, so presumably
it got most comments as well (except where there were typos).

Use a consistent coding style, and make proper use of the WS_DLL_* defines.

Group the functions appropriately in the header.

I ended up getting rid of most of the explanatory comments since many of them
duplicated what was in the value_string.c file (and were out of sync with the
recent updates I made to those in r48633). Presumably most of the comments
should be in the .h file not the .c file, but there's enough churn ahead that
it's not worth fixing yet.

Part of https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8467

svn path=/trunk/; revision=48634
2013-03-29 00:26:23 +00:00
Jeff Morriss 2552c750e5 We always HAVE_CONFIG_H so don't bother checking whether we have it or not.
svn path=/trunk/; revision=45017
2012-09-20 02:03:38 +00:00
Evan Huus 7ad756e9c2 From Martin Kupec via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7658
Manually expand some of the macros in packet-llrp.c that were only being used
in one place. Makes for a more traditional set of hf_ registrations.

svn path=/trunk/; revision=44817
2012-09-09 01:39:27 +00:00
Evan Huus cb5a320d22 Fix a dead store and a few duplicate 'break' statements as caught by
clang and cppcheck.

svn path=/trunk/; revision=44687
2012-08-28 02:08:42 +00:00
Evan Huus f3ca6e55ef Remove variadic macro that accidentally made it in a while back.
svn path=/trunk/; revision=44686
2012-08-28 02:01:25 +00:00
Jeff Morriss 64fa3d4036 Don't call dissect_custom_message() when it's not initialized.
svn path=/trunk/; revision=44646
2012-08-23 19:07:24 +00:00
Guy Harris e835108dda Get rid of unnecessary function pointer - just directly call the
function in the switch statement.  This keeps us from calling through an
uninitialized pointer for custome parameter numbers other than
LLRP_VENDOR_IMPINJ (as warned of by at least some compilers).

svn path=/trunk/; revision=44624
2012-08-23 04:45:56 +00:00
Evan Huus 272a95a9d6 From Martin Kupec via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7658
Major enhancements to the LLRP dissector.

svn path=/trunk/; revision=44621
2012-08-23 03:03:54 +00:00
Bill Meier f3fe778851 'tab-width/tabstop/tabSize' in editor modelines should really always be 8;
Also:
- In one case #include <epan/prefs.h> not needed;
- Do some minor whitespace reformatting.

svn path=/trunk/; revision=44537
2012-08-16 13:17:48 +00:00
Bill Meier 0c8e6e8f90 Remove unneeded #include;
Create/use extended value strings for several value-string arrays;
Minor whitespace cleanup.

svn path=/trunk/; revision=43960
2012-07-24 14:47:19 +00:00
Bill Meier 1b17369504 Fix: code under 'if(tree)' (in)directly calls sub-dissector/col_...()/expert...() fcns
svn path=/trunk/; revision=43226
2012-06-12 21:22:46 +00:00
pascal 47d57de0b1 Remove most blurbs (they do not have any added value) and fix a few typo errors
svn path=/trunk/; revision=42387
2012-05-02 15:29:50 +00:00
Anders Broman feab4f6bab From Evan Huus:
Given the problems with the original attempt, and the fact that there's a new
version of the protocol spec out (v1.1), I took a crack at writing a new
dissector from scratch. It doesn't decode the fields within the message
parameters (there are far too many to bother with for an initial draft), but it
decodes everything else.

Even though it's not complete, I feel it's worth checking in as an intermediate
step (assuming it passes review), since it's still far better than nothing, and
adding full parameter-field decoding is going to take a lot of time simply for
transcribing all the different fields.

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

svn path=/trunk/; revision=42383
2012-05-02 11:05:35 +00:00