diff options
| author |   Andreas Schultz <andreas.schultz@travelping.com> | 2020-05-15 11:50:07 +0200 |
|---|---|---|
| committer |   Ivan Shvedunov <ivan4th@gmail.com> | 2022-07-06 03:32:18 +0400 |
| commit | 972dc17634e430cb93b97c67b50096acc3164231 (patch) | |
| tree | 2a091bf4b3a16b7c750dda72f268052141ff6dbe /src/plugins | |
| parent | f8631ce7e8886136b4543a7926ffdf1bc760fb11 (diff) | |
misc: pass NULL instead of 0 for pointer in variadic functions
0 is not NULL (at least not in all cases), passing 0 into a variadic
function in a place where the consumer reads it as pointer might
leave parts of the pointer uninitilized and hence filled with random
data.
It seems that this used to work with gcc, but clang seems to treat the
0 in those places as a 32bit integer.
Type: fix
Signed-off-by: Ivan Shvedunov <ivan4th@gmail.com>
Signed-off-by: Andreas Schultz <andreas.schultz@travelping.com>
Change-Id: I37d975eef5a1ad98fbfb65ebe47d73458aafea00
Diffstat (limited to 'src/plugins')
| -rw-r--r-- | src/plugins/mactime/mactime_top.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/mactime/mactime_top.c b/src/plugins/mactime/mactime_top.c index abcb530608c..d7223f45e56 100644 --- a/src/plugins/mactime/mactime_top.c +++ b/src/plugins/mactime/mactime_top.c @@ -441,7 +441,7 @@ print_device_table (mt_main_t * mm) { mactime_device_t *dev; - fformat (stdout, "%U", format_device, 0 /* header */ , 0 /* verbose */ ); + fformat (stdout, "%U", format_device, NULL /* header */, 0 /* verbose */); /* *INDENT-OFF* */ pool_foreach (dev, mm->devices) { |