fix: remove spurious length check

Signed-off-by: Joshua Lock <jlock@vmware.com>
theupdateframework · Jun 23, 2022 · 39b9b0f · 39b9b0f
1 parent d73f89f
commit 39b9b0f
Showing 1 changed file with 15 additions and 16 deletions.
diff --git a/util/util.go b/util/util.go
@@ -92,24 +92,23 @@ func BytesMatchLenAndHashes(fetched []byte, length int64, hashes data.Hashes) er
 		return ErrWrongLength{length, flen}
 	}
 
-	if len(hashes) != 0 {
-		for alg, expected := range hashes {
-			var h hash.Hash
-			switch alg {
-			case "sha256":
-				h = sha256.New()
-			case "sha512":
-				h = sha512.New()
-			default:
-				return ErrUnknownHashAlgorithm{alg}
-			}
-			h.Write(fetched)
-			hash := h.Sum(nil)
-			if !hmac.Equal(hash, expected) {
-				return ErrWrongHash{alg, expected, hash}
-			}
+	for alg, expected := range hashes {
+		var h hash.Hash
+		switch alg {
+		case "sha256":
+			h = sha256.New()
+		case "sha512":
+			h = sha512.New()
+		default:
+			return ErrUnknownHashAlgorithm{alg}
+		}
+		h.Write(fetched)
+		hash := h.Sum(nil)
+		if !hmac.Equal(hash, expected) {
+			return ErrWrongHash{alg, expected, hash}
 		}
 	}
+
 	return nil
 }