src: remove redundant CHECK

tniessen · aduh95 · commit 41749eb5d6cd · 2026-02-24T00:37:08.000+01:00
The function `SetAuthTag()` returns early (before reaching this line) if `!cipher->IsAuthenticatedMode()`, which expands to the exact same condition that is being `CHECK()`ed here. PR-URL: #61130 Reviewed-By: Xuguang Mei <meixuguang@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com>
diff --git a/src/crypto/crypto_cipher.cc b/src/crypto/crypto_cipher.cc
@@ -635,7 +635,6 @@ void CipherBase::SetAuthTag(const FunctionCallbackInfo<Value>& args) {
   } else {
     // At this point, the tag length is already known and must match the
     // length of the given authentication tag.
-    CHECK(Cipher::FromCtx(cipher->ctx_).isSupportedAuthenticatedMode());
     CHECK_NE(cipher->auth_tag_len_, kNoAuthTagLength);
     is_valid = cipher->auth_tag_len_ == tag_len;
   }

Original file line number	Diff line number	Diff line change
`@@ -635,7 +635,6 @@ void CipherBase::SetAuthTag(const FunctionCallbackInfo<Value>& args) {`
`635`	`635`	`} else {`
`636`	`636`	`// At this point, the tag length is already known and must match the`
`637`	`637`	`// length of the given authentication tag.`
`638`		`- CHECK(Cipher::FromCtx(cipher->ctx_).isSupportedAuthenticatedMode());`
`639`	`638`	`CHECK_NE(cipher->auth_tag_len_, kNoAuthTagLength);`
`640`	`639`	`is_valid = cipher->auth_tag_len_ == tag_len;`
`641`	`640`	`}`