[Bro-Dev] #823: Remaining input framework todos

Bro Tracker bro at tracker.bro-ids.org
Wed May 30 19:28:37 PDT 2012


#823: Remaining input framework todos
---------------------+------------------------
 Reporter:  robin    |      Owner:  bernhard
     Type:  Problem  |     Status:  new
 Priority:  Normal   |  Milestone:  Bro2.1
Component:  Bro      |    Version:  git/master
 Keywords:           |
---------------------+------------------------
 The input framework is merged into git master. A few little things remain
 to do:

 - core.check-unused-event-handlers fails with this
    {{{
    == Diff ===============================
   --- /tmp/test-diff.16838..stderr.baseline.tmp 2012-05-31
 02:05:06.318172610 +0000
   +++ /tmp/test-diff.16838..stderr.tmp  2012-05-31 02:05:06.328172991
 +0000
   @@ -1 +1,2 @@
   +warning in <params>, line 1: event handler never invoked:
 Input::update_finished
   warning in <params>, line 1: event handler never invoked:
 this_is_never_used
   =======================================
    }}}

   I.e., the event will never be raised. I didn't see this before the merge
 into master, not sure why it now has started appearing?

 - Track down memory leaks.

 - Document ReaderMode in src/input/ReaderBackend.h:

 - Please address this one

    {{{
    ReaderFrontend.cc:

     // FIXME: cleanup of disabled inputreaders is missing. we need this,
 because
     // stuff can e.g. fail in init and might never be removed afterwards.
    }}}

 - Extend doc/input.rst with more examples.

-- 
Ticket URL: <http://tracker.bro-ids.org/bro/ticket/823>
Bro Tracker <http://tracker.bro-ids.org/bro>
Bro Issue Tracker



More information about the bro-dev mailing list