Index | Thread | Search

From:
Stefan Sperling <stsp@stsp.name>
Subject:
Re: sys/iwx: removed dead code by Coverity scan report
To:
tech@openbsd.org
Date:
Mon, 27 Apr 2026 18:27:34 +0200

Download raw body.

Thread
On Mon, Apr 27, 2026 at 06:15:48PM +0200, Kirill A. Korinsky wrote:
> tech@,
> 
> daniel@ forwarded to me a coverity scan report which highligh a now dead
> code from one of the last commit to iwx made by me.

I don't agree with Coverity about having to remove this check.
There is no bug and larger aggsize values might become possible when moving
to 11ax and 11be. I fear that if we remove this check we might be setting
ourselves up for trouble once someone tries to get 11ax or 11be working.
I haven't checked whether the 11ax/11be specs allow for more than 4M but
it is a possibility.