diff options
author | Daniel Stenberg <daniel@haxx.se> | 2022-09-05 23:21:15 +0200 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2022-09-06 08:34:30 +0200 |
commit | 6f9fb7ec2d7cb389a0da5a1d0617ce592115a6a8 (patch) | |
tree | 44b640deb9b58ca0f50a60395ee7259c19344506 /lib/openldap.c | |
parent | 8dd95da35be27c28da9e0d082f3ce02d2200b82a (diff) | |
download | curl-6f9fb7ec2d7cb389a0da5a1d0617ce592115a6a8.tar.gz |
misc: ISSPACE() => ISBLANK()
Instances of ISSPACE() use that should rather use ISBLANK(). I think
somewhat carelessly used because it sounds as if it checks for space or
whitespace, but also includes %0a to %0d.
For parsing purposes, we should only accept what we must and not be
overly liberal. It leads to surprises and surprises lead to bad things.
Closes #9432
Diffstat (limited to 'lib/openldap.c')
-rw-r--r-- | lib/openldap.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/openldap.c b/lib/openldap.c index 19f2ad9f0..3a93b6728 100644 --- a/lib/openldap.c +++ b/lib/openldap.c @@ -1068,8 +1068,8 @@ static ssize_t oldap_recv(struct Curl_easy *data, int sockindex, char *buf, if(!binary) { /* check for leading or trailing whitespace */ - if(ISSPACE(bvals[i].bv_val[0]) || - ISSPACE(bvals[i].bv_val[bvals[i].bv_len - 1])) + if(ISBLANK(bvals[i].bv_val[0]) || + ISBLANK(bvals[i].bv_val[bvals[i].bv_len - 1])) binval = 1; else { /* check for unprintable characters */ |