Use "(void) <variable/>" to avoid unused variable warnings similar to
Qt's Q_UNUSED macro.

svn path=/trunk/; revision=54110

Fix some warnings

svn path=/trunk/; revision=54109

sock diag netlink: dissect requests, stub for 'padding dissection'.

svn path=/trunk/; revision=54108

Rework code slightly to prevent VS Code Analysis warning:
  C6326: Potential comparison of a constant with another constant

svn path=/trunk/; revision=54107

More shorten-64-to-32 warnings.

svn path=/trunk/; revision=54106

Use proto_tree_add_item() rather than fetching a string an adding it, if
possible.  If (possibly) not possible, use tvb_get_string_enc() rather
than tvb_get_unicode_string().

svn path=/trunk/; revision=54105

Fix (I think) VS Code Analyzer warnings:
 C6297: Arithmetic overflow: 32-bit value is shifted, then cast to 64-bit value.
   Results might not be an expected value

svn path=/trunk/; revision=54104

Create/use several extended value-strings;
 (Sort & remove dups from underlying value_string arrays);
Reformat a few long lines;
Add editor modelines;

svn path=/trunk/; revision=54103

Fix VS Code Analysis warning:
  C6297: Arithmetic overflow: 32-bit value is shifted, then cast to 64-bit value. \
  Results might not be an expected value

'offset++' --> 'offset += 1' for consistency;
Add/adjust whitespace;
Add editor modelines

svn path=/trunk/; revision=54102

Cast away shorten-64-to-32 warnings.

svn path=/trunk/; revision=54101

Squelch a bunch of shorten-64-to-32 warnings by parsing all of our
numeric values using strtoul and casting it the result to a guint32.
Hopefully no user or session IDs are negative or greater than 32 bits.

svn path=/trunk/; revision=54100

Fix bug found by VS Code Analysis:
 warning C6385: Invalid data: accessing 'tag_to_type', \
 the readable size is '136' bytes, but '140' might be read

svn path=/trunk/; revision=54099

Take a preemptive strike at some likely future VS Code Analyzer warnings about strings possibly not being zero-terminated.

svn path=/trunk/; revision=54098

xmpp_attr_info, xmpp_attr_info_ext: Pass pointer to hf

This should fix most of warnings:
initializer element is not computable at load time

svn path=/trunk/; revision=54097

Prevent VS Code Analyzer warning; (AFAICT: not an actual bug).

svn path=/trunk/; revision=54096

Fix [-Wmissing-prototypes]

svn path=/trunk/; revision=54095

Set Proto col & Clear Info col before fetching data from the tvb;
Almost all of the dissection code can be under 'if (tree)';
Do some whitespace changes.

svn path=/trunk/; revision=54094

Add a cast to silence an unnecessary warning.

svn path=/trunk/; revision=54093

Fix const warnings.

svn path=/trunk/; revision=54092

From Peter Paluch via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9158

Currently, the LLC dissector in packet-llc.c displays the values of DSAP IG bit
and SSAP CR bit as separate items in the proto tree. This gives an impression
that these entries are separate fields in the LLC header while in reality, they
are only the least significant bits in DSAP/SSAP octets. In addition, the
importance of these bits is relatively low in today's LLC-based protocols (they
are mostly set to 0), so having them always displayed in the proto tree is
somewhat of a luxury.

Modify the LLC dissector by having added a subtree to both DSAP and SSAP items
that displays the IG and CR bits as bits in a bitfield, and moved the display
of IG and CR bits into these subtrees.

It may seem that adding a text item instead of a FT_UINT8 value is not a
sensible approach because such item is not filterable. However, if filtering by
the entire DSAP/SSAP value (which is the typical way of filtering on SAPs), this
value is always added to the tree in its entirety and indexed by "llc.dsap" and
"llc.ssap" filter strings. If the GI or CR bit are to be matched, "llc.dsap.ig"
and "llc.ssap.cr" filter strings are available. Searching for the value of the
DSAP/SSAP & 0xFE which would be the value currently added by the
proto_tree_add_text() is not done and should not be done, as IEEE stipulates:
"An individual actual address value does not necessarily have any relationship
with a group address of the same actual address value."
(http://standards.ieee.org/develop/regauth/tut/llc.pdf) Following this
consideration, the choice of displaying the SAP "actual address" using
proto_tree_add_text() is acceptable.

svn path=/trunk/; revision=54091

rdp_field_info_t struct: Pass pointer to hf

This should fix most of errors:
  warning: initializer element is not computable at load time [enabled by default]

svn path=/trunk/; revision=54090

Fix warnings

svn path=/trunk/; revision=54089

Remove not needed ';'

svn path=/trunk/; revision=54088

Fix constiness of GET_VALSP() macro.

svn path=/trunk/; revision=54087

Allow -Wmissing-prototypes only for C

It should fix:
cc1plus: warning: command line option `-Wmissing-prototypes' is valid for Ada/C/ObjC but not for C++ [enabled by default]

(only g++ complains, clang is OK with -Wmissing-prototypes)

svn path=/trunk/; revision=54086

Fix some trivial warnings.

svn path=/trunk/; revision=54085

Use proto_tree_add_item() for UTF-16 strings when dissecting string
descriptors.

Make sure the length in string descriptors is at least 2, so that we
have a valid string or string-list length.

svn path=/trunk/; revision=54084

Fix string length typo causing unitialized-value warnings caught by valgrind
fuzzing.

svn path=/trunk/; revision=54083

It seems sscanf requires the input string to be null-terminated, even if it is
longer than the maximum possible amount to read based on the format string. For
this reason, don't use sscanf on tvb_get_ptr directly, copy and null-terminate
the bytes we want.

Fixes an uninitialized value caught by valgrind fuzzing.

svn path=/trunk/; revision=54082

Add a bunch of tooltips.

svn path=/trunk/; revision=54081

Pass the information from ATP to the conversation dissector.

svn path=/trunk/; revision=54080

It should halve warnings count in tn5250 dissector.

svn path=/trunk/; revision=54079

Fix warnings + remove some v. old comment from strutil.h

svn path=/trunk/; revision=54078

Make sure all unused bits are 0 in bacapp, fixes uninitialized memory found by
valgrind.

svn path=/trunk/; revision=54077

Rename spotlight_get_utf16_string_encoding() to
spotlight_get_utf16_string_byte_order(), and have it return
ENC_BIG_ENDIAN or ENC_LITTLE_ENDIAN if it finds a BOM and 0xFFFFFFFF if
it doesn't, to make it a bit clearer what it's doing.

Use tvb_get_string_enc() rather than tvb_get_unicode_string().

svn path=/trunk/; revision=54075

In tvb_get_unicode_string(), if the byte count is odd, ignore the last
byte.  (We should perhaps add an expert info indication in those cases.)

svn path=/trunk/; revision=54074

netlink: update

- set pinfo->p2p_dir based on portid
- check for nlmsg type in dissect_netlink_sock_diag()
- sock diag support LINUX_AF_INET6, LINUX_AF_PACKET
- naming cleanup

svn path=/trunk/; revision=54073

Change the exit status of a fuzz failure to 255 so that if you use it with xargs
the failures don't get suppressed.

svn path=/trunk/; revision=54071

Note that apparently some BACnet implementations misuse the "ANSI X3.4"
encoding for ISO 8859-1; this means that those strings won't be
correctly interpreted if they're interpreted as UTF-8.

svn path=/trunk/; revision=54069

Simplify column sizing.

svn path=/trunk/; revision=54068

Fix various issues:
- Display of header bit fields was incorrect;
- Computation of the data length was incorrect;
- Display of trailer 'indicator enable' & 'indicator' bit fields was incorrect;
- 'Display' field of certain hf[] entries was incorrect.
- Pedantic: Use ENC_BIG_ENDIAN instead of ENC_NA in certain places.

See Bug #9539

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

svn path=/trunk/; revision=54067

Replace wmem_alloc with wmem_new in a few places, and wmem_new with wmem_new0 in
a few places to avoid uninitialized memory found while fuzz-testing with
valgrind.

svn path=/trunk/; revision=54066

Return FALSE from dissect_iwarp_mpa() if data is NULL.  Note that this is the conservative/safest approach since there do appear to be some code paths that don't rely on the data.

svn path=/trunk/; revision=54065

Return FALSE from dissect_zbee_nwk_heur() if data is NULL.

svn path=/trunk/; revision=54064

Add data arg to dissect_wassp_heur() in case it's ever used.

svn path=/trunk/; revision=54063

Return FALSE from dissect_tapa_heur() if data is NULL.

svn path=/trunk/; revision=54062

Add data arg to dissect_skype_heur() so it compiles cleanly if/when SKYPE_HEUR is ever defined.

svn path=/trunk/; revision=54061

Handle NFS4_OP_ILLEGAL (and other potential array bounds errors).

svn path=/trunk/; revision=54058

Remove _U_

svn path=/trunk/; revision=54057

Add another check to radiotap-iter initialization to catch another out-of-bounds
read found while fuzzing with valgrind. Hopefully doesn't break valid captures.

svn path=/trunk/; revision=54056

If data is NULL, return FALSE.

svn path=/trunk/; revision=54055

Remove _U_

svn path=/trunk/; revision=54054

Add recent Qt additions.

svn path=/trunk/; revision=54053

Capture Interfaces dialog cleanup.

Apply layouts as needed to allow resizing. Select the interface list by
row. We might want to switch the interface list to a QTreeWidget. Today
I learned that you can make QGroupBox titles checkable. Do that where we
enable and disable groups of items. Flatten all of the group boxes.
Change the output format combo to radio buttons. There are only two
choices and we have lots of real estate available. Use a
QDialogButtonBox so Qt can size and order the buttons appropriately.

svn path=/trunk/; revision=54052

Fix Visual Studio Code Analysis warning C6011: Dereferencing NULL pointer 'returnedDevmode'

svn path=/trunk/; revision=54051

Fix Visual Studio Code Analysis warning C6001: Using uninitialized memory 'keylist'

svn path=/trunk/; revision=54050

Fix Visual Studio Code Analysis warning C6054: String 'szStr' might not be zero-terminated

svn path=/trunk/; revision=54049

Fix (presumably) a VS Code analysis warning:
 C6385: Invalid data: accessing 'dgt->out', the readable size is '15' bytes, but '18' bytes might be read

Also: Do some trivial whitespace and formatting changes.

svn path=/trunk/; revision=54048

Value_string.h:  Minor changes to comment text
value_string.c:  value_string_ext_validate() always fails on Windows
                   when called from a different DLL (i.e. a plugin).
                   So: Add #ifndef _WIN32 around the offending code.

svn path=/trunk/; revision=54047

(Trivial) remove some trailing whitespace.

svn path=/trunk/; revision=54046

From Peter Lemenkov via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9556

Cosmetic change in a LF field representation in the RTPproxy dissector

Don't display any value for LF field

Signed-off-by: Peter Lemenkov <lemenkov@gmail.com>
svn path=/trunk/; revision=54045

Fix Dead Store (Dead assignement/Dead increment) warning found by Clang

svn path=/trunk/; revision=54044

From Hauke Mehrtens via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9499

DLTS: add cipher version for OpenSSL pre 0.9.8f

OpenSSL pre 0.9.8f uses the TLS version 0x0100 and is not completely
compatible with DTLS 1.0 or 1.2. One difference is that the encrypted
pre master from TLS 1.0 does not have an own length, which is needed by
TLS and DTLS 1.0, this makes decrypting impossible. This patch makes it
possible for the code to distinguish between this OpenSSL version and
real DTLS 1.0, because they are not using the same code any more. This
is needed to fix the snakeoil-dtls test.

Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
svn path=/trunk/; revision=54043

May this silences a warning of code that will go out next week.

svn path=/trunk/; revision=54042

Try to silence warnings and add a comment for Irene to
fix these appropriately next week.

svn path=/trunk/; revision=54041

Try to silence a warning due to a temporary workaround.

svn path=/trunk/; revision=54040

From Peter Wu

ssl-utils: remove SIG, rename mode and formatting

The changes seem huge, but actually involve a few structural changes
that do not change functionality, but aim to make maintenance easier and
lines shorter. The following changes were made:

 1. Remove the "sig" field and `SIG_*` constants as they are not used
    anywhere.
 2. Convert `SSL_CIPHER_MODE_*` macros to an enum, change the type in
    SslCipherSuite, change the field terminator in cipher_suites and
    drop the `SSL_CIPHER_` prefix to make it shorter.
 3. Add whitespace to align the cipher suites and convert the numbers to
    hex to match common usage (e.g. IANA docs). Done with the awk script
    below.

AWK script that takes the lines with `,KEX_` and applies changes (3):

    #!/usr/bin/gawk -f
    BEGIN { FS="[, {]+" }
    {
        split($0, c, "}, *");
        comment="";if(c[2])comment="   "c[2];
        sub("}", "", $10); # comment }
        printf("    {0x%04X,%-12s%-16s%2d,%3d,%3d,%-11s %-22s},%s\n",
               strtonum($2),
               $3 ",", # Key exchange
               # $4 is SIG_ - remove
               $5 ",", # Cipher
               $6, # blocksize
               $7, # keysize
               $8, # export keysize
               $9 ",", # Digest
               $10, # mode
               comment);
    }

Signed-off-by: Peter Wu <lekensteyn@gmail.com>
svn path=/trunk/; revision=54039

Add Copyright and modelines (Fix also indent to use 4 spaces)

svn path=/trunk/; revision=54038

Fix typo

svn path=/trunk/; revision=54037

Fix typo.

svn path=/trunk/; revision=54036

Fix gboolean bool conflict.

svn path=/trunk/; revision=54035

Fix build when use Qt4

ui/qt/sctp_chunk_statistics_dialog.cpp:16:40: error: ‘class QHeaderView’ has no member named ‘setSectionsClickable’
ui/qt/sctp_chunk_statistics_dialog.cpp:17:40: error: ‘class QHeaderView’ has no member named ‘setSectionsMovable’

svn path=/trunk/; revision=54034

Add sctp files to Makefile.am

svn path=/trunk/; revision=54033

Address more Windows errors.

svn path=/trunk/; revision=54032

Address Windows warning

svn path=/trunk/; revision=54031

"Fix"
/home/jmayer/work/wireshark/svn/trunk/ui/qt/sctp_graph_byte_dialog.cpp:36:13: error: variable ‘minBytes’ set but not used [-Werror=unused-but-set-variable]
     guint32 minBytes, maxBytes;
             ^
cc1plus: all warnings being treated as errors

svn path=/trunk/; revision=54030

"Fix"  'set but not used' warning turning error.

svn path=/trunk/; revision=54029

Address Windows warning.

svn path=/trunk/; revision=54028

Take sctp_stat.c off the list because it has been removed.

svn path=/trunk/; revision=54027

Add dialogs and graphs to analyse SCTP behavior similar to the GTK version.

svn path=/trunk/; revision=54026

Fix [-Wmissing-prototypes]

svn path=/trunk/; revision=54025

Try to fix building of captype

svn path=/trunk/; revision=54024

Add a file that probably should have been a part of
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=54015

svn path=/trunk/; revision=54023

Don't use optind, we don't actually call getopt or friends and we don't include
the necessary headers on all platforms to have optind defined. Just check that
we got arguments in addition to the executable name.

svn path=/trunk/; revision=54022

Add what appears to be a missing #include for windows.

svn path=/trunk/; revision=54020

(Try to) fix VS Clang Analysis warning:
C6297: Arithmetic overflow: 32-bit value is shifted, then cast to 64-bit value. Results might not be an expected value

svn path=/trunk/; revision=54019

And another one.

svn path=/trunk/; revision=54018

Add some more file types misidentified as VWR files.

svn path=/trunk/; revision=54017

Fix command name.

svn path=/trunk/; revision=54016

Add a "captype" file that just reports the type of a capture file, based
on what libwiretap thinks it is.

Update some comments to reflect the death of the hack used to include
(libwiretap) plugin support in programs not built with libwireshark.

svn path=/trunk/; revision=54015

Fix packet-lapd.c:186: warning: comparison between signed and unsigned.

svn path=/trunk/; revision=54014

Fix 4 instances of Visual Studio Code Analysis warnings C6001: Using uninitialized memory 'pduType2', 'numberCapabilities' (2x), and 'pduType'

svn path=/trunk/; revision=54013

The IxVeriWave heuristic now gets more false positives (after, I
suspect, the change to handle VWR files with no packets); shuffle it
after all the types we've seen misidentified as VWR files.

svn path=/trunk/; revision=54012

Let's see if this fixes VS Code Analysis warnings:
  C6297: Arithmetic overflow: 32-bit value is shifted, then cast to 64-bit value. \
                              Results might not be an expected value

svn path=/trunk/; revision=54011

Fix Visual Studio Code Analysis warning C6204: Possible buffer overrun in call to 'memcpy': use of unchecked parameter 'data_len'

svn path=/trunk/; revision=54010

Try to appease Visual Studio Code Analysis to avoid a false warning (as far as I can tell), namely: warning C6054: String 'forbidden' might not be zero-terminated

svn path=/trunk/; revision=54009

Fix Visual Studio Code Analysis warning C6001: Using uninitialized memory 'obj_type'

svn path=/trunk/; revision=54008

Switch the cherry-pick merge strategy to "recursive + theirs", which
seems to pull in fewer changes and be less prone to conflitcs. This was
the case for r53086, at least.  Add "-x" which adds a "cherry picked
from..." line to the commit message.

Show a diff before pushing.

svn path=/trunk/; revision=54007

netlink: update

- add netlink socket diag,
- dissect socket diag meminfo structures,
- add prototypes.

svn path=/trunk/; revision=54006

Do one or more of the following:
- Convert "4 space tabs" to spaces;
- Remove some unneeded initializers;
- 'offset++' --> 'offset += 1' for consistency;
- Reformat hf[] entries;
- Rework/add whitespace;
- Adjust editor modelines (set tab-width to 8).

svn path=/trunk/; revision=54005