Message ID | ce999865-2ae7-4b74-a01f-0a61a2fb613d@moroto.mountain |
---|---|
State | Accepted |
Commit | 939390b203dde8a0176dbfa272dcb7bc54949baf |
Delegated to: | Tom Rini |
Headers | show |
Series | test: fix a couple NULL vs IS_ERR() checks | expand |
Hi Dan, On Mon, 31 Jul 2023 at 08:08, Dan Carpenter <dan.carpenter@linaro.org> wrote: > > The x509_cert_parse() and pkcs7_parse_message() functions return error > pointers. They don't return NULL. Update the checks accordingly. > > Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> > --- > test/lib/asn1.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > Reviewed-by: Simon Glass <sjg@chromium.org> > diff --git a/test/lib/asn1.c b/test/lib/asn1.c > index 8661fdd30687..a66cdd77df0a 100644 > --- a/test/lib/asn1.c > +++ b/test/lib/asn1.c > @@ -120,7 +120,7 @@ static int lib_asn1_x509(struct unit_test_state *uts) > > cert = x509_cert_parse(cert_data, cert_data_len); > > - ut_assertf(cert != NULL, "decoding failed\n"); > + ut_assertf(!IS_ERR(cert), "decoding failed\n"); But how about creating ut_assertnoerr() or something like that? > ut_assertf(!strcmp(cert->subject, "Linaro: Tester"), > "subject doesn't match\n"); > ut_assertf(!strcmp(cert->issuer, "Linaro: Tester"), > @@ -313,7 +313,7 @@ static int lib_asn1_pkcs7(struct unit_test_state *uts) > > pkcs7 = pkcs7_parse_message(image_pk7, image_pk7_len); > > - ut_assertf(pkcs7 != NULL, "decoding failed\n"); > + ut_assertf(!IS_ERR(pkcs7), "decoding failed\n"); > ut_assertf(pkcs7->data_len == 104, "signature size doesn't match\n"); > ut_assertf(pkcs7->signed_infos != NULL, "sign-info doesn't exist\n"); > ut_assertf(pkcs7->signed_infos->msgdigest_len == 32, > -- > 2.39.2 > Regards, Simon
On Mon, Jul 31, 2023 at 05:08:41PM +0300, Dan Carpenter wrote: > The x509_cert_parse() and pkcs7_parse_message() functions return error > pointers. They don't return NULL. Update the checks accordingly. > > Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> > Reviewed-by: Simon Glass <sjg@chromium.org> Applied to u-boot/next, thanks!
diff --git a/test/lib/asn1.c b/test/lib/asn1.c index 8661fdd30687..a66cdd77df0a 100644 --- a/test/lib/asn1.c +++ b/test/lib/asn1.c @@ -120,7 +120,7 @@ static int lib_asn1_x509(struct unit_test_state *uts) cert = x509_cert_parse(cert_data, cert_data_len); - ut_assertf(cert != NULL, "decoding failed\n"); + ut_assertf(!IS_ERR(cert), "decoding failed\n"); ut_assertf(!strcmp(cert->subject, "Linaro: Tester"), "subject doesn't match\n"); ut_assertf(!strcmp(cert->issuer, "Linaro: Tester"), @@ -313,7 +313,7 @@ static int lib_asn1_pkcs7(struct unit_test_state *uts) pkcs7 = pkcs7_parse_message(image_pk7, image_pk7_len); - ut_assertf(pkcs7 != NULL, "decoding failed\n"); + ut_assertf(!IS_ERR(pkcs7), "decoding failed\n"); ut_assertf(pkcs7->data_len == 104, "signature size doesn't match\n"); ut_assertf(pkcs7->signed_infos != NULL, "sign-info doesn't exist\n"); ut_assertf(pkcs7->signed_infos->msgdigest_len == 32,
The x509_cert_parse() and pkcs7_parse_message() functions return error pointers. They don't return NULL. Update the checks accordingly. Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> --- test/lib/asn1.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)