diff options
author | Theo de Raadt <deraadt@cvs.openbsd.org> | 2014-05-30 20:10:01 +0000 |
---|---|---|
committer | Theo de Raadt <deraadt@cvs.openbsd.org> | 2014-05-30 20:10:01 +0000 |
commit | bfd6ef1d500dac8515b5b20643749b6aa98a2660 (patch) | |
tree | 0e9c291e4a8a1fb1544edb800d2503c5c6f01aca /lib/libcrypto | |
parent | 217d010e19e1933f05e4cdddfa2bca09042f2156 (diff) |
Don't add potentially nasty stderr uses to dead CRYPTO_dbg_mem functions.
But do use the abort(), which we are hoping all future vendors will move
towards the more modern "do not flush streams"; hint hint, if you didn't
do that already, there are grave risks because much software brings risk
without that behaviour. We didn't cause the change.. POSIX did...
ok beck
Diffstat (limited to 'lib/libcrypto')
-rw-r--r-- | lib/libcrypto/mem_dbg.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/libcrypto/mem_dbg.c b/lib/libcrypto/mem_dbg.c index 288ee907c34..3e56c79a25a 100644 --- a/lib/libcrypto/mem_dbg.c +++ b/lib/libcrypto/mem_dbg.c @@ -162,14 +162,14 @@ void CRYPTO_dbg_malloc(void *addr, int num, const char *file, int line, int before_p) { - fprintf(stderr, "this is a bad idea"); + /* CRYPTO_dbg_malloc is no longer permitted */ abort(); } void CRYPTO_dbg_free(void *addr, int before_p) { - fprintf(stderr, "this is a bad idea"); + /* CRYPTO_dbg_free is no longer permitted */ abort(); } @@ -177,7 +177,7 @@ void CRYPTO_dbg_realloc(void *addr1, void *addr2, int num, const char *file, int line, int before_p) { - fprintf(stderr, "this is a bad idea"); + /* CRYPTO_dbg_realloc is no longer permitted */ abort(); } |