Don't guard col_set_str (COL_INFO) with col_check

svn path=/trunk/; revision=29342

Don't guard col_set_str (COL_PROTOCOL) with col_check

svn path=/trunk/; revision=29341

Don't guard col_set_str (COL_PROTOCOL) with col_check

svn path=/trunk/; revision=29340

Don't use DISSECTOR_ASSERT() to check a packet's content.  Fixes bug
3827.

svn path=/trunk/; revision=29339

Check TLV lengths before we try to add them to the tree. Fixes bug 3824.

svn path=/trunk/; revision=29338

From Bahaa Naamneh via Bug 3538:

This patch ensures that the filter autocompletion code handles protocols (radius and wimaxasncp) which uses delayed initialisation of protocols fields.

svn path=/trunk/; revision=29337

Make sure that we pass on the same set of parameters to asn2wrs.py on Unix/Windows. This fix ensures that the generated copyright headers are identical on both platforms.

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

svn path=/trunk/; revision=29336

Pass on WinPcap/MSVC redistributable executables to the WiresharkPortable launcher and not the extractor.

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

svn path=/trunk/; revision=29335

From Ivan Sy:
Fixed IPv6 malformed packet when ip6_nxt (Next Header) is equal to "59". as per section 4.7 RFC 2460.

svn path=/trunk/; revision=29334

Note that the new behavior matches what Wireshark does.

svn path=/trunk/; revision=29333

Move one comment before all of the code to which it refers.

Update another comment with information from the corresponding comment
in tshark.c

Move some code so that it's run only if its result is used.

Add an additional comment about logging on UN*X.

svn path=/trunk/; revision=29332

If we have a format error, don't try adding something to the
tree. Fixes bug 3822.

svn path=/trunk/; revision=29331

fuzz-test: Fix/change WIRESHARK_ABORT_ON_DISSECTOR_BUG handling:
1. If enabled: the variable must be exported to the env to take effect;
2. Upon reflection: disable this feature:
   tshark has been changed to output WARNING messages to stderr as a
    default; This means that DISSECTOR_BUGs and failed DISSECTOR_ASSERTs
     which cause WARNING log level messages will thus be output to stderr and
     thus will be detected by the fuzz-test.

svn path=/trunk/; revision=29330

From Matt P via bug 3774:

Patch utilizes techniques incorporated in the packet-ieee8023.c dissect_802_3()
routine to dynamically calculate the remaining payload length.  This is then
compared to the length included in the CCSDS header.

svn path=/trunk/; revision=29329

tshark: console_log_level preference now controls which log msgs are sent to stderr;
Note: This is a behavior change:
      Given the default value for the console_log_level preference
       tshark will now show ERROR, CRITICAL and WARNING level log messages.
      Prior to this change tshark was hardwired to show only
       ERROR and CRITICAL level messages.
      Time will tell whether this change results in too much noise
       in tshark output.

svn path=/trunk/; revision=29328

- Add an AC macro to test linker flags.
- Test whether the linker supports --as-needed and if so,
  adds it to LDFLAGS

svn path=/trunk/; revision=29327

* Tue May 26 2009 cmorve69@yahoo.es
- made it compile with --as-needed

This patch was taken from the opensuse wireshark.spec file.
No thanks go to the author and the package maintainers of
this package for not sending this upstream - it would have
made it into 1.2.0.

svn path=/trunk/; revision=29326

- Add rfc number comments on not-yet-decoded options
- Decode autoconfigure option

svn path=/trunk/; revision=29325

Back out r29323. The non fully qualified path seems to work for everyone else so don't mess with it.

svn path=/trunk/; revision=29324

Use fully qualified path to bash

svn path=/trunk/; revision=29323

Free frame data slice on file reset

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

svn path=/trunk/; revision=29322

Clean up the last two DIRTY_DISSECTOR_SRC entries.

svn path=/trunk/; revision=29321

Fix some individual cases of 'if (unsigned) < 0'

svn path=/trunk/; revision=29320

'if (curr_len <= 0)' won't work as desired when curr_len is declared as unsigned.
The fix: change to 'if ((signed)curr_len <= 0)'.
This is a bit of a hack but I think it works.
See the comment in this file (packet-gsm_a_common.h).

svn path=/trunk/; revision=29319

Those items are 32 bits or longer, so the cast should be safe; add an
additional cast in the hopes that they squelch "dereferencing
type-punned pointer" warnings.

svn path=/trunk/; revision=29318

packet-sctp.c seems to be clean (at least it is on Mac OS X).

svn path=/trunk/; revision=29317

Added new_packet_list_set_font() and initialize fonts when using
new packet list.

svn path=/trunk/; revision=29316

Get it compiling when HAVE_LIBGCRYPT is not defined.

svn path=/trunk/; revision=29315

Fix some indentation and spacing

svn path=/trunk/; revision=29314

Fix compilation warnings in a few dissectors and try moving them to the
"clean" list.

svn path=/trunk/; revision=29313

Prevent possible (benign ?) array references with an array index of -1.

svn path=/trunk/; revision=29312

Remove inadvertantly committed comments re issue yet to be fixed ...

svn path=/trunk/; revision=29311

Register ett[] subtree array only once.

svn path=/trunk/; revision=29310

Register ett[] subtree array only once.

svn path=/trunk/; revision=29309

Convert a GString to an ep_strbuf_t (and squelch a static analysis
warning in the process). Remove an unused variable.

svn path=/trunk/; revision=29308

From Gerasimos Dimitriadis:
Fixes/enhances the decoding of the Mobile Identity IE for type of identity
"TMGI and optional MBMS Session Identity".

svn path=/trunk/; revision=29307

Fix for bug 3811:
Add missing shorthand application/vnd.ms-sync besides application/vnd.ms-sync.wbxml.

svn path=/trunk/; revision=29306

From Ivan Sy:
Add support for Support for DHCID RR (RFC 4701).

svn path=/trunk/; revision=29305

Squelch a couple of Clang Static Analyzer warnings.

svn path=/trunk/; revision=29304

Show power headroom report inside its own subtree.

svn path=/trunk/; revision=29303

Fix for bug 3806:
Add presentation of Block Ack Bitmap.

svn path=/trunk/; revision=29302

Add a couple of pointer checks.

svn path=/trunk/; revision=29301

Fix typo.

svn path=/trunk/; revision=29300

Put the initializers for the elements of value_string arrays into curly
brackets, so the API checks for non-terminated value_string arrays will
not falsely think the arrays are non-terminated.

svn path=/trunk/; revision=29299

Optimize empty strings to NULL pointers.

svn path=/trunk/; revision=29298

Fix benign double subtree registration of an ett_... variable.

svn path=/trunk/; revision=29297

Minor changes:
- Move proto_register...() to the end of the file as per convention;
- Move ett[] to be local to proto_register...();
- "if (...==-1)" not req'd in proto_register...();
- #include <moduleinfo.h> not req'd;
- Remove unneeded forward declarations.

svn path=/trunk/; revision=29296

Initialize several ett_... variables to -1

svn path=/trunk/; revision=29295

Init ett_... variables to -1

svn path=/trunk/; revision=29294

Remove duplicate entry from ett[]

svn path=/trunk/; revision=29293

Clean up white space a bit - use 2-space indenting.

Put a bunch of code that does nothing unless data_val is non-null inside
"if (data_val)", and get rid of the individual tests.

svn path=/trunk/; revision=29292

Miscellaneous cleanups.

svn path=/trunk/; revision=29291

Show number of predefined bytes in info column.

svn path=/trunk/; revision=29290

Warning fix: comma at end of enum

svn path=/trunk/; revision=29289

Don't exit with an error if we can't find the svn executable.

svn path=/trunk/; revision=29288

Rework an assignment to keep static analyzers from complaining.

svn path=/trunk/; revision=29287

Remove a couple of unused variables.

svn path=/trunk/; revision=29286

Add some null checks.

svn path=/trunk/; revision=29285

More minor cleanups:
- Move proto_register to near the end of the file as per convention;
- Remove unneeded #include <epan/reassemble.h>;
- Comment out unused ett[] array and associated;

svn path=/trunk/; revision=29284

Minor cleanups:
- Comment out code which creates two dissector tables and then
   stores a preference value in each of them;
   AFAICT the tables are not accessed elsewhere;
- Comment out the no longer required prefs callback specification
   in proto_register;
- "if (... == -1)" not req'd in proto_register;
- Remove unused proto_reg_wimax function.

svn path=/trunk/; revision=29283

Various minor cleanups:
- Do all find_dissector() calls only once during proto_reg_handoff
   (not each time a sub-dissector is called when dissecting frames).
- Move proto_register & proto_reg_handoff to the end of the file as per convention;
- Move a global variables to be local;
- Change a global variable to be static;
- Remove unneeded multiple #include <string.h>
- Remove an unneeded variable and a few unneeded lines of code.

svn path=/trunk/; revision=29282

Clean up indentation somewhat.

svn path=/trunk/; revision=29281

From Gerasimos Dimitriadis:
Added decoding support for the High Layer Compatibility IE in DTAP and updated
the relevant part in Q.931. Also some minor enhancements in Q.931.

svn path=/trunk/; revision=29280

Thou shalt terminate thy value_string arrays with {0, NULL}.

svn path=/trunk/; revision=29279

Add a null pointer check. Remove some unused variables.

svn path=/trunk/; revision=29278

From Ivan Sy:
Support for Host Identity Protocol (HIP) Domain Name System (DNS) Extension.

svn path=/trunk/; revision=29277

Remove 1 commented-out #include inadvertantly committed.

svn path=/trunk/; revision=29276

Add some debug code (commented out) to check for duplicate dissector port registrations,

svn path=/trunk/; revision=29275

Reword some help text slightly.

svn path=/trunk/; revision=29274

Use find_dissector() instead of create_dissector_handle();
Fix some indentation.

svn path=/trunk/; revision=29273

Do find_dissector_table() in proto_reg_handoff().

svn path=/trunk/; revision=29272

(Trivial) Fix two typos.

svn path=/trunk/; revision=29271

Added missing packet-usb-hid.h

svn path=/trunk/; revision=29270

Handle usb_hid_item_usage_vals as range_string.

svn path=/trunk/; revision=29269

Do dissector_add() only if port non-zero; register dissector for "decode-as".

svn path=/trunk/; revision=29268

packet-stun2 registers for TCP & UDP port 3478 so packet-stun need not register for same.

svn path=/trunk/; revision=29267

Change default range from 1401,1401 to 1401 so that the port isn't double-registered.

svn path=/trunk/; revision=29266

Don't do dissector_add twice on TCP port 3689;
packet-daap requests registration on that port via http_dissector_add
 and thus packet-http does not need to do a default registration on that port.
Also: fix a typo in a comment.

svn path=/trunk/; revision=29265

[Automatic manuf and enterprise-numbers update for 2009-08-02]

svn path=/trunk/; revision=29262

Remove unneeded #include <epan/emem.h>; fix some indentation.

svn path=/trunk/; revision=29261

Add additional cypher types.

svn path=/trunk/; revision=29260

Fix for bug 3800:
Correctly track the size of Feedback Control Information in Payload specific RTCP feedback packet.

svn path=/trunk/; revision=29259

Remember Msg3 bytes and compare with Msg4 Contention Resolution bodies.

svn path=/trunk/; revision=29258

Fix indentation, get rid of a comment that might no longer apply (dump
*what* as hex?  The stuff we now display as generic fields?).

svn path=/trunk/; revision=29257

From Adam Nielsen via bug 3659:

Here is a patch against SVN that implements preliminary support for USB HID
devices.  At the moment it only dissects the initial set up packets (which you
will see if you hotplug a keyboard or mouse.)

The patch also fixes a minor bug in the USB dissector code which reported
certain packets as malformed due to miscounting bytes, and it reports control
packet data and padding data in a more user-friendly manner.

svn path=/trunk/; revision=29256

Completely remove an unused variable, don't just comment it out
(especially with a C++/C99-style comment that not all compilers used to
compile Wireshark accept).

svn path=/trunk/; revision=29255

From Holger Freyther via bug 3776:

Part 1:
Rename packet-gsm_abis_ip.c to packet-gsm_ipa.c.

Part 2:
Rename to the content to match the filename.

Part 3:
Add GSM A/SCCP support to the packet-gsm_ipa.c dissector

Adjusted patch to take laforge's comment into account. Use guint16 for the
length and use the ntohs routine to convert the length.

svn path=/trunk/; revision=29254

From Ivan Sy via bug 3797:

Support for SPF RR (RFC 4408)

- this has the same format as TXT RR (see section 3.1.1) below.
Currently wireshark 1.2.1 shows this as Unknown type 99

svn path=/trunk/; revision=29253

From Ivan Sy:
Add support for SSHFP RR (RFC 4255).

svn path=/trunk/; revision=29252

Include packet_info.h before ui_utils.h since we just started passing a
packet_info structure to one of the functions.

svn path=/trunk/; revision=29251

From Ivan Sy:
NAPTR RR (RFC 3403) replacement MUST be a fully qualified domain-name.

svn path=/trunk/; revision=29250

Remove more unused variables.

svn path=/trunk/; revision=29249

Fix warnings:

the behavior of x = x++ is undefined by the ANSI C standard -
they obviously just meant x++;

get rid of unused variables.

svn path=/trunk/; revision=29248

From   swapnil:
Packet-gtpv2.c dissector extended to support more IE's.

svn path=/trunk/; revision=29247

From Kovarththanan Rajaratnam via bug 3506:

The exception throwing code in except.c/h should be annotated with
"noreturn" to indicate that they never return. Running static analysis
on Wireshark without this annotation causes a lot of false positives
since these analyzers assume that the exception handling code are
ordinary functions that will eventually return.

svn path=/trunk/; revision=29246

Remove a couple of unused variables.

svn path=/trunk/; revision=29245

Show backoff indicator value in subheader tree root.

svn path=/trunk/; revision=29244

Remove some unused variable assignments.

svn path=/trunk/; revision=29243

Remove some unused variables.

svn path=/trunk/; revision=29242

Fix a typo

svn path=/trunk/; revision=29241