[Bro-Dev] #538: lookup_location() should handle missing database gracefully.

Bro Tracker bro at tracker.bro-ids.org
Tue Aug 30 15:18:43 PDT 2011


#538: lookup_location() should handle missing database gracefully.
----------------------+------------------------
  Reporter:  gregor   |      Owner:  jsiwek
      Type:  Problem  |     Status:  assigned
  Priority:  High     |  Milestone:  Bro1.6
 Component:  Bro      |    Version:  git/master
Resolution:           |   Keywords:
----------------------+------------------------

Comment (by jsiwek):

 In [13a09aa488e21ec581047249460425d338afe3c4/bro]:
 {{{
 #!CommitTicketReference repository="bro"
 revision="13a09aa488e21ec581047249460425d338afe3c4"
 Fixes for GeoIP support (addresses #538).

 - Missing GeoIP databases now generate warnings/errors that go through
   the reporter framework instead of hitting GeoIP's internal use of
   stderr

 - lookup_location now just queries for country code if the city database
   was not loaded, which gets rid of invalid database type errors.

 - lookup_location now leaves missing fields uninitialized in the
   returned geo_location record value.  Updated existing scripts to
   check for initialized fields in geo_location records before use.

 - Fixed support for GeoIP's IPv6 API and databases
 }}}

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



More information about the bro-dev mailing list