From Alex Rodikov via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6871
TPNCP (proprietary Audiocodes) protocol dessector - wrong guint8 value presentation

The presentation of unsigned 8-bit integer is wrong. The (signed) gint8 is used which is displayed as unsigned integer (by proto_tree_add_uint) afterwards.
That causes wrong presentation of valus which bigger than 127.
Solution: New guint8 is introduced to present unsigned 8 bit integer value.

svn path=/trunk/; revision=41209

Mark the file as unsaved when updating packet comments.

svn path=/trunk/; revision=41208

More Warning fixes.

svn path=/trunk/; revision=41207

More warning fixes.

svn path=/trunk/; revision=41206

Try to fix warning: comparison between signed and unsigned.

svn path=/trunk/; revision=41205

Fix some unused parameter warnings etc.

svn path=/trunk/; revision=41204

Error: Found deprecated APIs in packet-ssl-utils.c: dissector_add

svn path=/trunk/; revision=41203

packet-applemidi.c:296:33: error: unused variable ‘p_conv_data’

svn path=/trunk/; revision=41202

From Michael Mann:
a dissector for the ODVA protocol: CIP Safety. More information
about the protocol can be found here:

http://www.odva.org/Home/ODVATECHNOLOGIES/CIP/CIPTechnologyOverview/CIPSafety/CIPSafetyOverview/tabid/131/lng/en-US/Default.aspx

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

svn path=/trunk/; revision=41201

From Tobias Erichsen:
RTP-MIDI - dissector for transmission of MIDI-commands via RTP (RFC 4695).

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

svn path=/trunk/; revision=41200

Re-apply r40728 and fix Coverity CID 1371 UNINIT again.

svn path=/trunk/; revision=41199

Mark the gile as unsaved if comments are edited

svn path=/trunk/; revision=41198

Dissect SSH over SCTP when using the IANA assigned SCTP port 22.

svn path=/trunk/; revision=41197

Move the new files to the same places as in automake.

svn path=/trunk/; revision=41196

Harald Welte <laforge@gnumonks.org>
via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6792

This is a new dissector for the non-standard Ericsson OM2000 protocol, as it is
used for the OML on A-bis of Ericsson RBS 2xxx BTSs.

It also includes a dissector for a shim-layer protocol that Ericsson uses for
IP-based A-bis like the RBS 2409.  As the protocol is not publicly documented,
I have invented the name "EHDLC" (Ericsson HLDC) for it.

svn path=/trunk/; revision=41195

Add entry for TPDU in the tree

svn path=/trunk/; revision=41194

Make it possible to edit capture comments

svn path=/trunk/; revision=41193

[Automatic manuf, services and enterprise-numbers update for 2012-02-26]

svn path=/trunk/; revision=41190

From Robin Seggelmann: Add support for RFC 6520.
From me: Some cleanup
Initial work was done by Denis Jaeger and
Lukas Scharlau, but the code got rewritten by Robin.

svn path=/trunk/; revision=41189

For LANalyzer files, make the "File type name" be the file comment.

Display the file comment in the Summary dialog.

svn path=/trunk/; revision=41188

Suppress invalid or non-meaningful statistics - for example, without
time stamps on all packets in a set, you can't determine the start and
end time of the packets in the set (even one timestampless packet throws
the determination off - was that packet before the first time-stamped or
after the last time-stamped packet, or between them?).

svn path=/trunk/; revision=41187

Don't show time stamp-based information if we don't have it (now that we
know whether packets - or "packets" - have time stamps).

Show the time stamp order as "Unknown" if we don't have enough
information to determine whether it's certainly in order or out of order
(XXX - we should do that if we have only one packet that has a time stamp).

svn path=/trunk/; revision=41186

Add a presence flag field to the packet information structure filled in
by Wiretap, to indicate whether certain fields in that structure
actually have data in them.

Use the "time stamp present" flag to omit showing time stamp information
for packets (and "packets") that don't have time stamps; don't bother
working very hard to "fake" a time stamp for data files.

Use the "interface ID present" flag to omit the interface ID for packets
that don't have an interface ID.

We don't use the "captured length, separate from packet length, present"
flag to omit the captured length; that flag might be present but equal
to the packet length, and if you want to know if a packet was cut short
by a snapshot length, comparing the values would be the way to do that.

More work is needed to have wiretap/pcapng.c properly report the flags,
e.g. reporting no time stamp being present for a Simple Packet Block.

svn path=/trunk/; revision=41185

From Jose Pedro Oliveira: pcapio is writing the if_speed value in the IDB_TSRESOL optiones

svn path=/trunk/; revision=41184

As suggested by multipleinterfaces in ask.wireshark.org/questions/9194/can-i-recover-a-discarded-capture :

Add the location of the temporary file on Windows 7.

svn path=/trunk/; revision=41183

Revisit https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3161
and r39501:

Setting _XOPEN_SOURCE to 600 is only allowed on Solaris 10 if the compiler is
set to C99 mode.  Conversely (and as reported in the bug), simply defining it
(but with no value) is not allowed if the compiler *is* compiling to C99.

So, don't define _XOPEN_SOURCE at all on Solaris.  Keep defining it as 600 on
other OS's as (also) requested in that bug.

Maybe there's a cleaner way to do this but all of this is a "trickery" mess
anyway...

svn path=/trunk/; revision=41182

g_fprintf() & g_vfprintf() should not be used....

svn path=/trunk/; revision=41181

CVARSDLL hasn't been used (is undefined) for a while....

svn path=/trunk/; revision=41180

Fix a typo and revert previous.

svn path=/trunk/; revision=41179

Fix a typo.

svn path=/trunk/; revision=41178

From Martin Kaiser:
Get rid of an unused variable.

svn path=/trunk/; revision=41177

From Irene Ruengeler: Always show snaplen.

svn path=/trunk/; revision=41176

From Irene Ruengeler: Fix a bug where the interface
names are not shown during capturing.

svn path=/trunk/; revision=41175

From Irene Ruengeler: Whitespace changes.

svn path=/trunk/; revision=41174

From Martin Kaiser via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6838
Support HDCP version 1 over I2c

the attached patch adds support for HDCP version 1. This is the authentication that runs between your DVD/Bluray player and your TV when they're connected via an HDMI cable.

svn path=/trunk/; revision=41173

From Martin Kaiser via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6838
Support HDCP version 1 over I2c

the attached patch adds support for HDCP version 1. This is the authentication that runs between your DVD/Bluray player and your TV when they're connected via an HDMI cable.

svn path=/trunk/; revision=41172

From Robert Shade: Mobile IP NVSE (RFC3115) Decoding https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6868

svn path=/trunk/; revision=41171

From Mark Cottrell: Attempting to view an RTP stream analysis graph causes program exit. https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6866

svn path=/trunk/; revision=41170

From Josip Medved: Diameter dictionary has few invalid gavp definitions https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6867

svn path=/trunk/; revision=41169

Revert SVN #40866 (which used g_fprintf() in place of fprintf()).

Using g_fprintf() fails (crashes) on Windows because the Windows GLib DLL
is linked with (depends upon) MSVCRT while editcap is linked with
(depends upon) MSVCR90.

IOW: "You can't do that ... (on Windows)"

See https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6695 (Comment 2)
for some additional information.

svn path=/trunk/; revision=41168

Add SHB info to Summary

svn path=/trunk/; revision=41167

From Pascal Quantin [NAS EPS] Update Configuration Protocol description in PCO IE https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6865

svn path=/trunk/; revision=41166

No debug output, please.

svn path=/trunk/; revision=41165

Fix the handling of snaplen.
While there fix some whitespace issues.

svn path=/trunk/; revision=41164

Have the used interfaces show up if number of interfaces > 0.
(pcapng file).

svn path=/trunk/; revision=41163

From Dirk Leinenbach:
Follow TCP IPv6 - wrong hostname.

#BACKPORT
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6861

svn path=/trunk/; revision=41162

Fix a 3GPP specification error.

svn path=/trunk/; revision=41161

As discussed with Anders: Only write a single ISB at the end. Proivde
start/end time in the ISBs.

svn path=/trunk/; revision=41160

interface id is 32-bit, not 64-bit.

svn path=/trunk/; revision=41159

From Kundok Park:
new_packet_list: crash in add_byte_views from decrypted zigbee data

The cause of the crash I saw was that the add_byte_views() function in
main_proto_draw.c relies on output from previous dissector run while the
function may eventually trigger dissector to run again which wipes out the
previous output.
The patch copies the output of the dissector before calling add_byte_tab() so
that even when add_byte_tab() updates the dissector output, the loop continues
with previous dissector output.

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

svn path=/trunk/; revision=41158

From Steve Magnani:
USB: Add dissection for Interface Association descriptor.

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

svn path=/trunk/; revision=41157

From Tyson Key:
Clean up redundant Data handlers in the CCID dissector.

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

svn path=/trunk/; revision=41156

Move 'static' qualifier before 'const' in a couple of places. I didn't
know it mattered, but gcc warns.

svn path=/trunk/; revision=41155

Mark some unused parameters. Still getting other warnings...

svn path=/trunk/; revision=41154

Do a check of the block total length in pcapng_read_unknown_block().

That means we don't need to do the block length check in
pcapng_read_block(); each block type reader, including the one for
unknown block types, does a check that's as stringent as that block
length check or more stringent, which means any block whose length is
less than the minimum will fail with the same error in both cases.

Fix the message for a too-short NRB.

svn path=/trunk/; revision=41152

From Tyson Key:
Support for switching USB CCID PC_to_RDR_XfrBlock payload dissectors

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

svn path=/trunk/; revision=41151

From Yaniv Kaul:
Enhance the Spice dissector - small additional dissections.

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

svn path=/trunk/; revision=41150

From Lei Chen:
a patch to support decode FDD_CELL_INFORMATION of "UTRAN FDD Description" in packet-gsm_rlcmac.c

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

svn path=/trunk/; revision=41149

From Jeff Morriss:
H223: Buildbot crash output: fuzz-2012-01-25-22599.pcap

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

svn path=/trunk/; revision=41148

From Josip Medved: Add 3GPP AVP 1506 and 1507

svn path=/trunk/; revision=41146

Fix typo introduced by previous checkin.

svn path=/trunk/; revision=41145

Added aplications and one AVP from Josip Medved

svn path=/trunk/; revision=41144

Add sanity checks to make sure the claimed block size is big enough to:

1) contain the block length fields and block type field;

2) contain that plus the fixed-length portion of the block;

3) for blocks that have a variable-length portion other than the
   options, contain that variable-length portion.

Fixes a crash we're seeing with a bad pcap-NG file in the Wireshark
menagerie (7799-lastPacketWithoutComment.pcapng - the last packet's
block length is 128, but it claims to have 98 bytes of packet data,
which requires a 132-byte block).

Clean up white space (use 8-space tabs).

svn path=/trunk/; revision=41143

From Josip Medved:
In line 4722 of dictionary.xml file there is comment:
<-- Requesting-Node-Type is from old (v8.1.0 - v8.2.0) versions of 29.272. -->

This is not a valid XML comment and that line should read:
<!-- Requesting-Node-Type is from old (v8.1.0 - v8.2.0) versions of 29.272. -->

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

svn path=/trunk/; revision=41141

Grr forgot debug flag  - again.

svn path=/trunk/; revision=41140

Whitespace cleanup: Convert 4 space tabs to spaces ...

svn path=/trunk/; revision=41139

Rename some enumeration constants; Do some minor code simplification & reformatting.

svn path=/trunk/; revision=41138

- Write ISB(s) at start and end of capture.
- Read all options.
- Prepare to write ISB.

svn path=/trunk/; revision=41137

Expand the API for ISB:s

svn path=/trunk/; revision=41136

Reverse Revision 41134 (Need to fix error/warning before...)

svn path=/trunk/; revision=41135

From Lars Ruoff:
Dissector for Alcatel-Lucent Enterprise Universal Alcatel- and NOE protocol
families.
Meant as a replacement for existing UA-dissector in trunk because of better
feature set:
- latest protocol specifiaction
- more detailed dissection and filtering possibilities on subprotocols
- RTP stream setup
- NOE over SIP

Lars Ruoff
On behalf of Alcatel-Lucent Enterprise

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

svn path=/trunk/; revision=41134

More debugging information - in at least one of the failures, neither of
the "Wrote xxx" messages was printed, although the compiler appeared to
have been run on the .c file that was never claimed to have been
written, but got an error because it couldn't find the .h file (also
never claimed to have been written), and in one of the successes on the
same buildbot, they were both printed for the same file.

svn path=/trunk/; revision=41133

Handle the case where there are no IDBs before the first non-SHB/non-IDB
block, which could be the case even in a *valid* file (consider a file
with an SHB, an NRB, an IDB, and a packet block, in that order); even if
there's no IDB before the first packet block, that should be reported to
the user as "interface N not less than interface count M", to more
precisely indicate the problem.

(Yes, the loop should probably keep going until it finds a packet block,
not just a non-IDB block.)

svn path=/trunk/; revision=41132

We need to build libui even if we're only building TShark.  Fixes bug
6855.

svn path=/trunk/; revision=41131

Small fix.

svn path=/trunk/; revision=41129

From Irene Ruengeler: Describe what we currently have in trunk/
                      related to capturing from multiple
                      interfaces.

svn path=/trunk/; revision=41128

From Irene Ruengeler: Updated screenshots.

svn path=/trunk/; revision=41127

Fix bugs which resulted in the buildbot errors.

svn path=/trunk/; revision=41126

Correctly write os string...

svn path=/trunk/; revision=41125

Whitespace changes.

svn path=/trunk/; revision=41124

Use wtap_dump_open_ng() in tshark.

svn path=/trunk/; revision=41123

This should fix the buildbots...

The problem was that when reading a .pcap file, we don't have any IDBs.
If reqested to write out an pcapng file, we (now) build a dummy IDB which
uses the file's encapsulation as the interface encapsulation. Therefore
it can't be per=packet.
We need to fix this by using wtap_dump_open_ng()...

svn path=/trunk/; revision=41122

When saving a file and the are no IDBs, create a default one.
This seems right, but doesn't make the buildbots happy, because
a read pcap file is reported a per packet encaps...

svn path=/trunk/; revision=41121

From Lori Tribble:
 BACnet: Allow vendor specified network layer messages.'

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

svn path=/trunk/; revision=41120

Disable debug output...

svn path=/trunk/; revision=41119

When writing an option in an IDB, also write an endofoption option.
Use (consistently) 0 as the default for the interface speed.
While there, do some whitespace cleanups.

svn path=/trunk/; revision=41118

Whitespace changes.

svn path=/trunk/; revision=41117

Whitespace changes.

svn path=/trunk/; revision=41116

Whitespace changes.

svn path=/trunk/; revision=41115

From Jose Pedro Oliveira:
dumpcap: only free os_version_str after the
libpcap_write_interface_description_block() calls.

svn path=/trunk/; revision=41114

Whitespace changes.

svn path=/trunk/; revision=41113

Improve the error handling.

svn path=/trunk/; revision=41112

Don't dereference idb_inf in wtap_dump_open_ng() when
you provide NULL when you call it via wtap_dump_open.

This does not make the buildbots happy, but at least
tshark doesn't crash anymore.

svn path=/trunk/; revision=41111

Use tab indentation as elsewhere in the files.
Only whitespace changes.

svn path=/trunk/; revision=41110

Check if the string exists first.

svn path=/trunk/; revision=41109

Extend libpcap_write_interface_description_block() wiyh more options
all exept os curently unused.

svn path=/trunk/; revision=41108

Fix compilation (hopefully).

svn path=/trunk/; revision=41107

Improve peeking for IDBs.
While there, cleanup some whitespaces.

svn path=/trunk/; revision=41106

From Irene Ruengeler:
* Add support in the GUI for pipes.
* Allow the local interfaces to be rescanned via the GUI.
* Allow remote interfaces to be added and deleted.
The GUI can be extended to support other ways of capturing.

svn path=/trunk/; revision=41105

From David Wei and me, update to 3GPP TS 25.413 V10.4.0 (2011-12).

svn path=/trunk/; revision=41104