| Message ID | 20230901123226.3656164-1-armbru@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | crypto/rsakey-builtin.c.inc: Clean up two error paths | expand |
On Fri, Sep 01, 2023 at 02:32:26PM +0200, Markus Armbruster wrote: > When qcrypto_builtin_rsa_public_key_parse() is about to fail, but no > error has been set, it makes one up. Actually, there's just one way > to fail without setting an error. Set it there instead. > > Same for qcrypto_builtin_rsa_private_key_parse(). > > Signed-off-by: Markus Armbruster <armbru@redhat.com> > --- > crypto/rsakey-builtin.c.inc | 8 ++------ > 1 file changed, 2 insertions(+), 6 deletions(-) Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> and queued. With regards, Daniel
diff --git a/crypto/rsakey-builtin.c.inc b/crypto/rsakey-builtin.c.inc index aeeacc8f9b..46cc7afe87 100644 --- a/crypto/rsakey-builtin.c.inc +++ b/crypto/rsakey-builtin.c.inc @@ -88,15 +88,13 @@ static QCryptoAkCipherRSAKey *qcrypto_builtin_rsa_public_key_parse( goto error; } if (seq_length != 0) { + error_setg(errp, "Invalid RSA public key"); goto error; } return rsa; error: - if (errp && !*errp) { - error_setg(errp, "Invalid RSA public key"); - } qcrypto_akcipher_rsakey_free(rsa); return NULL; } @@ -169,15 +167,13 @@ static QCryptoAkCipherRSAKey *qcrypto_builtin_rsa_private_key_parse( return rsa; } if (seq_length != 0) { + error_setg(errp, "Invalid RSA private key"); goto error; } return rsa; error: - if (errp && !*errp) { - error_setg(errp, "Invalid RSA private key"); - } qcrypto_akcipher_rsakey_free(rsa); return NULL; }
When qcrypto_builtin_rsa_public_key_parse() is about to fail, but no error has been set, it makes one up. Actually, there's just one way to fail without setting an error. Set it there instead. Same for qcrypto_builtin_rsa_private_key_parse(). Signed-off-by: Markus Armbruster <armbru@redhat.com> --- crypto/rsakey-builtin.c.inc | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-)