From f4aec38233010953cea72c367bccc71c3687b2f1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nguy=E1=BB=85n=20H=E1=BB=93ng=20Qu=C3=A2n?= Date: Tue, 16 Apr 2013 16:02:17 +0700 Subject: [PATCH 17/26] OpenPGP: Make code neater --- src/libopensc/card-openpgp.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) Index: opensc-20150513/src/libopensc/card-openpgp.c =================================================================== --- opensc-20150513.orig/src/libopensc/card-openpgp.c +++ opensc-20150513/src/libopensc/card-openpgp.c @@ -1230,10 +1230,7 @@ static int gnuk_write_certificate(sc_car r = sc_transmit_apdu(card, &apdu); LOG_TEST_RET(card->ctx, r, "APDU transmit failed"); /* Check response */ - r = sc_check_sw(card, apdu.sw1, apdu.sw2); - if (r < 0) - LOG_FUNC_RETURN(card->ctx, r); - LOG_FUNC_RETURN(card->ctx, length); + LOG_TEST_RET(card->ctx, sc_check_sw(card, apdu.sw1, apdu.sw2), "Certificate writing failed"); } /* Ref: gnuk_put_binary_libusb.py and gnuk_token.py in Gnuk source tree */ @@ -1270,8 +1267,7 @@ static int gnuk_write_certificate(sc_car r = sc_transmit_apdu(card, &apdu); LOG_TEST_RET(card->ctx, r, "APDU transmit failed"); /* Check response */ - r = sc_check_sw(card, apdu.sw1, apdu.sw2); - LOG_TEST_RET(card->ctx, r, "UPDATE BINARY returned error"); + LOG_TEST_RET(card->ctx, sc_check_sw(card, apdu.sw1, apdu.sw2), "UPDATE BINARY returned error"); /* To next part */ i++;