Commit Briefs

Omar Polo

convert remaining code to the imsg getters

Now gmid doesn't touch anymore the internals of the imsg structs.




Omar Polo

rename ibuf to imsgbuf in crypto

soon we'll be using a struct ibuf and it'll be confusing.



Omar Polo

don't let crypto_dispatch_server handle IMSG_CRYPTO_ECDSA_SIGN

in this codepath. otherwise we end up with a mismatch where we expect a request but were sent a response.


Omar Polo

pre-increment reqid

otherwise we send the request id N and expect to receive N+1



Omar Polo

fmt


Omar Polo

drop engine support


Omar Polo

remove the useless logging methods

it makes more clear where the magic is. adapted from the smtpd' ca.c diff.


Omar Polo

macos' clang is retarded

thinks rsa and ecdsa may be used un-initialized... if we enter the branch with fatalx(). sigh


Omar Polo

drop debug log


Omar Polo

simplify check

brought to my attention by gcc who isn't smart enough to figure out that `ret' is always set.


Omar Polo

avoid arithmetic on void pointers (GNU extension)

not really sold on this one, I don't see what other interpretation could be given, but it's not standard so...