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/conntrack/api.c | |
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/conntrack/api.c')
-rw-r--r-- | src/conntrack/api.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/conntrack/api.c b/src/conntrack/api.c index ffa5216..78d7d61 100644 --- a/src/conntrack/api.c +++ b/src/conntrack/api.c @@ -1099,9 +1099,9 @@ int nfct_catch(struct nfct_handle *h) * print the message just after you receive the destroy event. If you want * more accurate timestamping, use NFCT_OF_TIMESTAMP. * - * This function returns the size of the information that _would_ have been - * written to the buffer, even if there was no room for it. Thus, the - * behaviour is similar to snprintf. + * On error, -1 is returned and errno is set appropiately. Otherwise the + * size of what _would_ be written is returned, even if the size of the + * buffer is insufficient. This behaviour is similar to snprintf. */ int nfct_snprintf(char *buf, unsigned int size, |