pgpPubkeyFingerprint() can fail, propagate errors part II

- rpmPubkeyNew() needs to return NULL if we fail to grab the
  keyid, make it so...

1 files changed, 5 insertions, 1 deletions

diff --git a/rpmio/rpmkeyring.c b/rpmio/rpmkeyring.c
index 784bc3a48..208e29078 100644
--- a/rpmio/rpmkeyring.c
+++ b/rpmio/rpmkeyring.c
@@ -124,16 +124,20 @@ exit:
 rpmPubkey rpmPubkeyNew(const uint8_t *pkt, size_t pktlen)
 {
     rpmPubkey key = NULL;
+    pgpKeyID_t keyid;
     
     if (pkt == NULL || pktlen == 0)
 	goto exit;
 
+    if (pgpPubkeyFingerprint(pkt, pktlen, keyid))
+	goto exit;
+
     key = xcalloc(1, sizeof(*key));
-    pgpPubkeyFingerprint(pkt, pktlen, key->keyid);
     key->pkt = xmalloc(pktlen);
     key->pktlen = pktlen;
     key->nrefs = 0;
     memcpy(key->pkt, pkt, pktlen);
+    memcpy(key->keyid, keyid, sizeof(keyid));
 
 exit:
     return rpmPubkeyLink(key);