diff options
| author | tedu <> | 2014-04-24 17:13:11 +0000 |
|---|---|---|
| committer | tedu <> | 2014-04-24 17:13:11 +0000 |
| commit | f6965f84eadd015bfdde6271f13c2ccc18c0cd67 (patch) | |
| tree | 9aa3b2f7e74cea0ace24115598d320b508462aa5 /src | |
| parent | 5a3afc4dd0a88f4ced24a949ec2264fa4dd2ea7f (diff) | |
| download | openbsd-f6965f84eadd015bfdde6271f13c2ccc18c0cd67.tar.gz openbsd-f6965f84eadd015bfdde6271f13c2ccc18c0cd67.tar.bz2 openbsd-f6965f84eadd015bfdde6271f13c2ccc18c0cd67.zip | |
obvious use after free was less obviously not a use after free.
who uses '\0' as a null pointer constant? fix my mistake and restore
original semantics.
Diffstat (limited to 'src')
| -rw-r--r-- | src/lib/libssl/src/ssl/kssl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/libssl/src/ssl/kssl.c b/src/lib/libssl/src/ssl/kssl.c index 9e71eed9fa..b04c83e17f 100644 --- a/src/lib/libssl/src/ssl/kssl.c +++ b/src/lib/libssl/src/ssl/kssl.c | |||
| @@ -963,7 +963,7 @@ kssl_ctx_setstring(KSSL_CTX *kssl_ctx, int which, char *text) | |||
| 963 | free(*string); | 963 | free(*string); |
| 964 | *string = NULL; | 964 | *string = NULL; |
| 965 | 965 | ||
| 966 | if ((*string = strdup(text ? text : "")) == NULL) | 966 | if (text && (*string = strdup(text)) == NULL) |
| 967 | return KSSL_CTX_ERR; | 967 | return KSSL_CTX_ERR; |
| 968 | 968 | ||
| 969 | return KSSL_CTX_OK; | 969 | return KSSL_CTX_OK; |