diff options
| author | Daniel Gröber <dxld@darkboxed.org> | 2020-06-24 15:29:57 +0200 |
|---|---|---|
| committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2020-07-01 12:40:27 +0200 |
| commit | 37e19771bbe9ff5e0c5a0c2fedfa356dcbc40627 (patch) | |
| tree | f16c849f261f3d40961ac126df6f6365b429a512 /src/expect | |
| parent | f5ce6d7792e0b09cf67e100b0d03226b688e6ea0 (diff) | |
src: Handle negative snprintf return values properly
Currently the BUFFER_SIZE macro doesn't take negative 'ret' values into
account. A negative return should just be passed through to the caller,
snprintf will already have set 'errno' properly.
Signed-off-by: Daniel Gröber <dxld@darkboxed.org>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src/expect')
| -rw-r--r-- | src/expect/snprintf.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/expect/snprintf.c b/src/expect/snprintf.c index 3a104b5..8c2f3e4 100644 --- a/src/expect/snprintf.c +++ b/src/expect/snprintf.c @@ -30,6 +30,9 @@ int __snprintf_expect(char *buf, return -1; } + if (size < 0) + return size; + /* NULL terminated string */ buf[size+1 > len ? len-1 : size] = '\0'; |