diff options
author | Matt Caswell <matt@openssl.org> | 2015-10-09 15:48:30 +0100 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2015-11-20 23:35:19 +0000 |
commit | 4abc76811a74b45c8bab1bbe3add4b58b374a04b (patch) | |
tree | a531035908ff6e2bbf713e0ea4a6e19fa1ce8beb | |
parent | 05a6347fec744bd11ef94dd2ac9f68b4f679de8e (diff) | |
download | openssl-4abc76811a74b45c8bab1bbe3add4b58b374a04b.tar.gz |
Fix Linux crash
If config'd without -d (--debug), asynctest was crashing with:
*** longjmp causes uninitialized stack frame ***
This is because gcc will add certain checks for some functions
(including longjmp). The checks assume you can only longjmp down the
stack not up. However, if we are actually jumping to a different
fibre then it can appear as if we are going up the stack when we are
not really. This change disables the check.
Reviewed-by: Rich Salz <rsalz@openssl.org>
-rw-r--r-- | crypto/async/async.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/crypto/async/async.c b/crypto/async/async.c index 8fdff52d99..d08ac132b7 100644 --- a/crypto/async/async.c +++ b/crypto/async/async.c @@ -51,6 +51,14 @@ * ==================================================================== */ +/* + * Without this we start getting longjmp crashes because it thinks we're jumping + * up the stack when in fact we are jumping to an entirely different stack. The + * cost of this is not having certain buffer overrun/underrun checks etc for + * this source file :-( + */ +#undef _FORTIFY_SOURCE + #include <openssl/err.h> #include <openssl/async.h> #include <string.h> |