You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
75 lines
2.3 KiB
75 lines
2.3 KiB
2 years ago
|
From e5fd1728ef4c7a5bf7c7a7163ca60370460a6e23 Mon Sep 17 00:00:00 2001
|
||
|
|
From: Tomas Mraz <tomas@openssl.org>
|
||
|
|
Date: Tue, 26 Apr 2022 12:40:24 +0200
|
||
|
|
Subject: [PATCH] c_rehash: Do not use shell to invoke openssl
|
||
|
|
|
||
|
|
Except on VMS where it is safe.
|
||
|
|
|
||
|
|
This fixes CVE-2022-1292.
|
||
|
|
|
||
|
|
Reviewed-by: Matthias St. Pierre <Matthias.St.Pierre@ncp-e.com>
|
||
|
|
Reviewed-by: Matt Caswell <matt@openssl.org>
|
||
|
|
Upstream-Status: Backport [https://github.com/openssl/openssl/commit/e5fd1728ef4c7a5bf7c7a7163ca60370460a6e23]
|
||
|
|
---
|
||
|
|
tools/c_rehash.in | 29 +++++++++++++++++++++++++----
|
||
|
|
1 file changed, 25 insertions(+), 4 deletions(-)
|
||
|
|
|
||
|
|
diff --git a/tools/c_rehash.in b/tools/c_rehash.in
|
||
|
|
index fa7c6c9fef91..83c1cc80e08a 100644
|
||
|
|
--- a/tools/c_rehash.in
|
||
|
|
+++ b/tools/c_rehash.in
|
||
|
|
@@ -152,6 +152,23 @@ sub check_file {
|
||
|
|
return ($is_cert, $is_crl);
|
||
|
|
}
|
||
|
|
|
||
|
|
+sub compute_hash {
|
||
|
|
+ my $fh;
|
||
|
|
+ if ( $^O eq "VMS" ) {
|
||
|
|
+ # VMS uses the open through shell
|
||
|
|
+ # The file names are safe there and list form is unsupported
|
||
|
|
+ if (!open($fh, "-|", join(' ', @_))) {
|
||
|
|
+ print STDERR "Cannot compute hash on '$fname'\n";
|
||
|
|
+ return;
|
||
|
|
+ }
|
||
|
|
+ } else {
|
||
|
|
+ if (!open($fh, "-|", @_)) {
|
||
|
|
+ print STDERR "Cannot compute hash on '$fname'\n";
|
||
|
|
+ return;
|
||
|
|
+ }
|
||
|
|
+ }
|
||
|
|
+ return (<$fh>, <$fh>);
|
||
|
|
+}
|
||
|
|
|
||
|
|
# Link a certificate to its subject name hash value, each hash is of
|
||
|
|
# the form <hash>.<n> where n is an integer. If the hash value already exists
|
||
|
|
@@ -161,10 +178,12 @@ sub check_file {
|
||
|
|
|
||
|
|
sub link_hash_cert {
|
||
|
|
my $fname = $_[0];
|
||
|
|
- $fname =~ s/\"/\\\"/g;
|
||
|
|
- my ($hash, $fprint) = `"$openssl" x509 $x509hash -fingerprint -noout -in "$fname"`;
|
||
|
|
+ my ($hash, $fprint) = compute_hash($openssl, "x509", $x509hash,
|
||
|
|
+ "-fingerprint", "-noout",
|
||
|
|
+ "-in", $fname);
|
||
|
|
chomp $hash;
|
||
|
|
chomp $fprint;
|
||
|
|
+ return if !$hash;
|
||
|
|
$fprint =~ s/^.*=//;
|
||
|
|
$fprint =~ tr/://d;
|
||
|
|
my $suffix = 0;
|
||
|
|
@@ -202,10 +221,12 @@ sub link_hash_cert {
|
||
|
|
|
||
|
|
sub link_hash_crl {
|
||
|
|
my $fname = $_[0];
|
||
|
|
- $fname =~ s/'/'\\''/g;
|
||
|
|
- my ($hash, $fprint) = `"$openssl" crl $crlhash -fingerprint -noout -in '$fname'`;
|
||
|
|
+ my ($hash, $fprint) = compute_hash($openssl, "crl", $crlhash,
|
||
|
|
+ "-fingerprint", "-noout",
|
||
|
|
+ "-in", $fname);
|
||
|
|
chomp $hash;
|
||
|
|
chomp $fprint;
|
||
|
|
+ return if !$hash;
|
||
|
|
$fprint =~ s/^.*=//;
|
||
|
|
$fprint =~ tr/://d;
|
||
|
|
my $suffix = 0;
|