decode: big clean up (macros and functions)
decode.h seems to have some unused macros, functions that could replace macros, as well as functions that are named similarly than macros, but have slightly different behaviors.
I think it would be beneficial to go over the file and remove unused code and replace macros for functions when possible.
An interesting example is the PACKET_SET_ACTION macro - we have a PacketSetAction function which calls the macro, but takes into account packet tunneling - which doesn't seem to be accounted for in the corresponding macro.
Updated by Juliana Fajardini Reichow 8 months ago
I think that this https://github.com/OISF/suricata/pull/8030/commits/9e534e91175284e4fd70b5f85437b36798de754a#diff-48e701cfe9e2223cedaa68c2dad70955a5b41c803cc1a3feba33d75a18d35533 addresses most of the things this issue was supposed to cover.
Updated by Juliana Fajardini Reichow 6 months ago
- Target version changed from TBD to 8.0.0-beta1