summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>2014-05-15 16:45:14 +0200
committerDanny Al-Gaaf <danny.al-gaaf@bisect.de>2014-05-15 17:06:48 +0200
commit1776f1fb548026d70f2354002cb5316844b69879 (patch)
tree18bcb40ad7bf84d5bb02d4236b0c743ac55f6dc7
parent2fb69e98130da6f74c12dd8c3b595c960a821995 (diff)
downloadceph-libs3-1776f1fb548026d70f2354002cb5316844b69879.tar.gz
s3.c: fix potential null pointer dereference
Fix for scan-build report: 387 (*gb)->prev->next = buf; 25 Access to field 'next' results in a dereference of a null pointer (loaded from field 'prev') Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
-rw-r--r--src/s3.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/s3.c b/src/s3.c
index 65acc52..e6b683b 100644
--- a/src/s3.c
+++ b/src/s3.c
@@ -376,7 +376,7 @@ static int growbuffer_append(growbuffer **gb, const char *data, int dataLen)
}
buf->size = 0;
buf->start = 0;
- if (*gb) {
+ if (*gb && (*gb)->prev) {
buf->prev = (*gb)->prev;
buf->next = *gb;
(*gb)->prev->next = buf;