Hi!
While submitting the A-bis OML wireshark dissector I noticed one bug
in it that results in bogus message labeling.
The following patch should be applied to osmocom/wireshark.git tree.
P.S: I have submitted a patch to wireshark bugzilla that includes this
bugfix.
diff --git a/epan/dissectors/packet-gsm_abis_oml.c
b/epan/dissectors/packet-gsm_abis_oml.c
index 0066933..ee97fad 100644
--- a/epan/dissectors/packet-gsm_abis_oml.c
+++ b/epan/dissectors/packet-gsm_abis_oml.c
@@ -1184,6 +1184,7 @@ _match_oml_fom_msgtype(const guint32 val, const
value_string_ext *vs)
ret = match_strval_idx(val, _oml_fom_msgtype_vals, &idx);
if (!ret)
return NULL;
+ return &(_oml_fom_msgtype_vals[idx]);
}
return &(vs->_vs_p[idx]);
}
@@ -1198,6 +1199,7 @@ _match_oml_fom_attr(const guint32 val, const
value_string_ext *vs)
ret = match_strval_idx(val, _oml_fom_attr_vals, &idx);
if (!ret)
return NULL;
+ return &(_oml_fom_attr_vals[idx]);
}
return &(vs->_vs_p[idx]);
}