diff options
author | Kelvin M. Klann <kmk3.code@protonmail.com> | 2023-06-16 15:42:20 -0300 |
---|---|---|
committer | Kelvin M. Klann <kmk3.code@protonmail.com> | 2023-06-28 04:00:13 -0300 |
commit | b963fe41ae2cd669e5819aded531375ddaebc8b2 (patch) | |
tree | a82794efe2b8b925d24a3fd413663cef1e987926 /config.mk.in | |
parent | common.h: use __func__ instead of __FUNCTION__ (diff) | |
download | firejail-b963fe41ae2cd669e5819aded531375ddaebc8b2.tar.gz firejail-b963fe41ae2cd669e5819aded531375ddaebc8b2.tar.zst firejail-b963fe41ae2cd669e5819aded531375ddaebc8b2.zip |
Improve errExit error messages
Changes:
* Move msg to the end of errExit (right before perror(3p))
* Include the full file path (within the repository)
* Add "()" to function name for clarity
Before:
Error malloc: main.c:123 main: Cannot allocate memory
After:
Error src/firejail/main.c:123 main(): malloc: Cannot allocate memory
Note: This clarifies which is the exact file that the error message
comes from, as there are many source files with the same name. For
example:
$ git ls-files 'src/*/main.c' | wc -l
20
Diffstat (limited to 'config.mk.in')
-rw-r--r-- | config.mk.in | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/config.mk.in b/config.mk.in index f3c1f658c..c76ca1a98 100644 --- a/config.mk.in +++ b/config.mk.in | |||
@@ -83,7 +83,7 @@ LDFLAGS=@LDFLAGS@ | |||
83 | # Project variables | 83 | # Project variables |
84 | EXTRA_CFLAGS =@EXTRA_CFLAGS@ | 84 | EXTRA_CFLAGS =@EXTRA_CFLAGS@ |
85 | COMMON_CFLAGS = \ | 85 | COMMON_CFLAGS = \ |
86 | -ggdb -O2 -DVERSION='"$(VERSION)"' \ | 86 | -ggdb -O2 -DVERSION='"$(VERSION)"' -DMOD_DIR='"$(MOD_DIR)"' \ |
87 | -Wall -Wextra $(HAVE_FATAL_WARNINGS) \ | 87 | -Wall -Wextra $(HAVE_FATAL_WARNINGS) \ |
88 | -Wformat -Wformat-security \ | 88 | -Wformat -Wformat-security \ |
89 | -fstack-protector-all -D_FORTIFY_SOURCE=2 \ | 89 | -fstack-protector-all -D_FORTIFY_SOURCE=2 \ |