SysinternalsEBPF icon indicating copy to clipboard operation
SysinternalsEBPF copied to clipboard

Unconditional printing to stderr

Open josalem opened this issue 4 years ago • 1 comments

https://github.com/Sysinternals/SysinternalsEBPF/blob/b9035cf7780c9c733f4be06ff29f7b4fc447b395/telemetryLoader.c#L1426

The library unconditionally prints to stderr, which can be problematic for consumption. If you have a TUI for instance, this can corrupt the TUI output if the user doesn't redirect stderr.

You can work around this by using dup2 to point stderr at /dev/null, but it would be nice if the library used some kind of logging library or put fprintf(stderr...) behind a config flag.

josalem avatar Oct 14 '21 19:10 josalem

Agreed - will put this on the list. Feel free to suggest a change though if you happen to be fixing your local copy. :)

kesheldr avatar Nov 03 '21 15:11 kesheldr