mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-04 02:02:28 +09:00
media: saa7164: fix return value check in saa7164_initdev()
commit 89f4d45b27 upstream.
In case of error, the function kthread_run() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check
should be replaced with IS_ERR().
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
ac6d0ecfef
commit
ec0593f929
@@ -1348,9 +1348,11 @@ static int saa7164_initdev(struct pci_dev *pci_dev,
|
||||
if (fw_debug) {
|
||||
dev->kthread = kthread_run(saa7164_thread_function, dev,
|
||||
"saa7164 debug");
|
||||
if (!dev->kthread)
|
||||
if (IS_ERR(dev->kthread)) {
|
||||
dev->kthread = NULL;
|
||||
printk(KERN_ERR "%s() Failed to create "
|
||||
"debug kernel thread\n", __func__);
|
||||
}
|
||||
}
|
||||
|
||||
} /* != BOARD_UNKNOWN */
|
||||
|
||||
Reference in New Issue
Block a user