Merge pull request ceph#52491 from cbodley/wip-61566

valgrind: replace suppression for EVP_DecryptFinal_ex

Reviewed-by: Ilya Dryomov <[email protected]>
Reviewed-by: Radoslaw Zarzynski <[email protected]>

Author: idryomov

qa/valgrind.supp

@@ -664,29 +664,14 @@
 }
 
 # "Conditional jump or move depends on uninitialised value(s)" in OpenSSL
-# while using aes-128-gcm with AES-NI enabled. Not observed while running
-# with `OPENSSL_ia32cap="~0x200000200000000"`.
+# https://github.com/openssl/openssl/issues/19719
 {
-   uninitialised gcm.Xi in aes-128-gcm with AES-NI for msgr, part 1
-   Memcheck:Cond
-   ...
-   fun:EVP_DecryptFinal_ex
-   fun:_ZN4ceph6crypto6onwire25AES128GCM_OnWireRxHandler34authenticated_decrypt_update_finalEONS_6buffer7v*4listEj
-   fun:_ZN10ProtocolV231handle_read_frame_epilogue_mainEOSt10unique_ptrIN4ceph6buffer7v*8ptr_nodeENS4_8disposerEEi
-   fun:_ZN10ProtocolV216run_continuationER2CtIS_E
-   ...
-   fun:_ZN15AsyncConnection7processEv
-   fun:_ZN11EventCenter14process_eventsEjPNSt6chrono8durationImSt5ratioILl1ELl1000000000EEEE
-   ...
-}
-
-{
-   uninitialised gcm.Xi in aes-128-gcm with AES-NI for msgr, part 2
-   Memcheck:Cond
-   fun:_ZN4ceph6crypto6onwire25AES128GCM_OnWireRxHandler34authenticated_decrypt_update_finalEONS_6buffer7v*4listEj
-   fun:_ZN10ProtocolV231handle_read_frame_epilogue_mainEOSt10unique_ptrIN4ceph6buffer7v*8ptr_nodeENS4_8disposerEEi
-   fun:_ZN10ProtocolV216run_continuationER2CtIS_E
-   ...
-   fun:_ZN11EventCenter14process_eventsEjPNSt6chrono8durationImSt5ratioILl1ELl1000000000EEEE
-   ...
+  uninitialized value in gcm_cipher_internal
+  Memcheck:Cond
+  ...
+  fun:gcm_cipher_internal
+  ...
+  fun:ossl_gcm_stream_final
+  fun:EVP_DecryptFinal_ex
+  ...
 }