Commit c6bbba0b28 for openssl.org
commit c6bbba0b28a3758acdc72400280e86fd85235a37
Author: Neil Horman <nhorman@openssl.org>
Date: Wed Feb 11 12:44:35 2026 -0500
Add missing valgrind supressions
Running valgrind in CI triggered a few errors that local testing missed,
add those to the suppression file
Reviewed-by: Eugene Syromiatnikov <esyr@openssl.org>
Reviewed-by: Norbert Pocs <norbertp@openssl.org>
MergeDate: Tue Feb 24 15:11:14 2026
(Merged from https://github.com/openssl/openssl/pull/30003)
diff --git a/util/perl/OpenSSL/Test.pm b/util/perl/OpenSSL/Test.pm
index 529f4e4c52..1034533251 100644
--- a/util/perl/OpenSSL/Test.pm
+++ b/util/perl/OpenSSL/Test.pm
@@ -333,7 +333,7 @@ sub app {
$idx=$idx+1;
my $resultdir = result_dir();
my $srcdir = srctop_dir();
- return cmd([ "valgrind", "--leak-check=full", "--show-leak-kinds=all", "--gen-suppressions=all", "--log-file=$resultdir/valgrind.log.$idx", "--suppressions=$srcdir/util/valgrind.suppression", @prog, @cmdargs ],
+ return cmd([ "valgrind", "--leak-check=full", "--show-leak-kinds=all", "--gen-suppressions=all", "--suppressions=$srcdir/util/valgrind.suppression", "--log-file=$resultdir/valgrind.log.$idx", "--suppressions=$srcdir/util/valgrind.suppression", @prog, @cmdargs ],
exe_shell => $ENV{EXE_SHELL}, %opts) -> (shift);
} else {
return cmd([ @prog, @cmdargs ],
@@ -363,7 +363,7 @@ sub test {
$idx=$idx+1;
my $resultdir = result_dir();
my $srcdir = srctop_dir();
- return cmd([ "valgrind", "--leak-check=full", "--show-leak-kinds=all", "--gen-suppressions=all", "--log-file=$resultdir/valgrind.log.$idx", "--suppressions=$srcdir/util/valgrind.suppression", @prog, @cmdargs ],
+ return cmd([ "valgrind", "--leak-check=full", "--show-leak-kinds=all", "--gen-suppressions=all", "--suppressions=$srcdir/util/valgrind.suppression", "--log-file=$resultdir/valgrind.log.$idx", "--suppressions=$srcdir/util/valgrind.suppression", @prog, @cmdargs ],
exe_shell => $ENV{EXE_SHELL}, %opts) -> (shift);
} else {
return cmd([ @prog, @cmdargs ],
diff --git a/util/valgrind.suppression b/util/valgrind.suppression
index f34fce4672..4514e702b0 100644
--- a/util/valgrind.suppression
+++ b/util/valgrind.suppression
@@ -114,3 +114,32 @@
fun:ssl_release_record
...
}
+{
+ provider_activate_strdup
+ Memcheck:Leak
+ match-leak-kinds: reachable
+ fun:malloc
+ ...
+ fun:OSSL_provider_init
+ fun:provider_activate
+ ...
+}
+{
+ false_positive_conditional_check
+ Memcheck:Cond
+ fun:bcmp
+ fun:test_mem_eq
+ ...
+}
+{
+ false_positive_conditional_check
+ Memcheck:Cond
+ fun:ossl_gcm_stream_final
+ ...
+}
+{
+ false_positive_conditional_check
+ Memcheck:Cond
+ fun:gcm_cipher_internal
+ ...
+}