Commit Graph

2332 Commits

Author SHA1 Message Date
Ronnie Sahlberg 02f562359e remove one array from the stack
dissect one more bitfield properly
get rid of some strcpy()



svn path=/trunk/; revision=16176
2005-10-10 09:12:33 +00:00
Ronnie Sahlberg 6d67bce2e5 dissect a bitfield properly and get rid of some strcpy()
svn path=/trunk/; revision=16175
2005-10-10 08:36:24 +00:00
Ronnie Sahlberg bc655dfba3 improve get_dns_name now that we have cheap emem allocated buffers
and get rid of a few strcpy()
a whole lot of arrays from the stack
and make the function prototype slightly nicer.



svn path=/trunk/; revision=16166
2005-10-08 12:31:06 +00:00
Ronnie Sahlberg 9a0ebcc75c remove some redundant strcpy() and arrays on the stack
svn path=/trunk/; revision=16164
2005-10-08 11:37:00 +00:00
Ronnie Sahlberg 21ec37f287 strcpy() removal
svn path=/trunk/; revision=16163
2005-10-08 11:15:15 +00:00
Ronnie Sahlberg 6e6952badf low hanging fruit
remove two redundant strcpy()



svn path=/trunk/; revision=16162
2005-10-08 11:07:48 +00:00
Ronnie Sahlberg 0437e92843 fix the last remaining bitmap so it is dissected properly and remove the last strcpy from els
svn path=/trunk/; revision=16161
2005-10-08 10:24:31 +00:00
Ronnie Sahlberg f0709d0a7a one more bitmap dissected properly and a few less strcpy()
svn path=/trunk/; revision=16160
2005-10-08 09:52:47 +00:00
Ronnie Sahlberg ef5b9193cd one more bitfield properly dissected and some less strcpy()s
svn path=/trunk/; revision=16159
2005-10-08 09:11:23 +00:00
Ronnie Sahlberg f0b1d734c5 some more strcpy() removals
and one more bitfield dissected properly

only 157 strcpy()'s remain in epan/dissectors now   down from 300+ in 0.10.12



svn path=/trunk/; revision=16158
2005-10-08 08:49:53 +00:00
Ronnie Sahlberg ccaded007c From William Fiveash
Support for PA_ENCTYPE_INFO2
and defines for new aes crypto



svn path=/trunk/; revision=16156
2005-10-07 22:51:25 +00:00
Luis Ontanon f35f5e317e do not free the element name, as its been passed to the hf array.
svn path=/trunk/; revision=16154
2005-10-07 17:59:13 +00:00
Luis Ontanon f1b498587b remove location from dtd_build_data_t avoiding to (double) free it.
svn path=/trunk/; revision=16147
2005-10-06 22:01:52 +00:00
Anders Broman 5ddbeb2cb8 From Rene Pilz,
In the bssgp an IE was decoded as mobile identity and should be decoded as (p)tmsi only.

The patch is attached to this email. It also consists the new atm patch which was send yesterday.


svn path=/trunk/; revision=16146
2005-10-06 20:01:25 +00:00
Anders Broman 2a80fe283c Handle multiple messages in a TCP segment, reassembly still needed.
svn path=/trunk/; revision=16142
2005-10-06 16:00:26 +00:00
Anders Broman f3d6542e0f Octest string of (size 2) does not dissplay correctly.
svn path=/trunk/; revision=16141
2005-10-06 08:53:51 +00:00
Anders Broman 15aa4d692f From Richard van der Hoff:
Extensions to the h.245 dissector to extract H.223 Multiplex Code and Logical 
Channel setup messages, and pass this information to the H.223 dissector.

svn path=/trunk/; revision=16139
2005-10-06 06:38:17 +00:00
Guy Harris ba324831a0 Don't try to strip the last byte from a zero-byte OID. Fixes bug 503.
svn path=/trunk/; revision=16132
2005-10-05 21:41:12 +00:00
Jörg Mayer 402c4dd9a7 Don't reinvent the wheel, use val_to_str/match_strval
svn path=/trunk/; revision=16130
2005-10-05 15:44:04 +00:00
Ronnie Sahlberg 0a3ed5250a add the licence file for the samba4 copied files
svn path=/trunk/; revision=16128
2005-10-05 13:39:44 +00:00
Ronnie Sahlberg 6c1b9760f6 dissect one more bitfield
get rid of some strcpy()
get rid of a temporary string


svn path=/trunk/; revision=16127
2005-10-05 13:25:18 +00:00
Ronnie Sahlberg c858b89670 add dissection of another bitfield and remove some strcpy()
svn path=/trunk/; revision=16126
2005-10-05 12:42:34 +00:00
Ronnie Sahlberg 59cf2dad3a rewrite dissection of service options flags to proper booleans and get rid of some strcpy() calls.
svn path=/trunk/; revision=16125
2005-10-05 11:55:49 +00:00
Ulf Lamping 6b3cacb197 bugfix: don't crash, if not enough bytes of the header available
svn path=/trunk/; revision=16124
2005-10-05 06:37:35 +00:00
Guy Harris dd42bfef7c Get rid of an unused variable.
svn path=/trunk/; revision=16122
2005-10-04 22:08:20 +00:00
Anders Broman 24a1fd561d From Richard van der Hoff
I've spotted a bug in the new desegmentation in iax2, which 
means that higher-level PDUs aren't dissected when they ought to be, but 
tend to be grouped together in later packets - I've attached a patch 
against svn r16110 to fix this; it also makes the code a bit clearer...

svn path=/trunk/; revision=16121
2005-10-04 20:21:18 +00:00
Guy Harris 399bf74c47 Even a null object has a non-zero length (for the BER identifier and
length); update the offset when dissecting one.

svn path=/trunk/; revision=16118
2005-10-04 18:21:04 +00:00
Ronnie Sahlberg e1549f4f18 the t38 tap would only initialize the t38_info structure when called from UDP
and when called from TCP this pointer would be uninitialized and thus dump core when dereferenced.

at the same time remove the rotating buffers


this fixes bugs 498 and 501

svn path=/trunk/; revision=16117
2005-10-04 14:41:39 +00:00
Ronnie Sahlberg 4e4f6d67fa in svn 15335 the tcp analysis was changed to do its stuff and to populate (prepend to) COL_INFO before callking the subdissectors
instead of calling the tcp analysis (and prepend colingo) eitehr after the subdissector returned normally  or if an exception caused by a subdissector was rised.

this as a sideffect caused tcp analysis data to be overwritten if the subdissector caused any output to the info column. (and made tcp analysis suboptimal)


this change adds a new function   col_prepend_fence_fstr()  that will prepend
the info column with the string and also, if there was no fence already defined, create a fence and set it after the prepended col info text.

This way, even if the subdissectors generate and rewrite col info, the tcp analysis data will still be displayed on the info column.



svn path=/trunk/; revision=16116
2005-10-04 13:34:52 +00:00
Guy Harris 494b806bfc Fix a typo.
svn path=/trunk/; revision=16115
2005-10-04 10:26:44 +00:00
Guy Harris bfad2f3a0c Get rid of // comments - not all C compilers like them.
svn path=/trunk/; revision=16114
2005-10-04 10:26:17 +00:00
Guy Harris 2a8e47b11a Don't ensure all the bytes of a security descriptor exist before calling
"dissect_nt_sec_desc()".  Add a Boolean argument to
"dissect_nt_sec_desc()" to indicate whether a length was passed to it
(so we don't treat -1 as a special value; we want to stop treating -1 as
a special length value, and, in fact, want to stop treating *any*
negative length values specially, so that we don't have to worry about
passing arbitrary 32-bit values from packets as lengths), and have
"dissect_nt_sec_desc()" initially create the protocol tree item for the
security descriptor with a length of "go to the end of the tvbuff", and
set the length once we're done dissecting it - and, if the length was
specified, check at *that* point, *after* we've dissected the security
descriptor, whether we have the entire security descriptor in the
tvbuff.

That means that we don't have to worry about overflows after
"dissect_nt_sec_desc()" returns - if the length was so large that we
would have gotten an overflow, we'd have thrown an exception in the
"tvb_ensure_bytes_exist()" call at the end of "dissect_nt_sec_desc()".

Do sanity checks on offsets within the security descriptor, so we know
the item referred to by the offset is after the fixed-length portion of
the descriptor.

svn path=/trunk/; revision=16113
2005-10-04 10:23:40 +00:00
Guy Harris 72dd04b43c We can just use "get_ber_identifier()" and "get_ber_length()" to fetch
BER identifier and length information; calling
"dissect_ber_identifier()" and "dissect_ber_length()" with a null tree
argument doesn't have any advantages over that.

Don't put an octet string into the protocol tree twice, once with
"dissect_ber_octet_string()" and once with the real value; once is enough.

svn path=/trunk/; revision=16112
2005-10-04 10:17:45 +00:00
Anders Broman 1f02bfa0af From John Sullivan:
The current ethereal eDonkey dissector fails to handle a number of valid packet types - in particular the normal server<->client communication between current eserver and eMule versions (from both
sides) produces packets which ethereal incorrectly claims to be malformed.

This patch fills in these holes.

svn path=/trunk/; revision=16107
2005-10-04 05:36:31 +00:00
Anders Broman 4a5c674c04 From Dave Richards
1) Fixes context-tagged BACnet errors.
2) Fixes indeterminate tag data.  If the tag length/value/type variable 
was 6 or 7 you could not tell whether it was an open/close tag or had 
length 6/7.
3) Don't make assumptions about property types for proprietary BACnet 
object types.



svn path=/trunk/; revision=16104
2005-10-04 05:15:23 +00:00
Guy Harris f34436d517 Don't assume that every packet has a non-null "call_data" value. Fixes
bug 497 (as well as a crash on an *unfuzzed* IAX2 capture!).

Set "pinfo->can_desegment" before calling dissectors; that's expected
for dissectors that do TCP-style desegmentation.

svn path=/trunk/; revision=16099
2005-10-04 01:16:06 +00:00
Anders Broman f11ef4b7a0 Add dissection of some more OID's and use the #.REGISTER function.
svn path=/trunk/; revision=16081
2005-10-03 18:18:38 +00:00
Ulf Lamping 2ee44b8409 from Yaniv Kaul: add a Preference setting for IP checksum validation
svn path=/trunk/; revision=16080
2005-10-03 16:24:43 +00:00
Ulf Lamping 376e1adf58 from Peter Johansson:
The supplied patch resolves an issue with the use of the not yet initialized variable "fid".

/ Regards, Peter 

svn path=/trunk/; revision=16079
2005-10-03 15:52:14 +00:00
Luis Ontanon 6a37b19839 dissect_rtse_EXTERNAL is already declared in the header as extern.
svn path=/trunk/; revision=16078
2005-10-03 13:10:28 +00:00
Ronnie Sahlberg 8df0e21fba prettify mount,
if MNT fails with an error,   put the error string in the INFO column


svn path=/trunk/; revision=16076
2005-10-03 09:30:10 +00:00
Anders Broman 96c01ad3c6 Fix a typo.
svn path=/trunk/; revision=16074
2005-10-03 05:59:17 +00:00
Anders Broman 15a08ae521 From Alejandro Vaquero:
New "Fax T38 Analysis" added to the "Statistics" menu to:

- Reassemble the HDLC t30 frames and dissect the header.
- Analyze the UPDTLPacket seq num for packet lost
- Stats of V.x Data:
    - Count the Data bytes
    - Duration
    - Wrong seq num
    - Max Burst of packet lost

svn path=/trunk/; revision=16073
2005-10-03 05:55:26 +00:00
Anders Broman 9e273834dc From Graeme Lunt
X420 - incorrect ExtendedBodyPart handling
ACSE - support for implicitly tagged EXTERNALs and dissection based upon direct-reference RTSE - same change for RTSE EXTERNAL handling as above CMS  - support for ContentType, MessageDigest, SigningTime and CounterSignature attributes ESS  - support for ESSSecurityLabel and EquivalentLabels attributes


svn path=/trunk/; revision=16072
2005-10-03 05:17:51 +00:00
Anders Broman 2bac3e70c5 From Graeme Lunt
allows the dissection of ContentInfo based upon it's OID - something that is used for a secure X.400 messages (Protecting Content Type (PCT)).

svn path=/trunk/; revision=16069
2005-10-02 21:14:35 +00:00
Anders Broman 093040a179 From Jeff Snyder
00-iax.diff
 Modifications to the IAX2 dissector so that it offers desegmentation to 
subdissectors using the same API as TCP offers (pinfo->desegment_len etc)

01-amr.diff
 Modifications to the AMR dissector to allow AMR IF2 data to be dissected via 
call_dissector() from packet-h223.c. This patch also causes the AMR dissector 
to append the frame type string to the info column, so that the info column 
shows what protocols an H.223 frame contains.

02-h263-data.diff
 Modifications to packet-h263.c to separate the dissection of h.263 RTP 
encpasulation from the dissection of the actual h.263 data. The data 
dissection functions are added as a second dissector. This data-only 
dissector is used to dissect the video channel in our h.223 streams. As with 
the AMR modification, this makes the H.263 dissector append to the info 
column.

svn path=/trunk/; revision=16068
2005-10-02 20:59:54 +00:00
Ulf Lamping e11c4aac89 enhancements COL_INFO output and make bittorrent protocol filterable
svn path=/trunk/; revision=16067
2005-10-02 15:41:11 +00:00
Luis Ontanon 54695fec80 propperly initialize vendor->ett if the vendor is not found while radius_register_avp_dissector().
fixes bug 488.


svn path=/trunk/; revision=16064
2005-10-01 11:16:17 +00:00
Olivier Biot 6f5e84b2d7 Fix bug 492 - the MMSE dissector calls add_content_type() defined in WSP, which
uses proto_item_append_string(). The visibility hack must be present, otherwise
a dissector assert is generated within the MMSE dissector.

svn path=/trunk/; revision=16060
2005-09-30 19:29:21 +00:00
Anders Broman 855e39b91c Update to the latest version of the standard.
svn path=/trunk/; revision=16056
2005-09-29 20:24:02 +00:00