From 3df42ca949a8cd98c956e2ff127ee2a5d8ee8fef Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Sun, 17 May 2020 19:47:45 +0200 Subject: 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 --- lib/dynbuf.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'lib/dynbuf.c') 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; } /* -- cgit v1.2.3