diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2020-05-17 19:47:45 +0200 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2020-05-17 23:20:56 +0200 |
| commit | 3df42ca949a8cd98c956e2ff127ee2a5d8ee8fef (patch) | |
| tree | 3baafcaa6956296226e9a8c746c12155e4afa995 /lib/dynbuf.c | |
| parent | 2c598cc77822b27eef5ee6c2810ed6da49ff95bf (diff) | |
| download | curl-3df42ca949a8cd98c956e2ff127ee2a5d8ee8fef.tar.gz | |
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/dynbuf.c')
| -rw-r--r-- | lib/dynbuf.c | 4 |
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; } /* |