Hello,
The Q931 dissector applies a wrong mask when testing the odd/even indicator
bit. Here is a patch that fixes the issue.
2006-11-27 Richard Braun <rbraun@xxxxxxxxxxxxxxxxx>
* epan/dissectors/packet-q931.c (q931_odd_even_indicator_vals):
Fixed odd/even indicator mask.
(dissect_q931_party_subaddr_ie): Likewise.
Index: epan/dissectors/packet-q931.c
===================================================================
--- epan/dissectors/packet-q931.c (revision 20002)
+++ epan/dissectors/packet-q931.c (working copy)
@@ -2157,7 +2157,7 @@
static const value_string q931_odd_even_indicator_vals[] = {
{ 0x00, "Even number of address signals" },
- { 0x10, "Odd number of address signals" },
+ { 0x08, "Odd number of address signals" },
{ 0, NULL }
};
@@ -2176,7 +2176,7 @@
"Unknown (0x%02X)"));
proto_tree_add_text(tree, tvb, offset, 1,
"Odd/even indicator: %s",
- val_to_str(octet & 0x10, q931_odd_even_indicator_vals,
+ val_to_str(octet & 0x08, q931_odd_even_indicator_vals,
"Unknown (0x%02X)"));
offset += 1;
len -= 1;
--
Richard Braun
Proformatique - 67 rue Voltaire - 92800 Puteaux
Tel. : 01 41 38 99 65 - Fax. : 01 41 38 99 70
Email : rbraun@xxxxxxxxxxxxxxxxx - http://proformatique.com/
PS: I haven't subscribed to the mailing-list, please send any reply using
my address in Cc.
Attachment:
signature.asc
Description: Digital signature
