Redundant Condition in Cipher Mode Check

[XiaoLey]

Description

It has been observed that there are two lines of code where the condition check for the cipher mode appears to be redundant. Specifically, both instances check for the same condition twice within a single statement.

Code Location

The redundant condition appears in two different locations:

Link to code location 1

Link to code location 2

Possible Typo?

It might be a typo that the second condition was intended to check another cipher mode, potentially cipher_aes128cfb. It's worth verifying whether this was the intended logic.

Suggested Action

It would be advisable to review these conditions and correct them if one of them should indeed be cipher_aes128cfb, or if another cipher mode was intended to be checked.

[wangyu-]

Hi this is a bug, if --fix-gro is enabled for cipher_aes128cfb the 16bit gro header is by mistake left unencrpyted.

I will make a fix, hopefully next weekend.

Thanks for reporting.