summaryrefslogtreecommitdiffstats
path: root/providers
diff options
context:
space:
mode:
authorPauli <paul.dale@oracle.com>2020-05-21 13:38:35 +1000
committerPauli <paul.dale@oracle.com>2020-05-22 17:23:49 +1000
commite5cb3453fba01c264636d54440ca0eb81d1fcd6e (patch)
treeb6856f3fa30cd3f275694176d44d6436fd08b7f3 /providers
parent084b7bec0f615f70c108dfba988ed43d544e00ed (diff)
Coverity 1463576: Error handling issues (CHECKED_RETURN)
Reviewed-by: Kurt Roeckx <kurt@roeckx.be> (Merged from https://github.com/openssl/openssl/pull/11892)
Diffstat (limited to 'providers')
-rw-r--r--providers/implementations/serializers/serializer_rsa.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/providers/implementations/serializers/serializer_rsa.c b/providers/implementations/serializers/serializer_rsa.c
index ac685a09f2..7cc6027636 100644
--- a/providers/implementations/serializers/serializer_rsa.c
+++ b/providers/implementations/serializers/serializer_rsa.c
@@ -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);
/*