Kees Cook kees@outflux.net writes:
FIXME is sane for the "announce_bad_opaque_data", but I'd still like to use something that doesn't prefix the hexdumps with the function name for easier readability in the crypt_report_func_input, since there is already a TRACE/FIXME call being made prior to it's call. Is this okay?
No, you really want the prefix on all lines, otherwise it makes a big mess when multiple threads are printing things at the same time.