From fd246cd3bca6f8a162fc23c4fb7d3a825de618b8 Mon Sep 17 00:00:00 2001 From: deraadt <> Date: Fri, 20 Jun 2014 21:00:46 +0000 Subject: Remove the OPENSSL_*cap getenv's. A program should not be able to change the behaviour of the library in such a complicated fashion. ok miod --- src/lib/libcrypto/armcap.c | 9 +-------- src/lib/libcrypto/cryptlib.c | 14 +++----------- src/lib/libcrypto/sparcv9cap.c | 7 +------ src/lib/libssl/src/crypto/armcap.c | 9 +-------- src/lib/libssl/src/crypto/cryptlib.c | 14 +++----------- src/lib/libssl/src/crypto/sparcv9cap.c | 7 +------ 6 files changed, 10 insertions(+), 50 deletions(-) (limited to 'src/lib') diff --git a/src/lib/libcrypto/armcap.c b/src/lib/libcrypto/armcap.c index 330fea25a7..7ee94d48b1 100644 --- a/src/lib/libcrypto/armcap.c +++ b/src/lib/libcrypto/armcap.c @@ -1,4 +1,4 @@ -/* $OpenBSD: armcap.c,v 1.5 2014/06/12 15:49:27 deraadt Exp $ */ +/* $OpenBSD: armcap.c,v 1.6 2014/06/20 21:00:46 deraadt Exp $ */ #include #include #include @@ -44,13 +44,6 @@ OPENSSL_cpuid_setup(void) return; trigger = 1; -#ifndef __OpenBSD__ - if ((e = getenv("OPENSSL_armcap"))) { - OPENSSL_armcap_P = strtoul(e, NULL, 0); - return; - } -#endif - OPENSSL_armcap_P = 0; #if __ARM_ARCH__ >= 7 diff --git a/src/lib/libcrypto/cryptlib.c b/src/lib/libcrypto/cryptlib.c index 2cb6e643ce..bae59e2826 100644 --- a/src/lib/libcrypto/cryptlib.c +++ b/src/lib/libcrypto/cryptlib.c @@ -1,4 +1,4 @@ -/* $OpenBSD: cryptlib.c,v 1.26 2014/06/12 15:49:27 deraadt Exp $ */ +/* $OpenBSD: cryptlib.c,v 1.27 2014/06/20 21:00:46 deraadt Exp $ */ /* ==================================================================== * Copyright (c) 1998-2006 The OpenSSL Project. All rights reserved. * @@ -653,20 +653,12 @@ OPENSSL_cpuid_setup(void) static int trigger = 0; IA32CAP OPENSSL_ia32_cpuid(void); IA32CAP vec; - char *env; if (trigger) return; - trigger = 1; - if ((env = getenv("OPENSSL_ia32cap"))) { - int off = (env[0] == '~') ? 1 : 0; - if (!sscanf(env+off, "%lli",(long long *)&vec)) - vec = strtoul(env + off, NULL, 0); - if (off) - vec = OPENSSL_ia32_cpuid() & ~vec; - } else - vec = OPENSSL_ia32_cpuid(); + + vec = OPENSSL_ia32_cpuid(); /* * |(1<<10) sets a reserved bit to signal that variable diff --git a/src/lib/libcrypto/sparcv9cap.c b/src/lib/libcrypto/sparcv9cap.c index 7b1b7ed0a6..cc39c1bf45 100644 --- a/src/lib/libcrypto/sparcv9cap.c +++ b/src/lib/libcrypto/sparcv9cap.c @@ -1,4 +1,4 @@ -/* $OpenBSD: sparcv9cap.c,v 1.6 2014/06/12 15:49:27 deraadt Exp $ */ +/* $OpenBSD: sparcv9cap.c,v 1.7 2014/06/20 21:00:46 deraadt Exp $ */ #include #include #include @@ -53,11 +53,6 @@ OPENSSL_cpuid_setup(void) return; trigger = 1; - if ((e = getenv("OPENSSL_sparcv9cap"))) { - OPENSSL_sparcv9cap_P = strtoul(e, NULL, 0); - return; - } - /* Initial value, fits UltraSPARC-I&II... */ OPENSSL_sparcv9cap_P = SPARCV9_PREFER_FPU; diff --git a/src/lib/libssl/src/crypto/armcap.c b/src/lib/libssl/src/crypto/armcap.c index 330fea25a7..7ee94d48b1 100644 --- a/src/lib/libssl/src/crypto/armcap.c +++ b/src/lib/libssl/src/crypto/armcap.c @@ -1,4 +1,4 @@ -/* $OpenBSD: armcap.c,v 1.5 2014/06/12 15:49:27 deraadt Exp $ */ +/* $OpenBSD: armcap.c,v 1.6 2014/06/20 21:00:46 deraadt Exp $ */ #include #include #include @@ -44,13 +44,6 @@ OPENSSL_cpuid_setup(void) return; trigger = 1; -#ifndef __OpenBSD__ - if ((e = getenv("OPENSSL_armcap"))) { - OPENSSL_armcap_P = strtoul(e, NULL, 0); - return; - } -#endif - OPENSSL_armcap_P = 0; #if __ARM_ARCH__ >= 7 diff --git a/src/lib/libssl/src/crypto/cryptlib.c b/src/lib/libssl/src/crypto/cryptlib.c index 2cb6e643ce..bae59e2826 100644 --- a/src/lib/libssl/src/crypto/cryptlib.c +++ b/src/lib/libssl/src/crypto/cryptlib.c @@ -1,4 +1,4 @@ -/* $OpenBSD: cryptlib.c,v 1.26 2014/06/12 15:49:27 deraadt Exp $ */ +/* $OpenBSD: cryptlib.c,v 1.27 2014/06/20 21:00:46 deraadt Exp $ */ /* ==================================================================== * Copyright (c) 1998-2006 The OpenSSL Project. All rights reserved. * @@ -653,20 +653,12 @@ OPENSSL_cpuid_setup(void) static int trigger = 0; IA32CAP OPENSSL_ia32_cpuid(void); IA32CAP vec; - char *env; if (trigger) return; - trigger = 1; - if ((env = getenv("OPENSSL_ia32cap"))) { - int off = (env[0] == '~') ? 1 : 0; - if (!sscanf(env+off, "%lli",(long long *)&vec)) - vec = strtoul(env + off, NULL, 0); - if (off) - vec = OPENSSL_ia32_cpuid() & ~vec; - } else - vec = OPENSSL_ia32_cpuid(); + + vec = OPENSSL_ia32_cpuid(); /* * |(1<<10) sets a reserved bit to signal that variable diff --git a/src/lib/libssl/src/crypto/sparcv9cap.c b/src/lib/libssl/src/crypto/sparcv9cap.c index 7b1b7ed0a6..cc39c1bf45 100644 --- a/src/lib/libssl/src/crypto/sparcv9cap.c +++ b/src/lib/libssl/src/crypto/sparcv9cap.c @@ -1,4 +1,4 @@ -/* $OpenBSD: sparcv9cap.c,v 1.6 2014/06/12 15:49:27 deraadt Exp $ */ +/* $OpenBSD: sparcv9cap.c,v 1.7 2014/06/20 21:00:46 deraadt Exp $ */ #include #include #include @@ -53,11 +53,6 @@ OPENSSL_cpuid_setup(void) return; trigger = 1; - if ((e = getenv("OPENSSL_sparcv9cap"))) { - OPENSSL_sparcv9cap_P = strtoul(e, NULL, 0); - return; - } - /* Initial value, fits UltraSPARC-I&II... */ OPENSSL_sparcv9cap_P = SPARCV9_PREFER_FPU; -- cgit v1.2.3-55-g6feb