diff options
author | Dr. Lars Völker <lars.voelker@technica-engineering.de> | 2021-04-23 13:37:37 +0200 |
---|---|---|
committer | Wireshark GitLab Utility <gerald+gitlab-utility@wireshark.org> | 2021-04-24 11:40:57 +0000 |
commit | 8a96d2a90395421261e1ed5f2fb7a9d4e7f25a0f (patch) | |
tree | d2f50bf58e8d2dd4964513840a16a92c999ad3bf /epan/dissectors/packet-autosar-nm.c | |
parent | e35efdee8d0938585098f36045c414e8c63b348b (diff) |
CAN, AUTOSAR NM, Signal PDU: Fix the interpretation of CAN ID
This patches makes sure that the registered IDs are not influenced
by the three flags mapped into the same uint32.
This was a oversight in the AUTOSAR NM and Signal PDU dissectors.
Diffstat (limited to 'epan/dissectors/packet-autosar-nm.c')
-rw-r--r-- | epan/dissectors/packet-autosar-nm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-autosar-nm.c b/epan/dissectors/packet-autosar-nm.c index 43def25017..46051a99de 100644 --- a/epan/dissectors/packet-autosar-nm.c +++ b/epan/dissectors/packet-autosar-nm.c @@ -400,7 +400,7 @@ is_relevant_can_message(void *data) return FALSE; } - if ((can_info->id & g_autosar_nm_can_id_mask) != (g_autosar_nm_can_id & g_autosar_nm_can_id_mask)) { + if ((can_info->id & CAN_EFF_MASK & g_autosar_nm_can_id_mask) != (g_autosar_nm_can_id & CAN_EFF_MASK & g_autosar_nm_can_id_mask)) { /* Id doesn't match. The frame is not for us. */ return FALSE; } |