-
Notifications
You must be signed in to change notification settings - Fork 17
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
So it turns out that fuse_log() isn't marked with any of GCC's printf attributes. It's not so marked upstream, which is bad enough, but its replacment isn't marked in our tree either, so all the fuse_log()s aren't getting properly format-string checked. Fix that, fix a bunch of them, and fix up the logging output so it usually has timestamps in DTrace debug log format in it so testsuite output gets properly split up and so that you can correlate debug output from dtprobed with debug output from DTrace itself. Tie it into the daemon logging machinery (which sends error messages up the synchronization pipe after dtprobed has forked a daemonized child but before it has finished initializing and its parent has died off, for printing by the parent), so that fuse_log() can now be used everywhere without concern for whether this code is being run before dtprobed has daemonized or not. In the process, delete daemon_err(), a print-error-and-die function with no remaining uses. (Some format string errors remain after this commit, in code that is outright removed or rewritten in the following commits.) Signed-off-by: Nick Alcock <nick.alcock@oracle.com> Reviewed-by: Kris Van Hees <kris.van.hees@oracle.com>
- Loading branch information
1 parent
39e7324
commit ee75f90
Showing
4 changed files
with
84 additions
and
39 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters