diff options
| author | Peter Jones <pjones@redhat.com> | 2021-12-10 16:18:41 -0500 |
|---|---|---|
| committer | Peter Jones <pjones@redhat.com> | 2021-12-10 17:08:21 -0500 |
| commit | 2e78cd93390f83648a20b5b3bb934f846537e54c (patch) | |
| tree | 48c512859a3d9486114530095471803d90339a85 | |
| parent | 0dd4c78fc0846b36a334068ed5e5495e1d3d5f9f (diff) | |
| download | efi-boot-shim-2e78cd93390f83648a20b5b3bb934f846537e54c.tar.gz efi-boot-shim-2e78cd93390f83648a20b5b3bb934f846537e54c.zip | |
stdarg: use sysv varargs when we build with coverity
cov-analysis-linux64-2020.09 is a lot more successful than the older
versions at building, but it still has some... issues. Among them, it
is of the belief that this:
void
foo(char *fmt, ...)
{
__builtin_va_list ap;
__builtin_ms_va_start(ap, fmt); /* <- here */
...
}
is an uninitialized use of "ap".
This patch adds defined(__COVERITY__) to the list of criteria for using
sysv va lists, which it has no such confusion about.
Signed-off-by: Peter Jones <pjones@redhat.com>
| -rw-r--r-- | include/system/stdarg.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/system/stdarg.h b/include/system/stdarg.h index 4c956f70..68c171b8 100644 --- a/include/system/stdarg.h +++ b/include/system/stdarg.h @@ -24,7 +24,7 @@ typedef __builtin_va_list __builtin_sysv_va_list; #endif #if defined(__aarch64__) || defined(__arm__) || defined(__i386__) || \ - defined(__i486__) || defined(__i686__) + defined(__i486__) || defined(__i686__) || defined(__COVERITY__) typedef __builtin_va_list ms_va_list; typedef __builtin_va_list __builtin_ms_va_list; |