From 645e1eb4155bccb4586e63febfbba7828d4a61d7 Mon Sep 17 00:00:00 2001 From: Gergely Nagy Date: Fri, 22 Jun 2012 15:11:42 +0200 Subject: Fix error handling in buffer.c MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * 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č Signed-off-by: Gergely Nagy --- lib/buffer.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib/buffer.h') 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) -- cgit