Guy Harris wrote:
Pilz Rene wrote:
This patch fix the following parts:
- partly wrong cause values in the gmm (found by Miklos Szurdi)
- high/low nibble swap of the force to standby field (found by Miklos
Szurdi)
- dissect now all elements
Checked in.
Should the switch statement in "de_sm_pco()" instead first do
dissector_handle_t *handle;
...
handle = dissector_get_port_handle(gprs_sm_pco_subdissector_table,
prot);
if (handle != NULL) {
proto_tree_add_text(tree,tvb, curr_offset-3, 2, "Protocol:
(%u) %s", prot,
val_to_str(prot, ppp_vals, "Unknown"));
proto_tree_add_text(tree,tvb, curr_offset-1, 1, "Length:
0x%02x (%u)", e_len , e_len);
/*
* dissect the embedded message
*/
l3_tvb = tvb_new_subset(tvb, curr_offset, e_len, e_len);
call_dissector(handle, l3_tvb, &pco_pinfo, tree);
} else {
switch ( prot ) {
the case statements for values other than PPP protocol types
}
}
You'd include "packet-ppp.h" to get "ppp_vals[]" declared.
Hi
I will add the proposed change in my next patch.
Rene
_______________________________________________
Ethereal-dev mailing list
Ethereal-dev@xxxxxxxxxxxx
http://www.ethereal.com/mailman/listinfo/ethereal-dev
--
Dipl-Ing (FH) MSc. C.E René Pilz
ftw. Telekommunications Research Center Vienna http://www.ftw.at
Tech Gate Vienna, Donaucitystraße 1, A-1220 Wien
Mobile: +43 664 8269871 Office: +43 1 5052830-13 Fax: +43 1 5052830-99
