* [PATCH] fbdev: fbmem: implement error handling in fbmem_init()
@ 2016-05-03 2:22 Alexey Khoroshilov
2016-05-10 9:04 ` Tomi Valkeinen
0 siblings, 1 reply; 2+ messages in thread
From: Alexey Khoroshilov @ 2016-05-03 2:22 UTC (permalink / raw)
To: Jean-Christophe Plagniol-Villard, Tomi Valkeinen
Cc: Alexey Khoroshilov, linux-fbdev, linux-kernel, ldv-project
fbmem_init() ignores all errors, while fbmem_exit() does not
check if deallocating resources are valid.
Found by Linux Driver Verification project (linuxtesting.org).
Signed-off-by: Alexey Khoroshilov <khoroshilov@ispras.ru>
---
drivers/video/fbdev/core/fbmem.c | 20 +++++++++++++++++---
1 file changed, 17 insertions(+), 3 deletions(-)
diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/fbdev/core/fbmem.c
index 4e73b6f6b1c0..76c1ad96fb37 100644
--- a/drivers/video/fbdev/core/fbmem.c
+++ b/drivers/video/fbdev/core/fbmem.c
@@ -1854,17 +1854,31 @@ EXPORT_SYMBOL(fb_set_suspend);
static int __init
fbmem_init(void)
{
- proc_create("fb", 0, NULL, &fb_proc_fops);
+ int ret;
+
+ if (!proc_create("fb", 0, NULL, &fb_proc_fops))
+ return -ENOMEM;
- if (register_chrdev(FB_MAJOR,"fb",&fb_fops))
+ ret = register_chrdev(FB_MAJOR, "fb", &fb_fops);
+ if (ret) {
printk("unable to get major %d for fb devs\n", FB_MAJOR);
+ goto err_chrdev;
+ }
fb_class = class_create(THIS_MODULE, "graphics");
if (IS_ERR(fb_class)) {
- printk(KERN_WARNING "Unable to create fb class; errno = %ld\n", PTR_ERR(fb_class));
+ ret = PTR_ERR(fb_class);
+ pr_warn("Unable to create fb class; errno = %d\n", ret);
fb_class = NULL;
+ goto err_class;
}
return 0;
+
+err_class:
+ unregister_chrdev(FB_MAJOR, "fb");
+err_chrdev:
+ remove_proc_entry("fb", NULL);
+ return ret;
}
#ifdef MODULE
--
1.9.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] fbdev: fbmem: implement error handling in fbmem_init()
2016-05-03 2:22 [PATCH] fbdev: fbmem: implement error handling in fbmem_init() Alexey Khoroshilov
@ 2016-05-10 9:04 ` Tomi Valkeinen
0 siblings, 0 replies; 2+ messages in thread
From: Tomi Valkeinen @ 2016-05-10 9:04 UTC (permalink / raw)
To: Alexey Khoroshilov, Jean-Christophe Plagniol-Villard
Cc: linux-fbdev, linux-kernel, ldv-project
[-- Attachment #1.1: Type: text/plain, Size: 1652 bytes --]
Hi,
On 03/05/16 05:22, Alexey Khoroshilov wrote:
> fbmem_init() ignores all errors, while fbmem_exit() does not
> check if deallocating resources are valid.
>
> Found by Linux Driver Verification project (linuxtesting.org).
>
> Signed-off-by: Alexey Khoroshilov <khoroshilov@ispras.ru>
> ---
> drivers/video/fbdev/core/fbmem.c | 20 +++++++++++++++++---
> 1 file changed, 17 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/fbdev/core/fbmem.c
> index 4e73b6f6b1c0..76c1ad96fb37 100644
> --- a/drivers/video/fbdev/core/fbmem.c
> +++ b/drivers/video/fbdev/core/fbmem.c
> @@ -1854,17 +1854,31 @@ EXPORT_SYMBOL(fb_set_suspend);
> static int __init
> fbmem_init(void)
> {
> - proc_create("fb", 0, NULL, &fb_proc_fops);
> + int ret;
> +
> + if (!proc_create("fb", 0, NULL, &fb_proc_fops))
> + return -ENOMEM;
>
> - if (register_chrdev(FB_MAJOR,"fb",&fb_fops))
> + ret = register_chrdev(FB_MAJOR, "fb", &fb_fops);
> + if (ret) {
> printk("unable to get major %d for fb devs\n", FB_MAJOR);
> + goto err_chrdev;
> + }
>
> fb_class = class_create(THIS_MODULE, "graphics");
> if (IS_ERR(fb_class)) {
> - printk(KERN_WARNING "Unable to create fb class; errno = %ld\n", PTR_ERR(fb_class));
> + ret = PTR_ERR(fb_class);
> + pr_warn("Unable to create fb class; errno = %d\n", ret);
> fb_class = NULL;
> + goto err_class;
> }
> return 0;
> +
> +err_class:
> + unregister_chrdev(FB_MAJOR, "fb");
> +err_chrdev:
> + remove_proc_entry("fb", NULL);
> + return ret;
> }
>
> #ifdef MODULE
Looks good to me, queueing for 4.7.
Tomi
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-05-10 9:04 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-03 2:22 [PATCH] fbdev: fbmem: implement error handling in fbmem_init() Alexey Khoroshilov
2016-05-10 9:04 ` Tomi Valkeinen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).