diff options
author | Andy Polyakov <appro@openssl.org> | 2015-05-11 12:16:01 +0200 |
---|---|---|
committer | Andy Polyakov <appro@openssl.org> | 2015-05-13 16:48:08 +0200 |
commit | 34f39b062c76fbd3082521b26edee7f53afc061d (patch) | |
tree | e5058e3e87893ecaa721a629f439d71db52656b9 | |
parent | 6db8e3bdc9ef83d83b83f3eec9722c96daa91f82 (diff) |
util/incore update that allows FINGERPRINT_premain-free build.OpenSSL-fips-2_0_10
As for complementary fips.c modification. Goal is to ensure that
FIPS_signature does not end up in .bss segment, one guaranteed to
be zeroed upon program start-up. One would expect explicitly
initialized values to end up in .data segment, but it turned out
that values explicitly initialized with zeros can end up in .bss.
The modification does not affect program flow, because first byte
was the only one of significance [to FINGERPRINT_premain].
Reviewed-by: Dr. Stephen Henson <steve@openssl.org>
-rw-r--r-- | fips/fips.c | 2 | ||||
-rwxr-xr-x | util/incore | 7 |
2 files changed, 6 insertions, 3 deletions
diff --git a/fips/fips.c b/fips/fips.c index 8c9e187d7b..0269609a7e 100644 --- a/fips/fips.c +++ b/fips/fips.c @@ -151,7 +151,7 @@ extern const unsigned char FIPS_rodata_start[], FIPS_rodata_end[]; #ifdef _TMS320C6X const #endif -unsigned char FIPS_signature [20] = { 0 }; +unsigned char FIPS_signature [20] = { 0, 0xff }; __fips_constseg static const char FIPS_hmac_key[]="etaonrishdlcupfm"; diff --git a/util/incore b/util/incore index e6e6ecfd89..bb765b1966 100755 --- a/util/incore +++ b/util/incore @@ -382,7 +382,7 @@ if (!$legacy_mode) { } $FINGERPRINT_ascii_value - = $exe->Lookup("FINGERPRINT_ascii_value") or die; + = $exe->Lookup("FINGERPRINT_ascii_value"); } if ($FIPS_text_startX && $FIPS_text_endX) { @@ -439,9 +439,12 @@ $fingerprint = FIPS_incore_fingerprint(); if ($legacy_mode) { print unpack("H*",$fingerprint); -} else { +} elsif (defined($FINGERPRINT_ascii_value)) { seek(FD,$FINGERPRINT_ascii_value->{st_offset},0) or die "$!"; print FD unpack("H*",$fingerprint) or die "$!"; +} else { + seek(FD,$FIPS_signature->{st_offset},0) or die "$!"; + print FD $fingerprint or die "$!"; } close (FD); |