From Porus Mehta:
Wireshark's RADIUS dictionaries for Starent need an update.

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

svn path=/trunk/; revision=42896

From Deepti Ragha:
Additions to the ARP dissector as per the IANA specifications for ARP

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

svn path=/trunk/; revision=42895

From Aditya Ambadkar and Diana Chris:
Enhancement to LDP dissector to support changes proposed in RFC6391 - flow aware transport of PW over an MPLS PSN.

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

svn path=/trunk/; revision=42894

From Evan Huus:
Fix to
BACapp item length causes Buildbot crash output: randpkt-2011-04-13-1899.pcap

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

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

svn path=/trunk/; revision=42893

From Evan Huus:
Misc resource leaks.

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

svn path=/trunk/; revision=42892

Test was backwards - if fname_new is *NON*-null, it was the name of a
temporary file to which we were writing in order to do a "safe save".
Thanks to Coverity for pointing this out - should fix CIDs 703317 and
703316.

svn path=/trunk/; revision=42891

From Erwin Rol: fix a copy-and-paste leftover.

svn path=/trunk/; revision=42890

Keep backward compatibility with UDP framing format for UMTS RLC dissector
Fix a few typo errors while we are at it

svn path=/trunk/; revision=42889

Use C/T as rbid

svn path=/trunk/; revision=42888

Use ett instead of hf variable

svn path=/trunk/; revision=42887

Fix Coverity CID's 702388 and 702422: OVERFLOW_BEFORE_WIDEN.

svn path=/trunk/; revision=42886

Set but not used variable

svn path=/trunk/; revision=42885

Remove C++ style comments initilize some variables.

svn path=/trunk/; revision=42884

While iterating over string characters don't use strlen(), but check for NUL.

svn path=/trunk/; revision=42883

Set SVN properties.

svn path=/trunk/; revision=42882

Set SVN properties.

svn path=/trunk/; revision=42881

From Erwin Rol:
Artnet, RDM and DMX dissector updates

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

svn path=/trunk/; revision=42880

Extend the switch statement.

svn path=/trunk/; revision=42879

Move defenition of rlc_mode.

svn path=/trunk/; revision=42878

Try to pacify the Ubuntu buildbot.

svn path=/trunk/; revision=42877

Enhance CHANNEL_HSDSCH dissection.

svn path=/trunk/; revision=42876

Sometimes you might have plaintext LDAP for a SASL LDAP connection if the handshake never occured.

Observed with exchange in some configurations.

From Matthieu Patou <mat@matws.net>

svn path=/trunk/; revision=42875

That doesn't work correctly if we read in a file with no interface
information, as it'll fill in whatever stuff happens to be in the
capture info.  We should somehow figure uot whether this is a capture we
just did (note: cf->is_tempfile does *NOT* necessarily tell us that, as
it's set for the result of a merge, and is cleared if we save the
capture) and fill in the interface information only if that's the case.

Put in a comment to note that we need to do that.

svn path=/trunk/; revision=42874

Correctly implement what was my intent when removing the HAVE_LIBPCAP
stuff - arrange that the interface summary list is set, if we've read a
capture file that has interface information, *regardless* of whether we
have libpcap/WinPcap or not.

That means that summary_fill_in() should fill in the interface
information for the summary if there's interface information from the
capture file, and summary_fill_in_capture() - which is called only if
HAVE_LIBPCAP is defined, and can exist only if HAVE_LIBPCAP is defined
(as it takes an argument of a type that's defined only if HAVE_LIBPCAP
is defined) - just fills in interface information from the capture
options and does so only if there's none from the file.

svn path=/trunk/; revision=42873

Interface lists currently have to be constructed in the open routine
(otherwise, nobody sees them); do so.

Fix some cases where we weren't doing the proper post-processing after
doing the heuristics for format changes that didn't involve magic-number
changes (discovered because constructing the interface list is now being
done as part of that post-processing).

Fixes bug 7287.

svn path=/trunk/; revision=42872

Don't use pcap LINKTYPE_ values in the iface_options structure, use
Wiretap encapsulation values; rename the field in question encap_type to
emphasize that.  (Code that looks at that field already assumes it's a
Wiretap encapsulation value.)

For live captures, map the LINKTYPE_ value to a Wiretap encapsulation
value.

wtap_encap_string() never returns NULL, so don't check for a null return
value.

svn path=/trunk/; revision=42871

Clean up indentation (use spaces, not tabs).

svn path=/trunk/; revision=42870

Clean up indentation (remove tabs from one structure definition).

svn path=/trunk/; revision=42869

Clean up indentation (use spaces, not tabs).

svn path=/trunk/; revision=42868

From Evan Huus in bug 7295:

cppcheck realized that if_info is known not to be null in that code
path, and therefore that checking whether it's null in that code path is
unnecessary.  Remove it.

svn path=/trunk/; revision=42867

From Evan Huus from bug 7295:

Fix an ancient copy-and-pasteo of mine ("me" here meaning Guy Harris,
not Evan Huus) - remove an unused data structure (used in the code I
copied and pasted to make this code, not used here).

svn path=/trunk/; revision=42866

vwr_read_rec_header()'s return value is used only to distinguish success
(read a record header) from failure (got an EOF or an error).  Make it
just return a Boolean.

If it fails in vwr_read(), don't overwrite *err_info (yes,
vwr_read_rec_header() might have set *err_info, so don't lose - and
leak! - the value it returned) - trust vwr_read_rec_header(), or the
routines it calls, to have set it.  (If there's a code path where that
doesn't happen, that code path needs to be fixed; the setting of
*err_info in vwr_read() should *not* be restored.)

Thanks to Evan Huus for finding a useless variable with cppcheck, and
reporting it in bug 7295, provoking me to look at this.

svn path=/trunk/; revision=42865

[Automatic manuf, services and enterprise-numbers update for 2012-05-27]

svn path=/trunk/; revision=42862

- Update cmake, gnutls and pixman to current version
- unpack an xz file with xzcat instead of gzcat
- remove some trailing whitespace
- the update of gnutls required two changes in the gnutls
  build stuff: a) gnutls.pc is now generated from gnutls.pc.in
  and b) we now need to build with --without-p11-kit

svn path=/trunk/; revision=42861

I already got rid of that variable - it just mirrors cf->filename.

svn path=/trunk/; revision=42860

We can't save a live capture file with a ws_rename() on Windows, as we
have the file open.  Go back to doing it with a copy on Windows.

Explain what the problem is, and give a way in which we might be able to
make it work on Windows (without using any NT native API calls...).

svn path=/trunk/; revision=42859

See if this fixes the include problem.

svn path=/trunk/; revision=42857

Fix some build problems.

svn path=/trunk/; revision=42856

Use GTK+'s GtkMessageDialog for the questions we ask in the process of
saving files, and run it modal (which we're already doing with the
GtkFileChooserDialog); this means less callback-based state machine
stuff, simplifying the code paths a bit.

If we're saving a file before closing it, don't bother reloading it
after saving it.

svn path=/trunk/; revision=42855

On a read error, always return a pointer to the merge_in_file_t for the
file from which we got the error.

On a successful read, always clear out err - wtap_read() doesn't set *err
on success.

svn path=/trunk/; revision=42854

Add a null check to fix Coverity CID 280210. It doesn't look like we'd
run into this in the wild.

svn path=/trunk/; revision=42853

Well, "ui/gtk/capture_file_dlg.h" is needed on Win32.

svn path=/trunk/; revision=42851

ui/gtk/capture_file_dlg.h doesn't need to be included in these files.

svn path=/trunk/; revision=42850

Fix Coverity CID 700177 Unused pointer value.

svn path=/trunk/; revision=42849

The rest of the fix for Coverity CID's 280507 and 280508.  (There were multiple occurrences grouped into the 2 CID's, but I hadn't noticed them all.)

svn path=/trunk/; revision=42848

tvb_length_remaining() can return a negative number, so be sure to handle it.  Fixes Coverity CID's 280507 and 280508.

svn path=/trunk/; revision=42847

In ns_hrtime2nsec(), cast "val" to guint64 to fix Coverity CID 702389 Unintentional integer overflow.

svn path=/trunk/; revision=42846

Add a "fallthrough" comment to avoid a missing break report from Coverity.  Fixes Coverity CID 702382 Missing break in switch.

svn path=/trunk/; revision=42845

In cap_pipe_open_live(), be sure to call ws_close(fd) before returning due to a couple of error condtions.  Fixes Coverity CID 702395 Resource leak.

svn path=/trunk/; revision=42844

Fix Coverity CID 701035: Dereference after null check of hfinfo.

svn path=/trunk/; revision=42843

Fix Coverity CID 700201: Dereference after null check, by ensuring "decoder" is not NULL before passing it to ssl_decrypt_record().

svn path=/trunk/; revision=42842

If either create_persconffile_dir() or ws_fopen() fail, then bail out.  This avoids potentially dereferencing "daf" if it's NULL and fixes Coverity CID 281275.

svn path=/trunk/; revision=42841

tvb_length_remaining() can return a negative number, so be sure to handle it.  Fixes Coverity CID 281367.

svn path=/trunk/; revision=42840

tvb_length_remaining() can return a negative number, so be sure to handle it.  Fixes Coverity CID's 280233-280235.

svn path=/trunk/; revision=42839

Update asn1-generated dissector.

svn path=/trunk/; revision=42838

tvb_reported_length_remaining() can return a negative number.  In dissect_gadu_gadu_userlist80_compressed(), bail out if it does.  Fixes Coverity CID 703085.  In dissect_gadu_gadu_stringz_cp1250(), be sure to compare against its return value as being only positve rather than non-zero.  Also, fixed a couple of typos.

svn path=/trunk/; revision=42837

Don't include a Clear button in the packet- and capture-file-comment dialogs:
the user can just Ctrl-A + backspace if they want to do that.

svn path=/trunk/; revision=42836

Enable word wrapping for capture-file comments.

svn path=/trunk/; revision=42835

Get rid of the button row for the capture-file comments.  Instead, turn the
whole summary dialog into an editor-like dialog with an OK and Cancel buttons
(OK sets the new capture file comment, Cancel doesn't).

In order to keep the dialog the same regardless of the file type (and avoid
having a Cancel and OK button when there's no text field to edit), allow
users to create or edit capture-file comments even if the file type is not
PCAPNG (they can add a comment via the add/edit comment UI anyway).

Don't include a Clear button: the user can just Ctrl-A + backspace if they
want to do that.

Don't set the comment text to "[None]" if there's no comment, just leave it
blank.

Don't allow the user to create more than 1 Summary dialog at a time.

svn path=/trunk/; revision=42834

Word-wrap comments in the display.

svn path=/trunk/; revision=42833

Use cf_update_capture_comment() to save the capture comment (instead of
summary_update_comment() which is no longer necessary).
cf_update_capture_comment() has the advantage that it doesn't mark the file
as unsaved unless the comment actually changed.

svn path=/trunk/; revision=42832

Free the new comment after calling gtk_text_buffer_set_text().

svn path=/trunk/; revision=42831

cf_update_capture_comment(): free the shb_inf when we're done with it.

svn path=/trunk/; revision=42830

menu_dissector_filter(): free the action_name when we're done with it.

capture_cb(): if we strrchr() didn't find a seperator, don't use
g_strdup_printf() to format the action_name (since that would have to be
freed), just set it to the action_name.

svn path=/trunk/; revision=42829

Free the comment after calling gtk_text_buffer_set_text().

svn path=/trunk/; revision=42828

Also allow the user to edit the capture-file comment if there is one (not
only if the capture file format is PCAPNG).  This can happen if the user
does not have a PCAPNG file but has added a capture-file comment via the
add/edit capture file comment UI.

Replace some tabs with spaces and wrap a few long lines.

svn path=/trunk/; revision=42827

Followup to r42823: check for RVALS used without BASE_RANGE_STRING.

Also: identify problematic fields by hf_variable_name instead of name or
abbreviation (the variable name is sure to be unique).

svn path=/trunk/; revision=42826

Fix bug #7283

When Engine ID is 128 we should only break when we know the enterprise id.

svn path=/trunk/; revision=42825

From Stephen Donnelly:
Add frame.interface_id support for pcap DLT_ERF file format

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

svn path=/trunk/; revision=42824

From Martin Kaiser:
RVALS() needs BASE_RANGE_STRING

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

svn path=/trunk/; revision=42823

Now that cf_save_packets() can avoid doing a "fast path" save if the
source isn't compressed and the target is (or vice versa), enable the
"compressed" checkbox in the Save As and Export Specified Packets
dialog.  Fix it to clear the checkbox if the selected file format
doesn't support gzipping.

svn path=/trunk/; revision=42822

Cleanup AUTHORS file.

- Merge duplicated entries for: Dominic Bechaz, Holger Hans Peter Freyther
- Remove duplicated entry for: Shiang-Ming Huang

svn path=/trunk/; revision=42821

From Mayuresh Raut.
Addition of DDMAP TLV and Downstream FEC stack change sub-TLV to DDMAP [RFC 6424]

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

svn path=/trunk/; revision=42820

From Mayuresh Raut.
Addition of DDMAP TLV and Downstream FEC stack change sub-TLV to DDMAP [RFC 6424]

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

svn path=/trunk/; revision=42819

Keep track, in Wiretap, of whether the file is compressed, and provide
an API to fetch that.

When doing "Save" on a compressed file, write it out compressed.

In the Statistics -> Summary dialog and in capinfos, report whether the
file is gzip-compressed.

svn path=/trunk/; revision=42818

We have to tell cf_save_packets() whether to save them in compressed
form; say "don't" for now.

svn path=/trunk/; revision=42817

In the "Save As" and "Export Specified Packets" code path, do a "safe
save" if the destination file exists.

Don't forbid overwriting an existing file in either of those cases (we
still forbid overwriting the current capture file) - the GUI asks the
user whether they want to do the overwrite, and allows them to cancel
out of it - and don't remove the file before writing to it (doing so
makes the save *un*safe).

Attempt to do a save of an unedited temporary file by just moving the
file on Windows as well as on UN*X - ws_rename() will remove the target
if necessary on Windows (and won't do it as a separate operation before
attempting the rename), so it behaves like ws_rename() on UN*X (which is
just a wrapper around rename()).

svn path=/trunk/; revision=42816

From Martin Kaiser via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7275 :
iso7816: dissect ATR (answer to reset)

svn path=/trunk/; revision=42814

Shrink the height of the "Save As" dialog, now that it no longer has
widgets to specify a range.

svn path=/trunk/; revision=42813

Fix bug #7277

Check tvb length before fetching data.

svn path=/trunk/; revision=42812

Dissect MAC and RLC for CHANNEL_FACH_FDD

svn path=/trunk/; revision=42811

Add conv_id.h

svn path=/trunk/; revision=42810

Add file missing from anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=42806

svn path=/trunk/; revision=42809

From Florent Drouin:
(Jeff Morriss, Michael Mann )
Add absolute timestamps to -z io,stat

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

svn path=/trunk/; revision=42808

From Stephen Donnelly:
Add frame.interface_id support for ERF file format

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

svn path=/trunk/; revision=42807

From Robert Bullen:
The Wireshark and tshark TCP conversations stats tables aggregate reused connections into a single line item

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

svn path=/trunk/; revision=42806

Fix a typo

svn path=/trunk/; revision=42805

From Jim Wright:
Enhance the DTN dissector so that it understands ECOS blocks (Extended Class of Service)

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

svn path=/trunk/; revision=42804

Fix comparison of file extension with extensions for the file type -
compare extension with extension, not dot-plus-extension with extension.

Skip unnecesary test.

Clean up white space.

svn path=/trunk/; revision=42803

   All:
   - whitespace cleanup;
   - Remove redundant code;

   packet-ymsg.c:
   - Check if bytes available before doing tvb fetch for heuristic check;
   - Required tcp pdu "fixed part length" is 10 (not 8);
   - Use val_to_str_const() as appropriate;
   - Remove some unneeded initializations.

svn path=/trunk/; revision=42802

From Evan Huus: "Fix Potential padding issues in Yahoo dissectors"
   https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7276

svn path=/trunk/; revision=42801

In Windows, in the Save As and Export Selected Packets dialog, append
the default extension for the file type iff

the file type we're using has a list of extensions;

the file has no extension or it has one but it's not one of the
ones in the list.

*Don't* expect a file extension to be at most 5 characters plus the dot
- the extension for pcap-ng, our default capture file type, is "pcapng",
and that's 6 characters!

svn path=/trunk/; revision=42800

No packet range stuff for the "Save As" dialog.

svn path=/trunk/; revision=42799

Fix Coverity CID 703086: Dereference after null check in decode_tcp_ports().

svn path=/trunk/; revision=42798

Fix: packet-gsm_a_gm.c:3965:6: error: comparison is always true due to limited range of data type [-Werror=type-limits]

svn path=/trunk/; revision=42797

Suppress a (legitimate) compiler warning for now, and leave a comment
indicating that we should do *something* about nanoseconds values >= 1
billion.

svn path=/trunk/; revision=42796

Separate callback to control the file type combo box in the "Export
Specified Packets" dialog.

svn path=/trunk/; revision=42795

Do the "Save As always saves the entire capture, Export Specified
Packets saves specified packets" stuff for Windows.

svn path=/trunk/; revision=42794

nflog dissector:

 - dissect NFULA_TIMESTAMP as 16B time.
 - fix byte_order_types (selecting BE selected little endian)

svn path=/trunk/; revision=42793

"Save As" always saves everything and, when the save is done, makes the
new file the current file, as is the case in most if not all other GUI
applications.

A new "Export Specified Packets" menu option allows you to specify which
packets to write out, with the default being the displayed packets (and
those on which the displayed packets depend for, e.g.  reassembly), and
never makes the resulting file the current file.

The two operations are conceptually distinct.  Lumping them into one
menu item, with the default for "Save As" being "displayed packets only"
and thus making it behave like the latter operation, was causing some
confusion; see, for example, bug 6640.

Make the dialog popped up if you try to "Save As" or "Export Specified
Packets" on top of an existing file ask the "do you want to do this?"
question in the main part of the message, and note in the secondary text
that doing that will overwrite what's in the file; that matches what
TextEdit on OS X and the GNOME text editor say.

svn path=/trunk/; revision=42792