diff options
| author | doug <> | 2015-02-09 07:17:55 +0000 |
|---|---|---|
| committer | doug <> | 2015-02-09 07:17:55 +0000 |
| commit | c1f670c471e829ef687e5148edca56ef2101455f (patch) | |
| tree | 1ce81442299adac00144d8dfeee772e1d60ceae3 /src/lib/libcrypto/dsa | |
| parent | 44111c3c68252aa9db466727fb182b7b3c032efc (diff) | |
| download | openbsd-c1f670c471e829ef687e5148edca56ef2101455f.tar.gz openbsd-c1f670c471e829ef687e5148edca56ef2101455f.tar.bz2 openbsd-c1f670c471e829ef687e5148edca56ef2101455f.zip | |
Return NULL when there are no shared ciphers.
OpenSSL added this change to avoid an out-of-bounds write since
they're accessing p[-1]. We initialize buf and use strrchr() so we
aren't subject to the same OOB write.
However, we should return NULL rather than an empty string when there
are no shared ciphers.
Also, KNF a particularly bad section above here that miod noticed.
Based on OpenSSL commits:
4ee356686f72ff849f6f3d58562224ace732b1a6
308505b838e4e3ce8485bb30f5b26e2766dc7f8b
ok miod@
Diffstat (limited to 'src/lib/libcrypto/dsa')
0 files changed, 0 insertions, 0 deletions