summaryrefslogtreecommitdiff
path: root/lib/libcrypto
diff options
context:
space:
mode:
authorTheo de Raadt <deraadt@cvs.openbsd.org>2014-05-30 20:10:01 +0000
committerTheo de Raadt <deraadt@cvs.openbsd.org>2014-05-30 20:10:01 +0000
commitbfd6ef1d500dac8515b5b20643749b6aa98a2660 (patch)
tree0e9c291e4a8a1fb1544edb800d2503c5c6f01aca /lib/libcrypto
parent217d010e19e1933f05e4cdddfa2bca09042f2156 (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.c6
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();
}