vpax

Looks like this is because there's a capture that doesn't have an associated identifier due to the error, so this test https://github.com/zeek/zeek/blob/4751f6f31e9a88b81456a15006fb250bbd154c23/src/Stmt.cc#L1993 is de-ref'ing a nil pointer. A direct fix...

Any use of `{` _count_ `}` requires _count_ states (at least) to do the counting, so this isn't surprising. For these particular uses, certainly it'd be much better to have...

It's just needs a BiF - why not support both?

> It reads as if @vpax gets errors about unsupported LSAN when running `zeek` when building with address sanitizer and Spicy enabled, but then the errors go away when building...

@timwoj whoops ignore my earlier comment (now deleted), I hadn't correctly remembered the context. Here's the context. Event tracing works by stringing together a large chain of events that get...

Re the Alternative Fix, I just confirmed that indeed the events will be new ones not previously seen, so that would be a way to allow the draining to continue...

@Neverlord any update on this? It continues to bite us :-(

`&mergeable` was deprecated, and removed mid-2019.

I think in general it's using Broker, but @0xxon or @jsiwek can comment more definitively.

Main PR now available: https://github.com/zeek/zeek/pull/3716 That one and this one should be made at the same time.