aboutsummaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorDaniel Stenberg <daniel@haxx.se>2020-05-17 19:47:45 +0200
committerDaniel Stenberg <daniel@haxx.se>2020-05-17 23:20:56 +0200
commit3df42ca949a8cd98c956e2ff127ee2a5d8ee8fef (patch)
tree3baafcaa6956296226e9a8c746c12155e4afa995 /lib
parent2c598cc77822b27eef5ee6c2810ed6da49ff95bf (diff)
dynbuf: return NULL when there's no buffer length
... as returning a "" is not a good idea as the string is supposed to be allocated and returning a const string will cause issues. Reported-by: Brian Carpenter Follow-up to ed35d6590e72c Closes #5405
Diffstat (limited to 'lib')
-rw-r--r--lib/dynbuf.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/dynbuf.c b/lib/dynbuf.c
index 64004952f..dfc1d05c6 100644
--- a/lib/dynbuf.c
+++ b/lib/dynbuf.c
@@ -201,7 +201,7 @@ char *Curl_dyn_ptr(const struct dynbuf *s)
DEBUGASSERT(s);
DEBUGASSERT(s->init == DYNINIT);
DEBUGASSERT(!s->leng || s->bufr);
- return s->leng ? s->bufr : (char *)"";
+ return s->bufr;
}
/*
@@ -212,7 +212,7 @@ unsigned char *Curl_dyn_uptr(const struct dynbuf *s)
DEBUGASSERT(s);
DEBUGASSERT(s->init == DYNINIT);
DEBUGASSERT(!s->leng || s->bufr);
- return s->leng ? (unsigned char *)s->bufr : (unsigned char *)"";
+ return (unsigned char *)s->bufr;
}
/*