Oops, "magic eight" is necessary here afterall

- Fix regression from commit 807b402d95702f3f91e9e2bfbd2b5ca8c9964ed9,
  the array gets passed as a pointer (how else would it work at all),
  so despite having seemingly correct type, sizeof(keyid) depends
  on the pointer size. This happens to be 8 on x86_64 and friends
  but breaks pgp fingerprint calculation on eg i386.
- Also return the explicit size from pgpExtractPubkeyFingerprint(),
  this has been "broken" for much longer but then all callers should
  really care about is -1 for error.

1 files changed, 2 insertions, 2 deletions

diff --git a/rpmio/rpmpgp.c b/rpmio/rpmpgp.c
index 4aac23d59..e70cf706b 100644
--- a/rpmio/rpmpgp.c
+++ b/rpmio/rpmpgp.c
@@ -757,7 +757,7 @@ static int getFingerprint(const uint8_t *h, size_t hlen, pgpKeyID_t keyid)
 	    (void) rpmDigestFinal(ctx, (void **)&d, &dlen, 0);
 
 	    if (d) {
-		memcpy(keyid, (d + (dlen-sizeof(keyid))), sizeof(keyid));
+		memcpy(keyid, (d + (dlen-8)), 8);
 		free(d);
 		rc = 0;
 	    }
@@ -787,7 +787,7 @@ int pgpExtractPubkeyFingerprint(const char * b64pkt, pgpKeyID_t keyid)
     if (rpmBase64Decode(b64pkt, (void **)&pkt, &pktlen) == 0) {
 	if (pgpPubkeyFingerprint(pkt, pktlen, keyid) == 0) {
 	    /* if there ever was a bizarre return code for success... */
-	    rc = sizeof(keyid);
+	    rc = 8;
 	}
 	free(pkt);
     }