On Nov 26, 2012, at 1:11 PM, "Siwek, Jonathan Luke" <[email protected]> wrote:

> On Nov 20, 2012, at 10:21 AM, Seth Hall <[email protected]> wrote:
> 
>> I'm seeing Bro actually crash for the 
>> scripts.base.frameworks.metrics.cluster-intermediate-update test and I'm 
>> having a hard time figuring out why it's crashing.  Would someone mind 
>> taking a look at that soon?
> 
> See if the patch below makes sense.  Also not sure if that index_requests 
> table is missing some initialization somewhere.

Ah!  Thanks.  I bet you're right about this.  I'll test it tonight.

>    local ir = index_requests[uid, id, filter_name, index];
> 
> And as an optimization, maybe that "local ir" isn't needed since that index 
> was just assigned in the line above?


The ir is actually an optimization so that I don't have to keep indexing into 
that table over and over later in the event handler.  It's even possible that I 
only need to do that assignment when the first value comes in anyway.  I'll 
play around with that more tonight too.

Thanks for helping me out. :)

  .Seth

--
Seth Hall
International Computer Science Institute
(Bro) because everyone has a network
http://www.bro-ids.org/


_______________________________________________
bro-dev mailing list
[email protected]
http://mailman.icsi.berkeley.edu/mailman/listinfo/bro-dev

Reply via email to