Tim Wojtulewicz

The broker error was fixed in https://github.com/zeek/zeek/pull/3539

It's definitely possible there's some bug in c-ares on Windows. I'll try to get a backtrace out of a Windows build tomorrow for this.

Got a more useful backtrace finally (after I actually read your repro steps above 🤦🏼‍♂️ ): ``` Exception thrown: read access violation. filt was 0xFFFFFFFFFFFFFFD7. > zeek.exe!windows_kevent_copyout(kqueue * kq, int...

It's odd because DNS_Mgr and c-ares aren't doing anything untoward that I can tell. It drops the nameserver connection a couple of times but re-establishes at the same time both...

Opened a ticket with Cirrus about this one: https://github.com/cirruslabs/cirrus-ci-docs/issues/1241

I just checked this again and it's still failing on Cirrus. I did run a tsan build locally though and there aren't any other findings that we haven't already either...

> We actually have the LSAN macro games also in Zeek, but we do enable LSAN in the end from what it seems. Correct, the block in `zeek-config.h.in` only disables...

While it shouldn't crash, it's definitely incorrect usage of the ternary operator. That should fail parsing earlier than it is.

One real question here is what we should allow for the right side of a ternary operator. Technically the `Eval()` method of any expression should return a ValPtr and be...

At the very least there's a parsing error finding the various expressions in the line of script. I set a breakpoint in the `CondExpr` constructor and printed out the three...