llh created BIT-1573:
------------------------

             Summary: 3 useless EventHandlerPtr in the ARP Analyzer
                 Key: BIT-1573
                 URL: https://bro-tracker.atlassian.net/browse/BIT-1573
             Project: Bro Issue Tracker
          Issue Type: Improvement
          Components: Bro
            Reporter: llh
            Priority: Trivial


The class analyzer::arp::ARP_Analyzer declared in the file 
src/analyzer/protocol/arp/ARP.h declares 3 protected EventHandlerPtr that are 
never initialized and never used.
What the corresponding source file refer to are the following global variables :
* bad_arp
* arp_request
* arp_reply
which are declared as "extern" in the file 
build/src/analyzer/protocol/arp/events.bif.h which is generated by bifcl from 
src/analyzer/protocol/arp/events.bif.

Fixing this issue is trivial : deleting the 3 lines declaring the unused 
EventHandlerPtr.
The expected improvement is saving 3 bytes of memory and mostly not messing 
with those who will try to understand the code of this analyzer in the future. 



--
This message was sent by Atlassian JIRA
(v1000.5.0#72002)
_______________________________________________
bro-dev mailing list
bro-dev@bro.org
http://mailman.icsi.berkeley.edu/mailman/listinfo/bro-dev

Reply via email to