Commit Briefs

Omar Polo

add compat/*.o

got doesn't seem to match (say) compat/freezero.o with *.o


Omar Polo

move vhost_should_log call to server.c

log.o is linked to some regress/ stuff. Calling from there a vhost_* function means that we should link the regress/stuff to server.o too (and that would pull in other stuff...). Moving the call is easier, and also probably better.




Omar Polo

tests and compat for imsg


Omar Polo

moving logging to its own process



Omar Polo

other s/fnmatch/matches


Omar Polo

typo


Omar Polo

kill debug printf


Omar Polo

include config.h first


Omar Polo

fix various compilation errors

Include gmid.h as first header in every file, as it then includes config.h (that defines _GNU_SOURCE for instance). Fix also a warning about unsigned vs signed const char pointers in openssl.


Omar Polo

accept4 -> accept

accept4(2) isn't part of any standard (even though it'll be part in the future) and raises warnings on some linux distro. Moreover, we don't have thread that may fork at any time, so doing a mark_nonblock after isn't a big deal.


Omar Polo

BUFSIZ is variable, we need *exactly* 1024

Using BUFSIZ in sbuf is not OK. It's variable, and in various places we assume that sbuf is 1024 (like handle_cgi_reply). We could patch those, but we aren't sure BUFSIZ is >= 1024! Let's keep the hardcoded number. (found by debugging on arch on amd64, where BUFSIZ is bigger)


Omar Polo

typo