[Bro-Dev] #808: topic/icmp6

Bro Tracker bro at tracker.bro-ids.org
Tue Apr 17 19:02:37 PDT 2012

#808: topic/icmp6
  Reporter:  jsiwek         |      Owner:
      Type:  Merge Request  |     Status:  new
  Priority:  Normal         |  Milestone:  Bro2.1
 Component:  Bro            |    Version:  git/master
Resolution:                 |   Keywords:

Comment (by robin):

 One question though: In {{{ICMP_Analyzer::RouterAdvert}}}:

 vl->append(new Val(htons(icmpp->icmp_lifetime), TYPE_COUNT));
 vl->append(new Val(reachable, TYPE_INTERVAL));
 vl->append(new Val(retrans, TYPE_INTERVAL));

 These don't look right.

 (1) Is hton really correct for the lifetime? Should the lifetime be of
 (2) TYPE_INTERVAL must be initialized with doubles, not with uint32; and
 it's needs seconds, not ms, as I believe the reachable/retrans fields
 record, don't they?

 (Merging it as it is, please commit fixes to fast path as appropriate and
 then close this ticket).

Ticket URL: <http://tracker.bro-ids.org/bro/ticket/808#comment:2>
Bro Tracker <http://tracker.bro-ids.org/bro>
Bro Issue Tracker

More information about the bro-dev mailing list