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 /include/internal | |
| 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 'include/internal')
| -rw-r--r-- | include/internal/internal.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/include/internal/internal.h b/include/internal/internal.h index bb44e12..b1fc670 100644 --- a/include/internal/internal.h +++ b/include/internal/internal.h @@ -41,6 +41,8 @@ #endif #define BUFFER_SIZE(ret, size, len, offset) \ + if (ret < 0) \ + return -1; \ size += ret; \ if (ret > len) \ ret = len; \ |