diff options
| author | Pascal Quantin <pascal.quantin@gmail.com> | 2014-12-13 18:15:52 +0100 |
|---|---|---|
| committer | Pascal Quantin <pascal.quantin@gmail.com> | 2014-12-13 17:17:00 +0000 |
| commit | 7592d39d46ee0f2442cc5263979a0f036f98468b (patch) | |
| tree | 23b53b3d734e95de6ba4f57bb95403eabf83545f /epan/dissectors/packet-gsm_sms.c | |
| parent | 21d6ed40adec5b8d5942f0dfc21a96b2fe110ac5 (diff) | |
GSM SMS: fix 'msg_class' may be used uninitialized in this function warning
Change-Id: Ie1c53929df2dfec1b4935ca1088f375c666d3b36
Reviewed-on: https://code.wireshark.org/review/5746
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-gsm_sms.c')
| -rw-r--r-- | epan/dissectors/packet-gsm_sms.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/epan/dissectors/packet-gsm_sms.c b/epan/dissectors/packet-gsm_sms.c index 9d159fafe9..8566c657b3 100644 --- a/epan/dissectors/packet-gsm_sms.c +++ b/epan/dissectors/packet-gsm_sms.c @@ -712,7 +712,7 @@ dis_field_dcs(tvbuff_t *tvb, proto_tree *tree, guint32 offset, guint8 oct, proto_tree_add_item(subtree, hf_gsm_sms_gsm_7_bit_default_alphabet, tvb, offset, 1, ENC_NA); *seven_bit = TRUE; - return 0xff; + return msg_class; } default_5_bits = FALSE; @@ -730,7 +730,7 @@ dis_field_dcs(tvbuff_t *tvb, proto_tree *tree, guint32 offset, guint8 oct, break; case 2: - return 0xff; + return msg_class; case 3: switch ((oct & 0x30) >> 4) @@ -783,7 +783,6 @@ dis_field_dcs(tvbuff_t *tvb, proto_tree *tree, guint32 offset, guint8 oct, proto_tree_add_item(subtree, hf_gsm_sms_dcs_indication_sense, tvb, offset, 1, ENC_NA); proto_tree_add_item(subtree, hf_gsm_sms_dcs_reserved04, tvb, offset, 1, ENC_NA); proto_tree_add_item(subtree, hf_gsm_sms_dcs_message_waiting, tvb, offset, 1, ENC_NA); - msg_class = 0xff; } else if (default_data) { |