Commit Diff

Commit:
eed4085c0f4916f79731c3fc1a8adeb149b59a69
From:
Stefan Sperling <stsp@stsp.name>
Date:
Message:
restore abort() calls in lib/hash.c In the spirit of commit b808e01085d775b284878dfeea5c06339a9f2ec2, restore the abort() call in got_hash_final_object_id() and add additional ones to cover other functions in this file. ok op@
Actions:
Patch | Tree
commit - a08fda08dfda5473c47e3d6d0d01fc62d08dd5ec
commit + eed4085c0f4916f79731c3fc1a8adeb149b59a69
blob - d91f248fe08f8bda24931834c38c29dfd8d08fd2
blob + 5911bacfda535885152cd7e9f16c9481094c581c
--- lib/hash.c
+++ lib/hash.c
@@ -198,6 +198,8 @@ got_hash_init(struct got_hash *hash, enum got_hash_alg
 		SHA1Init(&hash->sha1_ctx);
 	else if (algo == GOT_HASH_SHA256)
 		SHA256Init(&hash->sha256_ctx);
+	else
+		abort();
 }
 
 void
@@ -207,6 +209,8 @@ got_hash_update(struct got_hash *hash, const void *dat
 		SHA1Update(&hash->sha1_ctx, data, len);
 	else if (hash->algo == GOT_HASH_SHA256)
 		SHA256Update(&hash->sha256_ctx, data, len);
+	else
+		abort();
 }
 
 void
@@ -216,6 +220,8 @@ got_hash_final(struct got_hash *hash, uint8_t *out)
 		SHA1Final(out, &hash->sha1_ctx);
 	else if (hash->algo == GOT_HASH_SHA256)
 		SHA256Final(out, &hash->sha256_ctx);
+	else
+		abort();
 }
 
 void
@@ -225,8 +231,10 @@ got_hash_final_object_id(struct got_hash *hash, struct
 	id->algo = hash->algo;
 	if (hash->algo == GOT_HASH_SHA1)
 		SHA1Final(id->hash, &hash->sha1_ctx);
-	else
+	else if (hash->algo == GOT_HASH_SHA256)
 		SHA256Final(id->hash, &hash->sha256_ctx);
+	else
+		abort();
 }
 
 int
@@ -236,5 +244,7 @@ got_hash_cmp(enum got_hash_algorithm algo, uint8_t *b1
 		return memcmp(b1, b2, SHA1_DIGEST_LENGTH);
 	else if (algo == GOT_HASH_SHA256)
 		return memcmp(b1, b2, SHA256_DIGEST_LENGTH);
+	else
+		abort();
 	return -1;
 }