Commit Graph

15032 Commits

Author SHA1 Message Date
Michael Tüxen ad619e3a04 Decode the SI value in DATA messages correclty.
This fixes bug 5834.

svn path=/trunk/; revision=36691
2011-04-17 19:37:46 +00:00
Stig Bjørlykke 4f809a7e67 One more unused argument.
svn path=/trunk/; revision=36690
2011-04-17 19:17:46 +00:00
Stig Bjørlykke 2a2dd7655c Removed unneeded assignments.
Found by clang.

svn path=/trunk/; revision=36689
2011-04-17 19:07:17 +00:00
Stig Bjørlykke d6e7705c96 Mark unused arguments with _U_.
svn path=/trunk/; revision=36688
2011-04-17 19:04:54 +00:00
Stig Bjørlykke 95447e23f8 Removed unneeded assignments to self.
Found by clang.

svn path=/trunk/; revision=36687
2011-04-17 19:01:26 +00:00
Stig Bjørlykke a3391d1a95 Removed an unneeded assignment, they are both always 0.
Found by clang.

svn path=/trunk/; revision=36686
2011-04-17 18:59:17 +00:00
Stig Bjørlykke 3c53c25a0c Corrected equality check in frag_equal().
Found by clang.

svn path=/trunk/; revision=36685
2011-04-17 18:49:58 +00:00
Stig Bjørlykke 722dba0495 Check for valid docsis_handle, and use data_handle if not.
This fixes bug 5839.

svn path=/trunk/; revision=36681
2011-04-17 17:47:39 +00:00
Alexis La Goutte df3293c9d4 Add QoS Capability element (46) and Extended Supported Rates element (50)
Enhance RSN Info (48), Extended Capabilities information element (127)

* Replace proto_tree_add_uint*/boolean/text by proto_tree_add_item
* use ENC_* macro for encoding

svn path=/trunk/; revision=36680
2011-04-17 17:20:58 +00:00
Gerald Combs b05a8ef84e [Automatic manuf, services and enterprise-numbers update for 2011-04-17]
svn path=/trunk/; revision=36677
2011-04-17 15:03:50 +00:00
Martin Mathieson d4e9304fd2 Add a way for configuration protocol (e.g. RRC) to configure MAC-> RLC mappings, and add a preference to use this or the static UAT.
API is not yet called by RRC.

svn path=/trunk/; revision=36676
2011-04-17 11:22:03 +00:00
Alexis La Goutte 5f6a579854 Using the routing subtree (Miss in my last patch)
Fix a typo error (shoubl => should)

svn path=/trunk/; revision=36674
2011-04-17 08:58:15 +00:00
Gerald Combs 2d2109c0c7 Remove another unneeded NULL check. Fixes Coverity CID 605.
svn path=/trunk/; revision=36673
2011-04-16 22:02:14 +00:00
Gerald Combs 43b089db87 Fix Coverity CIDs 603 & 604.
svn path=/trunk/; revision=36672
2011-04-16 21:46:15 +00:00
Gerald Combs ed25109219 Fix the warnings in Coverity CIDs 606-608.
svn path=/trunk/; revision=36671
2011-04-16 21:38:21 +00:00
Gerald Combs 0ea5ded80c Add NULL checks. Fixes Coverity CIDs 509 and 510.
svn path=/trunk/; revision=36670
2011-04-16 21:25:34 +00:00
Anders Broman c63471823d Try to make the OSX buildbot happy.
svn path=/trunk/; revision=36669
2011-04-16 17:32:30 +00:00
Alexis La Goutte 9b5f811b4e Convert ICMPv6 dissector to "new-style" dissector
Fix Dead Store Warning ( warning: Value stored to '*offset*' is never read )


svn path=/trunk/; revision=36668
2011-04-16 17:14:39 +00:00
Gerald Combs 033366bda6 From Edwin Groothuis via bug 5813:
In the explanation of TCP Option 78 (Riverbed Transparency), the labels
are "CSH IP Addr/Port" and "SSH IP Addr/Port". This should be "Src SH IP
Addr/Port" and "Dst SH IP Addr/Port".
The filter keys for these labels are correct.

svn path=/trunk/; revision=36667
2011-04-16 16:45:38 +00:00
Stig Bjørlykke 1bc2077b3e Initialize feedback_data_len to avoid a compiler warning.
svn path=/trunk/; revision=36666
2011-04-16 16:37:53 +00:00
Anders Broman 05172d4476 From Dirk:
The attachted patch cleans up the output of DCE/RPC fragment information in the
dissector.
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5772

From me get rid of check_col()

svn path=/trunk/; revision=36665
2011-04-16 16:00:02 +00:00
Gerald Combs 76b26d9a2b Fix compilation errors.
svn path=/trunk/; revision=36664
2011-04-16 15:33:24 +00:00
Anders Broman 6a0cbfbce8 Fix:
packet-rohc.c: In function 'dissect_rohc_ir_rtp_profile_dynamic':
packet-rohc.c:285: warning: 'val_len' may be used uninitialized in this function
packet-rohc.c:186: warning: 'num_bits' may be used uninitialized in this function
packet-rohc.c:186: note: 'num_bits' was declared here

svn path=/trunk/; revision=36663
2011-04-16 12:28:03 +00:00
Anders Broman ffae0f912d From Neil Piercy:
Adds BMC protocol, including adding support for MAC and RLC CTCH channels to carry it.
From me:
Removed hf blurbs = def and removed check_col added tp CMakeList.

svn path=/trunk/; revision=36662
2011-04-16 12:22:16 +00:00
Anders Broman 214b9c3ad0 From Motonori Shindo:
Current NetFlow V9/IPFIX dissector treats IN_BYTES (IE=1) and
IN_PERMANENT_BYTES (IE=85) exactly in the same way. The same applies to IN_PKTS
(IE=2) and IN_PERMANENT_PKTS (IE=86). However, IN_BYTES/IN_PKTS and
IN_PERMANENT_BYTES/IN_PERMANENT_PKTS have different semantics so they should be
distinguishable when they are displayed or specified in a filter. Please find
attached the patch
which does that.
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5807

svn path=/trunk/; revision=36661
2011-04-16 12:03:50 +00:00
Anders Broman 8c98248308 Add rohc
svn path=/trunk/; revision=36660
2011-04-16 11:49:51 +00:00
Anders Broman 0a1437bfac Add a basic ROCH dissector.
Limitations:
- Small CID
- RTP profile
- IPv4

svn path=/trunk/; revision=36659
2011-04-16 11:44:05 +00:00
Martin Mathieson 960376a8f3 Show max UEs/TTI in MAC stats window (UL/DL separately).
svn path=/trunk/; revision=36658
2011-04-16 10:56:18 +00:00
Martin Mathieson ae149510e6 Keep track of how many UL/DL UEs in a TTI, and attach this info to each UL-SCH/DL-SCH frame..
svn path=/trunk/; revision=36657
2011-04-15 23:32:44 +00:00
Stephen Fisher 2a028a16ba From Edwin Groothuis via bug #5833: Make number of bits in Frame header
optional


svn path=/trunk/; revision=36652
2011-04-15 17:53:23 +00:00
Martin Mathieson bdf271e79e Remove more unnecessary assignments. Coverity 1022 (again).
svn path=/trunk/; revision=36649
2011-04-15 12:53:20 +00:00
Chris Maynard 3e6bd6b558 From Andrew Feren via bug 5774: Make V9TEMPLATE_MAX_FIELDS a user preference.
From me: Use lower-case v9template_max_fields instead of upper-case to avoid
any confusion with that variable being a define.  Use STRINGIFY() so we always
keep the default and the displayed default the same.  Fix bug introduced by
Andrew's patch where option_scope_field_count was inadvertently changed to
option_field_count.  Append "Maximum value can be adjusted ..." message to all
relevant expert infos.

svn path=/trunk/; revision=36643
2011-04-14 16:17:09 +00:00
Chris Maynard 75deae992a Revert r24047. Fixes bug 5822.
svn path=/trunk/; revision=36642
2011-04-14 13:58:40 +00:00
Alexis La Goutte 12dae38bb7 Update to last IANA parameters (2011-04-08)
Update to last RPL Draft (19)
Add new ICMPv6 Option: Proxy Signature (PS) draft-ietf-csi-proxy-send-05
Fix 2 Clang Warning (unused variable):
packet-icmpv6.c:2755:9: warning: Value stored to 'oplength' is never read
        oplength = tvb_get_guint8(tvb, rr_offset);
packet-icmpv6.c:2916:5: warning: Value stored to 'nb_mcast_records' is never read
    nb_mcast_records = tvb_get_ntohs(tvb, mldr_offset);

svn path=/trunk/; revision=36641
2011-04-14 09:40:07 +00:00
Guy Harris c0e5dc8489 Zero is zero is zero, as Gertrude Stein didn't say.
svn path=/trunk/; revision=36639
2011-04-14 06:28:25 +00:00
Anders Broman 312ad4e850 From Mathieu Parent:
- use 4-bytes hex for messages ids
- reorder skinny messages ids
- remove obsolete C-comments about status
- rewrite field names (from camel case to plain words)

From me:
Replaced meaningless hf blurbs with NULL

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

svn path=/trunk/; revision=36638
2011-04-14 06:07:42 +00:00
Anders Broman 7e17419e6c From Mathieu Parent:
packet-skinny:Add DialedPhoneBookMessage and DialedPhoneBookAckMessage

Replaced meaningless hf blurbs with NULL

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

svn path=/trunk/; revision=36637
2011-04-14 05:37:58 +00:00
Chris Maynard f627b9911a Via bug5774, Andrew Feren requested V9TEMPLATE_MAX_FIELDS be increased to 100.
svn path=/trunk/; revision=36633
2011-04-14 02:08:19 +00:00
Bill Meier b9784b4f21 From Solomon Peachy:Fix "Missing indication that we recognize the Radiotap MCS header field."
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5827

The radiotap header has a pile of optional fields, and uses a bitmask field to
indicate which fields are present.  Wireshark marks the fields it decodes, and
then decodes the payload.

The recently-added MCS field is decoded now, but the bit is not displayed as
recognized. This was presumably an oversight, as this bit is recognized and the
field decoded properly.

svn path=/trunk/; revision=36632
2011-04-13 22:07:53 +00:00
Bill Meier 9b36f03fc9 Don't assign to a proto_item * if the value won't be used: Coverity 1054-1056.
svn path=/trunk/; revision=36631
2011-04-13 21:27:08 +00:00
Bill Meier 8c4a4c09b9 Comment out code to create a subtree since it's (currently ?) unused: Coverity 1039;
Don't assign to a proto_item * if the value won't be used: Coverity 1040 & 1041.
Remove some unneede #includes;
Do some minor indentation & whitespace cleanup.

svn path=/trunk/; revision=36630
2011-04-13 21:00:44 +00:00
Chris Maynard 39013df13f Allow TCP segment data to be filterable via "tcp.data".
svn path=/trunk/; revision=36629
2011-04-13 18:15:41 +00:00
Gerald Combs dd2f52fd3c Update to reflect recent changes in epan/strutil.[ch].
svn path=/trunk/; revision=36628
2011-04-13 17:53:28 +00:00
Stephen Fisher 9246c41703 Change ws_strdup_escape_underscore() function to be more general, by
accepting any character as the escaped character.  Change existing
uses to use '_' for the underscore escaping.


svn path=/trunk/; revision=36627
2011-04-13 16:56:24 +00:00
Alexis La Goutte f163f3fd8c Add Tag Quiet (40) and IBSS DFS (41)
Enhance Tag ERP Information (42), TS Delay (43), TCLAS Processing (44)

* Replace proto_tree_add_uint*/boolean/text by proto_tree_add_item
* use ENC_* macro for encoding


svn path=/trunk/; revision=36625
2011-04-13 15:59:22 +00:00
Stephen Fisher fcfcfa347c Rollback r36614 and change dissect_SMUX_PDUs_PDU() to use previously
unused smux_tree instead of tree.  Per suggestion from Stig on -dev.


svn path=/trunk/; revision=36624
2011-04-13 15:48:24 +00:00
Jörg Mayer 0c484c8cf9 + Fix bug listed in CID 1182:
Move test out of switch statement.
 Remove unused variable.

Remove trailing whitespace.

Whitespace in that file is a mess.

svn path=/trunk/; revision=36621
2011-04-13 15:16:41 +00:00
Stephen Fisher 4b416d92d0 Fix Coverity 524: NEGATIVE_RETURNS
Coverity sees that two calls to tvb_length_remaining() with an offset
of 0 may return -1 (for an "out of bounds" condition), which would lead
to an assignment of -1 to an unsigned variable and then passing it
to functions further down that cannot accept negative values.

Use tvb_ensure_length_remaining() to throw an exception in a case of
out of bounds access (though unlikely with an offset of 0) instead of
tvb_length_remaining(), which would return -1.


svn path=/trunk/; revision=36615
2011-04-12 22:23:03 +00:00
Stephen Fisher ce2eea5b95 Fix Coverity 1047: UNUSED_VALUE in dissect_smux
svn path=/trunk/; revision=36614
2011-04-12 22:12:05 +00:00
Stig Bjørlykke e98f5fc23d Cleanup ProcessError().
Coverity 865.

svn path=/trunk/; revision=36610
2011-04-12 21:00:26 +00:00