cisagov / icsnpp-enip

Zeek Ethernet/IP and CIP Parser - CISA ICSNPP
BSD 3-Clause "New" or "Revised" License
19 stars 10 forks source link

Extended Status Codes #9

Closed cjweelborg closed 1 year ago

cjweelborg commented 2 years ago

💡 Summary

What is the work, as a high-level summary? Addition of extended status codes that provide more detailed information on the base cip_status field.

Motivation and context

Why does this work belong in this project?

This would be useful because...

When searching for connection errors in CIP devices more detail from these extended status codes may provide extra useful information for the analyst when performing a hunt or analysis of an attack. This may help create the possibility of detecting specific types of attacks that have specific changes noted by the extended status.

Implementation notes

Please provide details for implementation, such as:

Addition of cip_extended_status or similar field in the parser. It would be helpful to include both the extended status code and extended status code value for easier parsing/searching.

Lines 1224 and 1342 contain the beginning of the wireshark dissector's extended status codes https://github.com/wireshark/wireshark/blob/master/epan/dissectors/packet-cip.c

Acceptance criteria

How do we know when this work is done?

Kleinspider commented 1 year ago

The parsing of extended status code has been added in the latest commit (0c610cdbfca6072127e5dd00d69084c98681ff43).

The following fields have been added to cip.log: