Removed asserts followed by error handling (cloudflare#142)

The removed asserts resulted in the error handling code being unreachable.
sharjeel · Sep 10, 2020 · 2660747 · 2660747
1 parent f824287
commit 2660747
Showing 1 changed file with 0 additions and 2 deletions.
diff --git a/src/crypto/blake2s/mod.rs b/src/crypto/blake2s/mod.rs
@@ -371,8 +371,6 @@ impl Blake2s {
 
 /// Checks if both MACs are equal in constant time.
 pub fn constant_time_mac_check(mac1: &[u8], mac2: &[u8]) -> Result<(), WireGuardError> {
-    assert!(mac1.len() == 16);
-    assert!(mac2.len() == 16);
     if mac1.len() != 16 || mac2.len() != 16 {
         return Err(WireGuardError::InvalidMac);
     }