Bill Fenner

Results 57 comments of Bill Fenner

> Do you think a separate thread around `print_packets_captured()` would be easier to implement? Would it still be portable enough? I've always tried to avoid threads in portable code, but...

> So does that test program catch SIGINT? Not explicitly, but strace shows that someone is (maybe python's KeyboardInterrupt handler). I'm ok with punting on that for now and focusing...

> > The next thing it does is register fd in our event manager, and it's plausible that this (perhaps incorrectly) causes a call to pcap_next() - but given that...

> > > > The next thing it does is register fd in our event manager, and it's plausible that this (perhaps incorrectly) causes a call to pcap_next() - but...

This looks correct to me: multiple child processes exiting can result in only a single SIGCHLD, so looping with waitpid() to reap them all is correct.

> > Thank you for preparing these changes. Do you still have the UBSan messages? Does `make check` pass in your working copy? I will recreate the UBSan messages, by...

P.S. I will also squash the "Address integer overflows" commit into the "ISO:" commit once my discussion with fxlb is complete.

The integer overflow commit is still separate, because I want to validate it against a separate implementation of this checksum implementation that I have access to. I updated the .out...

I didn't think it was worth pushing my fix for the illumos build, but I have done so now to avoid getting confused by ``` Your branch and 'origin/fenner-ubsan-fixes' have...