X-Git-Url: https://the.earth.li/gitweb/?a=blobdiff_plain;f=sigcheck.c;h=74c2e2bf295cd24ceca0b98ba7c6f54a2c28d0cc;hb=f6d38365b7e7e1e88ba02136ffe6f0fb08903ef0;hp=29ab652e71dda830273b4f45c1bd994094ece9e0;hpb=3a9aaa4369f8078b801cddd2353db5730d2388c8;p=onak.git

diff --git a/sigcheck.c b/sigcheck.c
index 29ab652..74c2e2b 100644
--- a/sigcheck.c
+++ b/sigcheck.c
@@ -116,7 +116,7 @@ static void onak_free_key_material(struct onak_key_material *key)
 static onak_status_t onak_parse_key_material(struct openpgp_packet *pk,
 		struct onak_key_material *key)
 {
-	int i, len, ofs;
+	int len, ofs;
 	enum onak_oid oid;
 	mpz_t x, y;
 	onak_status_t ret = ONAK_E_OK;
@@ -263,6 +263,12 @@ static onak_status_t onak_parse_key_material(struct openpgp_packet *pk,
 		return ONAK_E_UNSUPPORTED_FEATURE;
 	}
 
+	/*
+	 * Keep scan-build happy; we bump this in MPI_TO_MPZ and then don't use
+	 * it again the last time we do so.
+	 */
+	(void)ofs;
+
 	key->type = pk->data[5];
 
 	if (ret != ONAK_E_OK) {
@@ -282,7 +288,6 @@ onak_status_t onak_check_hash_sig(struct openpgp_publickey *sigkey,
 	struct dsa_signature dsasig;
 	uint8_t sigkeytype;
 	uint8_t edsig[64];
-	uint64_t keyid;
 	int len, ofs;
 	mpz_t s;
 
@@ -311,7 +316,7 @@ onak_status_t onak_check_hash_sig(struct openpgp_publickey *sigkey,
 		}
 
 		/* Skip to the signature material */
-		ofs += 19;
+		ofs = 19;
 		sigkeytype = sig->data[15];
 	} else if (sig->data[0] >= 4) {
 		/* Skip the hashed data */
@@ -524,7 +529,6 @@ onak_status_t calculate_packet_sighash(struct openpgp_publickey *key,
 	uint8_t keyheader[5];
 	uint8_t packetheader[5];
 	uint8_t trailer[10];
-	int i;
 	uint64_t keyid;
 	onak_status_t res;