Commit graph

9951 commits

Author SHA1 Message Date
Guy Harris
7d2d5048ec If we don't have any of the packet padding in the tvbuff, don't even try
to add the trailer - that avoids the problem with throwing an exception
inside a CATCH* clause for that same exception.  (XXX - is this a
problem with our exception mechanism? What happens in languages with
built-in exceptions if a

	try
		XXX
	catch YYY
		ZZZ
	end

construct throws exception YYY inside code block ZZZ?  If it doesn't
re-enter ZZZ, that argues that our exception mechanism is buggy or, at
least, violates the Principle of Least Surprise.)

Move the code to put the trailer information into the protocol tree out
of the CATCH2 clause, so the trailer shows up if we *do* have the
trailer information.

svn path=/trunk/; revision=9898
2004-01-28 20:09:44 +00:00
Guy Harris
d2e4e9ecc9 Fix a comment.
svn path=/trunk/; revision=9897
2004-01-28 10:37:14 +00:00
Guy Harris
a92d9edf0f Clean up the comparison code a bit.
svn path=/trunk/; revision=9896
2004-01-28 10:19:36 +00:00
Guy Harris
a66012b68c For columns whose data comes from the frame_data structure for a packet,
sort the columns based on that data, rather than on the raw text of the
column - that's probably faster, and also doesn't get rudely surprised
by "atof()" being locale-dependent.

svn path=/trunk/; revision=9895
2004-01-28 09:53:09 +00:00
Guy Harris
a06e94af42 "gtk/main.c" no longer has any code to manage the packet list, so it no
longer needs to define the "column_arrows" structure.

svn path=/trunk/; revision=9894
2004-01-28 09:10:50 +00:00
Gerald Combs
fa5b2237f9 Catch any exceptions that add_ethernet_trailer() throws in add_dix_trailer().
Otherwise, they're caught by ethertype() which results in an infinite loop.

svn path=/trunk/; revision=9893
2004-01-28 03:36:37 +00:00
Guy Harris
677db2dadd From Tomas Kukosa: reserve private WTAP_ENCAP values and map them to
reserved private DLT_ values.

svn path=/trunk/; revision=9892
2004-01-28 01:24:30 +00:00
Ulf Lamping
bd0c381cfd "Export Selected Packet Bytes" -> change all appearances to this text,
don't show up this dialog if no data *is* selected (didn't find an easy way
to control sensitivity of the menus by some mechanism :-(

svn path=/trunk/; revision=9891
2004-01-27 20:58:19 +00:00
Guy Harris
b179725c39 Make "{byte,tree}_view_select()" return "gboolean", not "gint", as they
return TRUE or FALSE as a success/failure indication.

svn path=/trunk/; revision=9890
2004-01-27 20:36:48 +00:00
Guy Harris
cdc2d3d318 From Alastair Maw: the IAX2 type is unsigned, not signed.
svn path=/trunk/; revision=9889
2004-01-27 20:25:06 +00:00
Guy Harris
2a2c7f6c07 Update Andy Hood's e-mail address.
svn path=/trunk/; revision=9888
2004-01-27 20:13:31 +00:00
Ulf Lamping
aa34344c3c always select the current item under cursor, before showing up the
corresponding context menu (right mouse click)

svn path=/trunk/; revision=9887
2004-01-27 19:46:52 +00:00
Olivier Biot
6dd857b59c From Lars Ruoff:
- use create_tempfile() to create the temporary files (causes temp files to be
  created at the same location as other Ethereal temp files). Changed temp
  filenames to "ether_rtp_dXXXXXX", where d is the stream direction.
- fixed a bug with temp files not being removed

svn path=/trunk/; revision=9886
2004-01-27 18:05:32 +00:00
Olivier Biot
8b8db29111 From Chris Wilson: add missing initialization of CL-WSP dissector handle.
As proposed by some postings on Ethereal-Dev, remove the email addresses
from the authors and refer to the AUTHORS file or the man page.

svn path=/trunk/; revision=9885
2004-01-27 17:17:31 +00:00
Guy Harris
2528c053ce Supply a pseudo-header for all 802.11 packets; add an "fcs_len" field to
it, similar to the Ethernet pseudo-header's "fcs_len" field, and use it
in the 802.11 dissector.

svn path=/trunk/; revision=9884
2004-01-27 08:06:12 +00:00
Guy Harris
70e3d8a097 Add a detail-pane context menu item for "Export As Raw Packet Data", so
you can select something and export it without shoving the rodent up to
the menu bar.

svn path=/trunk/; revision=9883
2004-01-27 05:05:58 +00:00
Guy Harris
83cafdc1d6 For EtherNet/IP-over-TCP, handle multiple PDUs in a TCP segment and PDUs
split across segment boundaries.

svn path=/trunk/; revision=9882
2004-01-27 04:43:35 +00:00
Guy Harris
90b9aa5b64 Update Jaime Fournier's address.
svn path=/trunk/; revision=9881
2004-01-27 04:24:41 +00:00
Guy Harris
5e6ae703c7 From Jaime Fournier: update his e-mail address.
svn path=/trunk/; revision=9880
2004-01-27 04:15:49 +00:00
Guy Harris
a31ced6662 Map SMPP and smpp-gsm-sms preferences to their new homes in gsm-sms-ud.
svn path=/trunk/; revision=9879
2004-01-27 04:11:48 +00:00
Guy Harris
2e56c9ad21 Update from Jaime Fournier.
svn path=/trunk/; revision=9878
2004-01-27 03:55:24 +00:00
Guy Harris
99b9c9d72e From David Richards: don't stuff the packet length into a guint8, as it
can be bigger than 255.

svn path=/trunk/; revision=9877
2004-01-27 03:49:35 +00:00
Guy Harris
f97ed617de Clear the Info column.
svn path=/trunk/; revision=9876
2004-01-27 01:43:41 +00:00
Guy Harris
580a8706eb From Alastair Maw: IAX2 support.
svn path=/trunk/; revision=9875
2004-01-27 01:35:25 +00:00
Olivier Biot
13147843ba Add a correct RCS ID (starts with capitalized "i").
svn path=/trunk/; revision=9874
2004-01-27 00:30:46 +00:00
Olivier Biot
dad66aaa99 Get rid of a debug printf statement.
svn path=/trunk/; revision=9873
2004-01-27 00:26:24 +00:00
Olivier Biot
4293724ef5 From Chris Wilson:
Separate the GSM SMS User Data dissector from the SMPP dissector.

	Rename the protocol.

svn path=/trunk/; revision=9872
2004-01-27 00:20:37 +00:00
Olivier Biot
f63fd700f8 Add application/smil to the registered line-based text media dissector.
svn path=/trunk/; revision=9871
2004-01-26 23:49:43 +00:00
Guy Harris
0694894f57 Rename "dissect_t38_tcp()" to "dissect_t38_tcp_pdu()" to make it
clearer what it does (dissect a single T.38-over-TCP or T.38-over-TPKT
PDU).

Have "dissect_t38_udp()" that does T.38-over-UDP dissection and
"dissect_t38_tcp()" that does T.38-over-TCP dissection; register handles
for them for the appropriate protocols, rather than having a single
dissector that checks the IP protocol type.

Use -1 rather than "tvb_length(tvb)" as the length for top-level
protocol tree items.

Clean up indentation.

svn path=/trunk/; revision=9870
2004-01-26 22:52:22 +00:00
Guy Harris
5633166fbe Newlines in filter strings are a Bad Idea, as they don't display well in
the title or contents of the filtering progress bar window and don't
work very well in files such as "recent" or "dfilters".

svn path=/trunk/; revision=9869
2004-01-26 22:34:55 +00:00
Olivier Biot
0c81a75ed3 From Martin Regner:
- support both T.38IFP/TCP and T38IFP/TPKT/TCP

There is one new preference setting tpkt_usage where you can specify if TPKT
header is used. With the default setting (tpkt_usage=Maybe) the dissector
will look on the first four octets and try to determine if TPKT is used or
not. This setting seems to work well with the captures I have tried,
but there might be some scenarios where it will be better to set the value
to 'Always' or 'Never' dependant on if TPKT header is used or not.

There is also a new preference setting regarding TCP desegmentation. TCP
desegmentation is currently only supported when TPKT header
is used.
You will need to set the preference setting "Allow subdisectors to desegment
TCP streams" for TCP and also the "Desegment all TPKT
messages spanning multiple TCP segments" for TPKT if you want to use this
feature. I don't have any T.38 captures with segmented packets
so I have not been able to test this so much.

- Support for T.38 (2002) ASN.1 specification
http://www.itu.int/ITU-T/asn1/database/itu-t/t/t38/2003/T38(2002).html
 Note: This ASN.1 specification is incompatible with the Pre-Corigendum T.38
ASN.1 specification (1998) so you will need to set the preference setting
accordingly.

- When there are extra octets after the T.38 UDPTL packet you will now see
[Malformed?] in the Info-column. I thought that it
was quite irritating that non-T.38 packets could look like they were
correctly coded T.38 packets.

- There might be several T.38 IFP packets in a TCP packet for the T.38
directly over TCP scenario. There is now at least some support for this, but
since
TCP desegmentation is only supported for the TPKT scenario this will not
work if an IFP packet is splitted into several TCP packets.

svn path=/trunk/; revision=9868
2004-01-26 22:16:43 +00:00
Guy Harris
1c8f8924ac Fix a compiler warning.
svn path=/trunk/; revision=9867
2004-01-26 21:26:52 +00:00
Ulf Lamping
c6201489da updated NSIS generation to build GTK version 1 or 2,
added new target "packaging" to root Makefile.nmake,
which will generate version 1 or 2 or both,
depending on config.nmake GTK?_DIR setting

svn path=/trunk/; revision=9866
2004-01-26 21:12:41 +00:00
Olivier Biot
a165b0d6be Add support for the "matches" operator if HAVE_LIBPCRE is defined.
svn path=/trunk/; revision=9865
2004-01-26 21:02:36 +00:00
Guy Harris
1f078d0fff Add a dissector table for LAPD SAPIs, and move the #defines for them
into "lapd_sapi.h".  Use that to register the Q.931 dissector atop LAPD.

From Rolf Fiedler: ISDN TEI management frame support.

svn path=/trunk/; revision=9864
2004-01-26 20:48:39 +00:00
Olivier Biot
3b1f5f17b3 From Lars Ruoff:
- made streams list sortable by column
- added a button "Find Reverse" that finds the reverse stream to the
  selected forward stream. (If different reverse streams available,
  pushing the button repeatedly will cycle through all of them,
  one after the other.)
- tidied up lots of superflous code.

Also get rid of a const pointer cast warning, and remove the _U_ attribute
from rtpstream_reset().

svn path=/trunk/; revision=9863
2004-01-26 19:16:30 +00:00
Ulf Lamping
359fb7d5f6 added a tooltip to explain the behaviour of the syntax check in the filter string
svn path=/trunk/; revision=9862
2004-01-26 18:51:04 +00:00
Ulf Lamping
c1a7c24df3 added a tooltip to explain the behaviour of the syntax check in the filter toolbar
svn path=/trunk/; revision=9861
2004-01-26 18:40:44 +00:00
Ulf Lamping
276930f3f9 slightly changed the menuitem names in View->Show->Packet xy
svn path=/trunk/; revision=9860
2004-01-26 18:35:56 +00:00
Guy Harris
d9a836175f On Windows OT, interface names *can* have spaces in them; to find the
interface name, look forward in the interface string for "\Device\", not
backwards for a space, on Windows.  (Continue to look backwards for a
space on UNIX; names don't start with "\Device\", or even "/dev/", and
don't have spaces in them on any UNIX-flavored OS I know of.)

svn path=/trunk/; revision=9859
2004-01-26 09:05:59 +00:00
Ulf Lamping
fdc839f11b filter_packets: don't refilter all the packets,
if the new filter string is identical to the old one (as it's useless then)

svn path=/trunk/; revision=9858
2004-01-26 06:43:00 +00:00
Guy Harris
bbf3806ba7 Don't muck with the Ethernet pseudo-header if we have an 802.11 capture.
svn path=/trunk/; revision=9857
2004-01-25 23:50:48 +00:00
Guy Harris
f78b963082 Don't use "cf_read_error_message()" when reporting errors not from
Wiretap.

svn path=/trunk/; revision=9856
2004-01-25 22:27:12 +00:00
Guy Harris
d6c17f144b Free the error info string after using it.
svn path=/trunk/; revision=9855
2004-01-25 22:21:39 +00:00
Guy Harris
d20e50f996 Get rid of an unused variable.
svn path=/trunk/; revision=9854
2004-01-25 22:21:01 +00:00
Guy Harris
e608fdfda3 In "main_filter_packets()", use the "cf" argument rather than always
passing "&cfile" to "filter_packets()".

Mark the "w" argument to "filter_activate_cb()" as unused.

svn path=/trunk/; revision=9853
2004-01-25 22:20:21 +00:00
Guy Harris
d6cd61061e Have the Wiretap open, read, and seek-and-read routines return, in
addition to an error code, an error info string, for
WTAP_ERR_UNSUPPORTED, WTAP_ERR_UNSUPPORTED_ENCAP, and
WTAP_ERR_BAD_RECORD errors.  Replace the error messages logged with
"g_message()" for those errors with g_strdup()ed or g_strdup_printf()ed
strings returned as the error info string, and change the callers of
those routines to, for those errors, put the info string into the
printed message or alert box for the error.

Add messages for cases where those errors were returned without printing
an additional message.

Nobody uses the error code from "cf_read()" - "cf_read()" puts up the
alert box itself for failures; get rid of the error code, so it just
returns a success/failure indication.

Rename "file_read_error_message()" to "cf_read_error_message()", as it
handles read errors from Wiretap, and have it take an error info string
as an argument.  (That handles a lot of the work of putting the info
string into the error message.)

Make some variables in "ascend-grammar.y" static.

Check the return value of "erf_read_header()" in "erf_seek_read()".

Get rid of an unused #define in "i4btrace.c".

svn path=/trunk/; revision=9852
2004-01-25 21:55:17 +00:00
Ulf Lamping
34bddb3c1a adding a function to the help dialog, showing a specific topic (context help),
beautifying the placement of the help button in the button_row,
using the context help for the filter dialog

svn path=/trunk/; revision=9851
2004-01-25 21:27:16 +00:00
Gerald Combs
d69016e552 Delete generated files during distclean instead of maintainerclean, so
that we pass distcheck.

svn path=/trunk/; revision=9850
2004-01-25 18:52:33 +00:00
Ulf Lamping
c890e29a99 apply automatically generated filters strings to the recent dropdown list,
e.g. when applying a "Follow TCP stream",
fix a bug in the recent function, discarding the newest entries when
saving a full list (now discarding the oldest).

svn path=/trunk/; revision=9849
2004-01-25 18:51:26 +00:00