Commit 8f46bd9
src: remove redundant CHECK
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>1 parent 6330385 commit 8f46bd9
1 file changed
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -549,7 +549,6 @@ void CipherBase::SetAuthTag(const FunctionCallbackInfo<Value>& args) { | |||
| 549 | 549 | } else { | |
| 550 | 550 | // At this point, the tag length is already known and must match the | |
| 551 | 551 | // length of the given authentication tag. | |
| 552 | - CHECK(Cipher::FromCtx(cipher->ctx_).isSupportedAuthenticatedMode()); | ||
| 553 | 552 | CHECK_NE(cipher->auth_tag_len_, kNoAuthTagLength); | |
| 554 | 553 | is_valid = cipher->auth_tag_len_ == tag_len; | |
| 555 | 554 | } | |
0 commit comments