[PATCH 2/3] graphic_utils: do not allocate info in fb_open
Bastian Stender
bst at pengutronix.de
Thu Feb 23 08:45:26 PST 2017
info was errorneously allocated, but it really is a pointer to a fb_info
struct from the framebuffer. This fixes a memory leak.
Signed-off-by: Bastian Stender <bst at pengutronix.de>
---
lib/gui/graphic_utils.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/lib/gui/graphic_utils.c b/lib/gui/graphic_utils.c
index f6ab5ea0d1..7d238e9ff9 100644
--- a/lib/gui/graphic_utils.c
+++ b/lib/gui/graphic_utils.c
@@ -277,8 +277,6 @@ struct screen *fb_open(const char * fbdev)
if (fd < 0)
return ERR_PTR(fd);
- info = xzalloc(sizeof(*info));
-
ret = ioctl(fd, FBIOGET_SCREENINFO, &info);
if (ret) {
goto failed_screeninfo;
@@ -291,7 +289,6 @@ struct screen *fb_open(const char * fbdev)
}
sc->fd = fd;
- sc->info = info;
return sc;
--
2.11.0
More information about the barebox
mailing list