diff options
author | morriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7> | 2009-09-16 13:39:37 +0000 |
---|---|---|
committer | morriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7> | 2009-09-16 13:39:37 +0000 |
commit | 66be9c2f5a759aeed3c02b4308237298fd5f9293 (patch) | |
tree | bf40546f222d489a96fd502ff93f69ffab25652a /epan/dissectors/packet-rsl.c | |
parent | 48730e146e2fef8396dc165eaf1dbbe5ab9bc67c (diff) |
Add a couple of new hf_ entries to deal with the different bit positions
for the MS Power Level and FPC in the L1 Information and MS Power IEs.
This should fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4017
(though I don't have a sample capture to verify the fix and that I didn't
break anything.)
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@29944 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-rsl.c')
-rw-r--r-- | epan/dissectors/packet-rsl.c | 27 |
1 files changed, 16 insertions, 11 deletions
diff --git a/epan/dissectors/packet-rsl.c b/epan/dissectors/packet-rsl.c index 7cfea64c78..d915353531 100644 --- a/epan/dissectors/packet-rsl.c +++ b/epan/dissectors/packet-rsl.c @@ -65,10 +65,11 @@ static int hf_rsl_req_ref_T3 = -1; static int hf_rsl_req_ref_T2 = -1; static int hf_rsl_timing_adv = -1; static int hf_rsl_ho_ref = -1; +static int hf_rsl_l1inf_power_lev = -1; +static int hf_rsl_l1inf_fpc = -1; static int hf_rsl_ms_power_lev = -1; static int hf_rsl_ms_fpc = -1; static int hf_rsl_act_timing_adv = -1; -static int hf_rsl_power_lev = -1; static int hf_rsl_phy_ctx = -1; static int hf_rsl_na = -1; static int hf_rsl_ch_type = -1; @@ -1087,9 +1088,9 @@ dissect_rsl_ie_l1_inf(tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *tree, i * The information fields and codings are as defined in 3GPP TS 44.004. */ /* Power level */ - proto_tree_add_item(ie_tree, hf_rsl_ms_power_lev, tvb, offset, 1, FALSE); + proto_tree_add_item(ie_tree, hf_rsl_l1inf_power_lev, tvb, offset, 1, FALSE); /* FPC */ - proto_tree_add_item(ie_tree, hf_rsl_ms_fpc, tvb, offset, 1, FALSE); + proto_tree_add_item(ie_tree, hf_rsl_l1inf_fpc, tvb, offset, 1, FALSE); offset++; /* Actual Timing Advance */ proto_tree_add_item(ie_tree, hf_rsl_act_timing_adv, tvb, offset, 1, FALSE); @@ -3621,27 +3622,31 @@ void proto_register_rsl(void) FT_UINT8, BASE_DEC, NULL, 0x0, NULL, HFILL } }, - { &hf_rsl_ms_power_lev, + { &hf_rsl_l1inf_power_lev, { "MS power level", "rsl.ms_power_lev", FT_UINT8, BASE_DEC, NULL, 0xf8, NULL, HFILL } }, - { &hf_rsl_ms_fpc, + { &hf_rsl_l1inf_fpc, { "FPC/EPC", "rsl.ms_fpc", FT_BOOLEAN, 8, TFS(&rsl_ms_fpc_epc_mode_vals), 0x04, NULL, HFILL } }, + { &hf_rsl_ms_power_lev, + { "MS power level", "rsl.ms_power_lev", + FT_UINT8, BASE_DEC, NULL, 0x1f, + NULL, HFILL } + }, + { &hf_rsl_ms_fpc, + { "FPC/EPC", "rsl.ms_fpc", + FT_BOOLEAN, 8, TFS(&rsl_ms_fpc_epc_mode_vals), 0x20, + NULL, HFILL } + }, { &hf_rsl_act_timing_adv, { "Actual Timing Advance", "rsl.act_timing_adv", FT_UINT8, BASE_DEC, NULL, 0x0, NULL, HFILL } }, - { &hf_rsl_power_lev, - { "Power level", "rsl.power_lev", - FT_UINT8, BASE_DEC, NULL, 0x1f, - NULL, HFILL } - }, - { &hf_rsl_dtxd, { "DTXd", "rsl.dtxd", FT_BOOLEAN, 8, TFS(&rsl_dtxd_vals), 0x40, |