Re: [PATCH] staging: r8188eu: remove unnecessary null check

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 24, 2022 at 08:03:50AM +0000, cgel.zte@xxxxxxxxx wrote:
> From: Minghao Chi <chi.minghao@xxxxxxxxxx>
> 
> container_of is never null, so this null check is
> unnecessary.
> 

I can't Ack a patch with this commit message because container_of()
*CAN* be NULL.  Here, it requires two things:
1) That ->list is the first struct member of struct wlan_network which
   is true.
2) That "pmlmepriv->pscanned" is NULL.  Which I have not looked at.

It's really ugly to check container_of() for NULL but some people do it
deliberately.  Some people also will add a build time assert to ensure
that ->list is always the first element so that the check always works.

regards,
dan carpenter





[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux