Commit graph

2684 commits

Author SHA1 Message Date
Guy Harris
ab797734dd Get rid of the private "my_match_strval()" routine in many dissectors;
add a "match_strval_idx()" routine that does the same thing, and have
"match_strval()" call it.

Make those routines, and "val_to_str()", return a "const" pointer. 
Update dissectors as necessary to squelch compiler warnings produced by
that.

Use "val_to_str()" rather than using "match_strval()" and then, if the
result is null, substituting a specific string.  Clean up some other
"match_strval()"/"val_to_str()" usages.

Add a null pointer check in the NDPS dissector's "attribute_value()"
routine, as it's not clear that "global_attribute_name" won't be null at
that point.

Make some global variables in the AFS4INT dissector local.

Make some routines not used outside the module they're in static.

Make some tables "static const".

Clean up white space.

Fix Gerald's address in some files.

svn path=/trunk/; revision=14786
2005-06-26 19:56:52 +00:00
Guy Harris
2b07720c2f Set the Info column to the correctly-capitalized name.
Use "val_to_str()" to set the Info column, and give the "hyperscsi.cmd"
field a value_string table.

svn path=/trunk/; revision=14785
2005-06-26 19:55:06 +00:00
Ulf Lamping
6c3b559e21 value_string.c(71) : warning C4090: 'return' : different 'const' qualifiers
svn path=/trunk/; revision=14784
2005-06-26 11:40:26 +00:00
Ulf Lamping
ba1d304f7a prefs.c(1218) : warning C4761: integral size mismatch in argument; conversion supplied
prefs.c(1230) : warning C4761: integral size mismatch in argument; conversion supplied
prefs.c(1240) : warning C4761: integral size mismatch in argument; conversion supplied
prefs.c(1245) : warning C4761: integral size mismatch in argument; conversion supplied

svn path=/trunk/; revision=14783
2005-06-26 11:32:02 +00:00
Ulf Lamping
d5ae7ee8af proto.c(2941) : warning C4090: 'function' : different 'const' qualifiers
proto.c(2941) : warning C4022: 'g_tree_insert' : pointer mismatch for actual parameter 2

svn path=/trunk/; revision=14782
2005-06-26 11:29:04 +00:00
Ulf Lamping
1e0675a01b packet-x11.c(1780) : warning C4090: 'function' : different 'const' qualifiers
packet-x11.c(1780) : warning C4022: 'g_tree_insert' : pointer mismatch for actual parameter 3
packet-x11.c(2209) : warning C4090: 'initializing' : different 'const' qualifiers

svn path=/trunk/; revision=14781
2005-06-26 11:24:26 +00:00
Ulf Lamping
debc4cd849 packet-smb.c(14406) : warning C4090: 'return' : different 'const' qualifiers
svn path=/trunk/; revision=14780
2005-06-26 11:21:39 +00:00
Ulf Lamping
0265375589 packet-pgsql.c(891) : warning C4090: 'return' : different 'const' qualifiers
svn path=/trunk/; revision=14779
2005-06-26 11:18:47 +00:00
Ulf Lamping
10ffd2da48 packet-diameter.c(856) : warning C4090: 'function' : different 'const' qualifiers
packet-diameter.c(856) : warning C4024: 'addVendor' : different types for formal and actual parameter 2
packet-diameter.c(857) : warning C4090: 'function' : different 'const' qualifiers
packet-diameter.c(857) : warning C4024: 'addVendor' : different types for formal and actual parameter 3
packet-diameter.c(863) : warning C4090: 'function' : different 'const' qualifiers
packet-diameter.c(863) : warning C4024: 'addCommand' : different types for formal and actual parameter 2

svn path=/trunk/; revision=14778
2005-06-26 11:10:26 +00:00
Anders Broman
1bad9775dd Add a signal
svn path=/trunk/; revision=14777
2005-06-26 11:03:39 +00:00
Ulf Lamping
c14334db25 packet-dcerpc-samr.c(2795) : warning C4090: '=' : different 'const' qualifiers
packet-dcerpc-samr.c(2900) : warning C4090: '=' : different 'const' qualifiers
packet-dcerpc-samr.c(2933) : warning C4090: '=' : different 'const' qualifiers

svn path=/trunk/; revision=14776
2005-06-26 11:02:13 +00:00
Anders Broman
7e2903bdc8 Fix a compiler warning.
svn path=/trunk/; revision=14774
2005-06-26 11:00:19 +00:00
Ulf Lamping
a96c3b0f73 packet-ansi_map.c(760) : warning C4090: 'return' : different 'const' qualifiers
svn path=/trunk/; revision=14772
2005-06-26 10:58:32 +00:00
Ulf Lamping
ac5e6ff1c6 packet-ansi_a.c(460) : warning C4090: 'return' : different 'const' qualifiers
svn path=/trunk/; revision=14771
2005-06-26 10:55:34 +00:00
Ulf Lamping
d2081016d5 packet-ansi_801.c(93) : warning C4090: 'return' : different 'const' qualifiers
svn path=/trunk/; revision=14770
2005-06-26 10:51:56 +00:00
Ulf Lamping
7589aef14f packet-ansi_683.c(93) : warning C4090: 'return' : different 'const' qualifiers
svn path=/trunk/; revision=14769
2005-06-26 10:48:27 +00:00
Ulf Lamping
c660aa2e54 packet-ansi_637.c(288) : warning C4090: 'return' : different 'const' qualifiers
svn path=/trunk/; revision=14768
2005-06-26 10:45:25 +00:00
Ulf Lamping
5b9f563924 packet-alcap.c(195) : warning C4090: 'return' : different 'const' qualifiers
svn path=/trunk/; revision=14767
2005-06-26 10:41:38 +00:00
Ronnie Sahlberg
f5009dc138 rename the protocol and the protocol fields to msmms from mms to make way for the telco mms protocol.
svn path=/trunk/; revision=14761
2005-06-26 08:12:54 +00:00
Ronnie Sahlberg
070e3b0970 update libethereal.def to the new name dissect_ber_CHOICE
svn path=/trunk/; revision=14760
2005-06-26 06:28:53 +00:00
Ronnie Sahlberg
e55dcb785e prettify ftam by puttign the command name in the info column
svn path=/trunk/; revision=14759
2005-06-26 06:16:03 +00:00
Ronnie Sahlberg
3a51ba4293 make dissect_ber_choice take a guint* that will return the
index of the branch taken or -1  to make prettifications easier to implement.


change the signature of dissect_ber_choice and rename it to dissect_ber_CHOICE to catch all
occurences of the use of this function

update asn2eth to use the new name/signature

update all occurences of this function to the new name and new signature.



svn path=/trunk/; revision=14758
2005-06-26 05:48:50 +00:00
Ronnie Sahlberg
2facda00c0 from Kashyap Desai
fix bug in conversation_delete_proto_data
second argument to g_slist_remove() is a pointer to the data, not a GSlist containing a list of such pointers.


svn path=/trunk/; revision=14755
2005-06-25 22:15:51 +00:00
Ronnie Sahlberg
06df848879 From Martin Mathieson
New protocol : MS MediaServer protocol


svn path=/trunk/; revision=14754
2005-06-25 21:57:11 +00:00
Guy Harris
4841b02c60 The data in an Identification packet is a message intended to be
human-readable; displayh it with "tvb_format_text()".

svn path=/trunk/; revision=14752
2005-06-25 19:14:17 +00:00
Anders Broman
44d693bf55 Major rewrite to update to version TS 129 002 V6.9.0
svn path=/trunk/; revision=14749
2005-06-25 17:26:30 +00:00
Ronnie Sahlberg
12c6e46a49 new FTAM dissector generated by asn2eth
svn path=/trunk/; revision=14747
2005-06-25 12:40:53 +00:00
Ronnie Sahlberg
d0a7c6efcd Make ACSE export the EXTERNAL construct since both CMIP and FTAM rely on it.
svn path=/trunk/; revision=14746
2005-06-25 07:46:14 +00:00
Ronnie Sahlberg
0392684ab7 for unknown GraphicString
just decode them as an octet string and say in the decode pane this is an unknown GraphicString



svn path=/trunk/; revision=14742
2005-06-24 10:03:20 +00:00
Ronnie Sahlberg
7f08072192 fix to ber NULL handling
1, make it actually show the hf field when present
2, make it handle imlicit_tag properly


svn path=/trunk/; revision=14741
2005-06-24 08:57:41 +00:00
Guy Harris
b67f63560e More constification.
svn path=/trunk/; revision=14740
2005-06-24 02:51:01 +00:00
Guy Harris
76258fe8e3 Constify a bunch of structure members and function arguments, to squelch
compiler warnings.

Clean up indentation.

svn path=/trunk/; revision=14739
2005-06-24 01:32:42 +00:00
Anders Broman
6f9d56ddd0 From Valery Vybornov:
add register_dissector to packet-tcap.c

svn path=/trunk/; revision=14738
2005-06-23 22:17:03 +00:00
Ronnie Sahlberg
a52b44c54b fix for CMIP, we should set the opcode_type BEFORE we try to use it in the subdissector,
it doesnt work really that well if we have opcode_type set to INVOKE   from the previous packet and this new packet contains a ReturnResult.



svn path=/trunk/; revision=14733
2005-06-23 09:28:46 +00:00
Gerald Combs
6a4b3f9cd7 Fix an off-by-four error, which should take care of bugs 254 - 258. Print
OID values as unsigned ints instead of signed.

svn path=/trunk/; revision=14729
2005-06-22 15:37:46 +00:00
Ronnie Sahlberg
299ce87bfe update to packet-ber
1, start making indefinite length  constructions actually work
2, when attempting to decode an unknown BER octet sequence, do not generate [malformed packet] just because the length does not make sense and might point outside the tvb.   it might just be that there are implicit tags in the asn1 specification and that it is just impossible to decode the octet stream without knowledge of the asn.



svn path=/trunk/; revision=14728
2005-06-22 09:58:46 +00:00
Ronnie Sahlberg
d9826d34d5 add a dummy variable frame_number to dissect_frame() and assign it the current frame number so that the crashdump that buildbot generates will show which frame the problem occured in.
this is useful since some bugs only trigger ia crash on some platforms.
example   bug 255 which does not trigger a crash on linux redhat 7.1



svn path=/trunk/; revision=14727
2005-06-22 08:41:58 +00:00
Ronnie Sahlberg
1a663222b4 only create the ACSE protocol tree if there is an ACSE layer present.
do not create it if it is just a SES_DATA_TRANSFER PDU


svn path=/trunk/; revision=14726
2005-06-22 08:28:59 +00:00
Ronnie Sahlberg
5a3ff9906c remove a dissector table we no longer use from ACSE
and also from the CMIP and FTAM dissectors that tried to register against it.

Implement a workaround for the asn2eth bug (can not yet handle tagged assignments) to the conformance file and remove the warnings from the template file.

Since workaround is implemented in conformance file, make make copy_files install the dissector instead of printing a warning message.



svn path=/trunk/; revision=14725
2005-06-22 08:15:18 +00:00
Luis Ontanon
acddab0452 add ipbcp attribute as in Q.1970
svn path=/trunk/; revision=14724
2005-06-22 02:18:10 +00:00
Gerald Combs
8de654b9ce Make a couple of variables dynamic, as suggested by Ulf.
svn path=/trunk/; revision=14723
2005-06-21 22:02:18 +00:00
Ronnie Sahlberg
6f189c8082 terminate some value_strings properly with a {0,NULL}
svn path=/trunk/; revision=14721
2005-06-21 10:19:10 +00:00
Ronnie Sahlberg
33f4400f83 the maximum ASN OID length is 256 bytes and there is a define to control this.
fix bug251 and all other occurances where an oid string  passed to packet_ber_object_identifier()
is not defined as foo[MAX_OID_STR_LEN]

svn path=/trunk/; revision=14720
2005-06-21 09:38:59 +00:00
Luis Ontanon
bfa7197fcf if exists call the h248term subdissector.
svn path=/trunk/; revision=14717
2005-06-20 21:21:06 +00:00
Anders Broman
46a2aec7bc From Mike Duigou:
In order to get filtering of conversations and
> endpoints to work from the endpoints and conversations windows I found 
> it necessary to add a new AT_URI address type and a SAT_JXTA. This also 
> necessitated a change to to_str.c to avoid a buffer overflow problem. 
> Please review these changes carefully.
> 
> Also includes some changes to the jxta dissector to fix filtering on
> generated fields and some changes to the types used for ints/unsigned ints.
> 
> Fixes a bug with the processing of messages containing namespaces.

svn path=/trunk/; revision=14716
2005-06-20 20:15:33 +00:00
Anders Broman
7ec6ab1d78 From Mike Duigou:
In order to get filtering of conversations and
> endpoints to work from the endpoints and conversations windows I found 
> it necessary to add a new AT_URI address type and a SAT_JXTA. This also 
> necessitated a change to to_str.c to avoid a buffer overflow problem. 
> Please review these changes carefully.
> 
> Also includes some changes to the jxta dissector to fix filtering on
> generated fields and some changes to the types used for ints/unsigned ints.
> 
> Fixes a bug with the processing of messages containing namespaces.

svn path=/trunk/; revision=14715
2005-06-20 20:14:57 +00:00
Gerald Combs
3a6c9ab134 Fix a format string vulneratility in the AFP dissector identified
by iDEFENSE.  Add constant format strings to proto_item_append_text()
in a bunch of other dissectors.  Copy a comment from proto.c to proto.h.

svn path=/trunk/; revision=14713
2005-06-20 19:35:58 +00:00
Ronnie Sahlberg
c3cfe365cf convert the function
dissect_spnego_mechTypes() tu use the packet-ber api instead of the 
asn1 api.


svn path=/trunk/; revision=14712
2005-06-20 10:22:03 +00:00
Ronnie Sahlberg
bdac892d95 convert one function to use packet-ber instead of asn1
svn path=/trunk/; revision=14711
2005-06-20 09:43:29 +00:00
Ronnie Sahlberg
5f334f7f7b move two functions that are not used by tcap and only used by gsm_a and gsm_ss into gsm_ss.
remove the include <epan.asn1.h> from tcap since it no longer needs that file.

only 9 dissectors remain that use asn1.h



svn path=/trunk/; revision=14710
2005-06-20 07:18:10 +00:00