]> granicus.if.org Git - python/commit
bpo-34759: Fix error handling in ssl 'unwrap()' (GH-9468)
authorNathaniel J. Smith <njs@pobox.com>
Sat, 22 Sep 2018 04:44:12 +0000 (21:44 -0700)
committerMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Sat, 22 Sep 2018 04:44:12 +0000 (21:44 -0700)
commitc0da582b227f311126e278b5553a7fa89c79b054
tree9b858acb418f0a5225146bfc2c78638ccb84b3ac
parent026337a7101369297c8083047d2f3c6fc9dd1e2b
bpo-34759: Fix error handling in ssl 'unwrap()' (GH-9468)

OpenSSL follows the convention that whenever you call a function, it
returns an error indicator value; and if this value is negative, then
you need to go look at the actual error code to see what happened.

Commit c6fd1c1c3a introduced a small mistake in
_ssl__SSLSocket_shutdown_impl: instead of checking whether the error
indicator was negative, it started checking whether the actual error
code was negative, and it turns out that the error codes are never
negative. So the effect was that 'unwrap()' lost the ability to raise
SSL errors.

https://bugs.python.org/issue34759
Lib/test/test_ssl.py
Modules/_ssl.c