29 Jan
2023
29 Jan
'23
8:09 a.m.
On Sun, Jan 29, 2023 at 02:54:43PM +0800, Zhou Furong wrote:
+void *snd_soc_usb_get_priv_data(struct device *usbdev) +{
- struct snd_soc_usb *ctx;
- if (!usbdev)
return NULL;
How could usbdev ever be NULL?
The method is exported to public, valid check should be reasonable as someone may call it by mistake
We do not protect the kernel from itself like this, no need to check things that should never happen. If the caller gets it wrong, their code will break :)
thanks,
greg k-h