diff options
author | Gergely Nagy <algernon@balabit.hu> | 2012-06-22 15:11:42 +0200 |
---|---|---|
committer | Gergely Nagy <algernon@balabit.hu> | 2012-06-22 15:11:42 +0200 |
commit | 645e1eb4155bccb4586e63febfbba7828d4a61d7 (patch) | |
tree | ed8d47a00533e1806579353ac028eb1025c885c2 /lib/buffer.h | |
parent | 88c48725d12663dd807e28d65871f8db4ac7273e (diff) | |
download | libumberlog-645e1eb4155bccb4586e63febfbba7828d4a61d7.tar.gz libumberlog-645e1eb4155bccb4586e63febfbba7828d4a61d7.tar.xz libumberlog-645e1eb4155bccb4586e63febfbba7828d4a61d7.zip |
Fix error handling in buffer.c
* Don't lose old msg on realloc() failure
* Don't dereference NULL on failure when
buffer = (...)
if (!buffer) { buffer->len = orig_len; ...}
* To avoid the above pattern, use integer return values:
"buffer" itself will not change in any case.
* Add missing error handling
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Gergely Nagy <algernon@balabit.hu>
Diffstat (limited to 'lib/buffer.h')
-rw-r--r-- | lib/buffer.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/buffer.h b/lib/buffer.h index 0d412e8..359c9c6 100644 --- a/lib/buffer.h +++ b/lib/buffer.h @@ -37,7 +37,7 @@ typedef struct char *msg; } ul_buffer_t; -ul_buffer_t *ul_buffer_reset (ul_buffer_t *buffer) +int ul_buffer_reset (ul_buffer_t *buffer) __attribute__((visibility("hidden"))); ul_buffer_t *ul_buffer_append (ul_buffer_t *buffer, const char *key, const char *value) |