Commit 42119fa3 authored by Markus Armbruster's avatar Markus Armbruster Committed by Michael Tokarev
Browse files

libcacard/vcard_emul_nss: Drop a redundant conditional 



Bailing out when PK11_FindGenericObjects() returns null ensures the
loop that follows it executes at least once.  The "loop did not
execute" test right after it is useless.  Drop it.

Spotted by Coverity.

Signed-off-by: default avatarMarkus Armbruster <armbru@redhat.com>
Reviewed-by: default avatarAlon Levy <alevy@redhat.com>
Signed-off-by: default avatarMichael Tokarev <mjt@tls.msk.ru>
parent 26b78f4d

libcacard/vcard_emul_nss.c

+0 −5
Original line number Diff line number Diff line
@@ -618,11 +618,6 @@ vcard_emul_mirror_card(VReader *vreader) 
        cert_count++;

    }



    if (cert_count == 0) {

        PK11_DestroyGenericObjects(firstObj);

        return NULL;

    }



    /* allocate the arrays */

    vcard_emul_alloc_arrays(&certs, &cert_len, &keys, cert_count);