From: kernel test robot <lkp(a)intel.com&gt; drivers/uio/uio_pci_generic.c:247:2-7: WARNING: NULL check before some freeing functions is not needed. drivers/uio/uio_pci_generic.c:264:2-7: WARNING: NULL check before some freeing functions is not needed. NULL check before some freeing functions is not needed. Based on checkpatch warning "kfree(NULL) is safe this check is probably not required" and kfreeaddr.cocci by Julia Lawall. Generated by: scripts/coccinelle/free/ifnullfree.cocci Fixes: 709a04896cb6 ("kernel[v2]: extend uio driver to supports msix") CC: Yuan Liu <yuan1.liu(a)intel.com&gt; Reported-by: kernel test robot <lkp(a)intel.com&gt; Signed-off-by: kernel test robot <lkp(a)intel.com&gt; --- tree: https://github.com/intel/linux-intel-lts.git 5.4/yocto head: 709a04896cb6ab56a8ce54575f2cc4d346d71dea commit: 709a04896cb6ab56a8ce54575f2cc4d346d71dea [11/11] kernel[v2]: extend uio driver to supports msix uio_pci_generic.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) --- a/drivers/uio/uio_pci_generic.c +++ b/drivers/uio/uio_pci_generic.c @@ -243,8 +243,7 @@ static int probe(struct pci_dev *pdev, err_register: #ifdef CONFIG_PCI_MSI - if (gdev->msix_info.entries != NULL) - kfree(gdev->msix_info.entries); + kfree(gdev->msix_info.entries); #endif kfree(gdev); err_alloc: @@ -260,8 +259,7 @@ static void remove(struct pci_dev *pdev) uio_unregister_device(&gdev->info); pci_disable_device(pdev); #ifdef CONFIG_PCI_MSI - if (gdev->msix_info.entries != NULL) - kfree(gdev->msix_info.entries); + kfree(gdev->msix_info.entries); #endif kfree(gdev); }