Skip to contentSkip to conversation

invalid error code in do_register_con_driver() console

report Dan Carpenter 2 participants 2 messages Feb 25, 2016 – May 3, 2016

lore
Share & tools
lore
#0 lore raw

Hi Jiri,

I'm getting the following static checker warning:

	drivers/tty/vt/vt.c:3588 do_register_con_driver()
	warn: missing error code here? 'csw->con_startup()' failed. 'retval' = '0'

drivers/tty/vt/vt.c
3582 if (retval)
3583 goto err;
3584
3585 desc = csw->con_startup();
3586
3587 if (!desc)
3588 goto err;
3589
3590 retval = -EINVAL;
3591

Probably, we could more the "retval = -EINVAL;" before the goto? I don't think it's intentional?

regards,
dan carpenter

#1 lore raw
Quoted 19 lines · Dan Carpenter

Hi Jiri,

I'm getting the following static checker warning:

	drivers/tty/vt/vt.c:3588 do_register_con_driver()
	warn: missing error code here? 'csw->con_startup()' failed. 'retval' = '0'

drivers/tty/vt/vt.c
3582 if (retval)
3583 goto err;
3584
3585 desc = csw->con_startup();
3586
3587 if (!desc)
3588 goto err;
3589
3590 retval = -EINVAL;
3591

Quoted 2 lines

Probably, we could more the "retval = -EINVAL;" before the goto? I don't think it's intentional?

Yes, you are right. I've prepared a fix to send out.

thanks,

Signature · 2 lines
js
suse labs