diff options
| author | Martin Kaiser <wireshark@kaiser.cx> | 2016-05-17 22:52:42 +0200 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2016-05-17 23:33:07 +0000 |
| commit | 21f798e50f6f641f62b9f56a14abb58da4f5dc4d (patch) | |
| tree | cc5c9e29e4d40d1103c8b4a5925cf2b810d2f73e /epan | |
| parent | 260ef14e095f5120dabd93a2118d6b125ef10f05 (diff) | |
| download | wireshark-21f798e50f6f641f62b9f56a14abb58da4f5dc4d.tar.gz | |
lbmpdm (CID 1201451): result of shift operation is always 0
apply a mask to get the lower 32 bit of a guint64
(there's no need for a right shift)
Change-Id: Ifa8281a64bb327bd9b8591cc2e54ee35ab15ae11
Reviewed-on: https://code.wireshark.org/review/15491
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan')
| -rw-r--r-- | epan/dissectors/packet-lbmpdm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-lbmpdm.c b/epan/dissectors/packet-lbmpdm.c index 40367caee4..c23878a657 100644 --- a/epan/dissectors/packet-lbmpdm.c +++ b/epan/dissectors/packet-lbmpdm.c @@ -525,7 +525,7 @@ static int lbmpdm_get_segment_length(tvbuff_t * tvb, int offset, int encoding, i static void lbmpdm_definition_build_key(guint32 * key_value, wmem_tree_key_t * key, guint64 channel, guint32 id, guint8 version_major, guint8 version_minor) { key_value[LBMPDM_DEFINITION_KEY_ELEMENT_CHANNEL_HIGH] = (guint32) ((channel >> 32) & 0xffffffff); - key_value[LBMPDM_DEFINITION_KEY_ELEMENT_CHANNEL_LOW] = (guint32) ((channel & 0xffffffff) >> 32); + key_value[LBMPDM_DEFINITION_KEY_ELEMENT_CHANNEL_LOW] = (guint32) (channel & 0xffffffff); key_value[LBMPDM_DEFINITION_KEY_ELEMENT_ID] = id; key_value[LBMPDM_DEFINITION_KEY_ELEMENT_VERS_MAJOR] = version_major; key_value[LBMPDM_DEFINITION_KEY_ELEMENT_VERS_MINOR] = version_minor; |