> On Sep 21, 2018, at 11:24 AM, Jon Siwek <jsi...@corelight.com> wrote:
> 
> On Thu, Sep 20, 2018 at 2:14 PM Michael Dopheide <dophe...@es.net> wrote:
>> 
>> bro.bif:3630:47: error: cannot convert ‘stat’ to ‘__dev_t {aka long unsigned 
>> int}’ in initialization
> 
> Thanks, it's fixed by a simple patch [1] now in master.
> 
> - Jon
> 
> [1] https://github.com/bro/bro/commit/d7097635f4b1ec9adca548b81ee42968629987a5

That fixed the build, but I think I found a new issue.

If bro is built with maxminddb support but the db files are not present yet, 
bro logs this twice for each connection:

1537548892.975875       Reporter::ERROR Failed to open GeoIP location database 
(lookup_location(Conn::c$id$resp_h))     
/srv/bro/spool/installed-scripts-do-not-touch/site/ncsa/./bro-scripts/./conn-add-geodata.bro,
 line 23
1537548892.978846       Reporter::ERROR Failed to open GeoIP location database 
(lookup_location(Conn::c$id$orig_h))     
/srv/bro/spool/installed-scripts-do-not-touch/site/ncsa/./bro-scripts/./conn-add-geodata.bro,
 line 19

I think I remember the older geoip bifs logging this as well, but only once at 
startup, not once per call to lookup_location.

— 
Justin Azoff


_______________________________________________
bro-dev mailing list
bro-dev@bro.org
http://mailman.icsi.berkeley.edu/mailman/listinfo/bro-dev

Reply via email to