Hi,

I noticed a bug in the dissector, which seems to happen when then Exist_A5_bits field is set to 0, and results in the following fields (Exist_Multislot_capability/Exist_GPRS_multislot_class) being decoded incorrectly. Thus, get_ms_class_by_capability() is returning NULL instead of the GPRS_multislot_class, which is interestingly dissected correctly.

This is one of the messages where this happens: 4017dfb83a3f628a7045500898f28109100297e0080b2b

If I add it to the RLCMACTest it fails, so I propose it should be added ;)

My quite recent version of Wireshark (1.8.2) is decoding this message fine, see the attachment. I've gone through the recent changes on packet-gsm_rlcmac.c and packet-csn1.c in Wireshark, but couldn't spot the commit that fixed it so far.

Regards, Steve