Re: [PATCH] serial: qcom-geni: Fix off-by-one error in ida_alloc_range()

[Greg KH <gregkh@linuxfoundation.org>]

On Fri, Aug 22, 2025 at 11:35:32AM +0800, Zong Jiang wrote:
> The ida_alloc_range() function expects an inclusive range, meaning both
> the start and end values are valid allocation targets. Passing nr_ports
> as the upper bound allows allocation of an ID equal to nr_ports, which
> is out of bounds when used as an index into the port array.
> 
> Fix this by subtracting 1 from nr_ports in both calls to ida_alloc_range(),
> ensuring the allocated ID stays within the valid range
> [start, nr_ports - 1].
> 
> This prevents potential out-of-bounds access when the allocated ID is used
> as an index.
> 
> | Reported-by: kernel test robot <lkp@intel.com>
> | Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
> | Closes: https://lore.kernel.org/r/202508180815.R2nDyajs-lkp@intel.com/

Why do you have "| " here?  Have you ever seen that in other kernel
commits?


> Signed-off-by: Zong Jiang <quic_zongjian@quicinc.com>

What commit id does this fix?

thanks,

greg k-h