Re: [PATCH] scsi: ufs: ufshcd-pltfrm: Signedness bug in ufshcd_parse_clock_info()

From: Manivannan Sadhasivam
Date: Fri Aug 16 2024 - 02:36:29 EST


On Fri, Aug 16, 2024 at 12:35:22AM +0300, Dan Carpenter wrote:
> On Thu, Aug 15, 2024 at 10:47:30AM -0700, Bart Van Assche wrote:
> > On 8/15/24 4:24 AM, Dan Carpenter wrote:
> > > The "sz" variable needs to be a signed type for the error handling to
> > > work as intended.
> >
> > What error handling are you referring to? I haven't found any code that
> > assigns a negative value to 'sz' in ufshcd_parse_clock_info(). Did I
> > perhaps overlook something?
> >
>
> Rob's patch in linux-next.
>

It would've been helpful if you added 'next' in the patch subject prefix.

Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx>

- Mani

> - if (!of_get_property(np, "freq-table-hz", &len)) {
> + sz = of_property_count_u32_elems(np, "freq-table-hz");
> + if (sz <= 0) {
> dev_info(dev, "freq-table-hz property not specified\n");
> goto out;
>
> regards,
> dan carpenter
>

--
மணிவண்ணன் சதாசிவம்