Coverity 1463576: Error handling issues (CHECKED_RETURN)
authorPauli <paul.dale@oracle.com>
Thu, 21 May 2020 03:38:35 +0000 (13:38 +1000)
committerPauli <paul.dale@oracle.com>
Fri, 22 May 2020 07:23:49 +0000 (17:23 +1000)
Reviewed-by: Kurt Roeckx <kurt@roeckx.be>
(Merged from https://github.com/openssl/openssl/pull/11892)

providers/implementations/serializers/serializer_rsa.c

index ac685a09f2b9ff844acc9045c922fd621d8073f3..7cc6027636c03dd52be4b9cbb0c3af18fb65f984 100644 (file)
@@ -215,9 +215,9 @@ int ossl_prov_prepare_rsa_params(const void *rsa, int nid,
                     break;
                 }
                 if (!DER_w_RSASSA_PSS_params(&pkt, -1, pss)
-                    || !WPACKET_finish(&pkt))
+                    || !WPACKET_finish(&pkt)
+                    || !WPACKET_get_total_written(&pkt, &str_sz))
                     goto err;
-                WPACKET_get_total_written(&pkt, &str_sz);
                 WPACKET_cleanup(&pkt);
 
                 /*