Просмотр исходного кода

video: fbdev: matrox: the list iterator can't be NULL

My static checker is complaining because we check "drv" for NULL and
then we dereference it to get the next item in the list.  It can't be
NULL so we can remove this check.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Dan Carpenter 8 лет назад
Родитель
Сommit
360772f264
1 измененных файлов с 1 добавлено и 1 удалено
  1. 1 1
      drivers/video/fbdev/matrox/matroxfb_base.c

+ 1 - 1
drivers/video/fbdev/matrox/matroxfb_base.c

@@ -2001,7 +2001,7 @@ static void matroxfb_register_device(struct matrox_fb_info* minfo) {
 	for (drv = matroxfb_driver_l(matroxfb_driver_list.next);
 	     drv != matroxfb_driver_l(&matroxfb_driver_list);
 	     drv = matroxfb_driver_l(drv->node.next)) {
-		if (drv && drv->probe) {
+		if (drv->probe) {
 			void *p = drv->probe(minfo);
 			if (p) {
 				minfo->drivers_data[i] = p;