From b14cb6ccb4d3b4816c4a6a4c5ecf575822ac9e8d Mon Sep 17 00:00:00 2001 From: jianglianfang Date: Fri, 21 Jul 2023 16:54:02 +0800 Subject: [PATCH] driver/fb: fix that vtable.priv is not freed when fb register failed. The vtable.priv is better to be assigned after fb is registered sucessfully. Signed-off-by: jianglianfang --- drivers/video/fb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/video/fb.c b/drivers/video/fb.c index 0f5fd0d857..85107e305b 100644 --- a/drivers/video/fb.c +++ b/drivers/video/fb.c @@ -1084,8 +1084,6 @@ int fb_register(int display, int plane) goto errout_with_fb; } - fb->vtable->priv = fb; - /* Initialize the frame buffer instance. */ DEBUGASSERT(fb->vtable->getvideoinfo != NULL); @@ -1151,6 +1149,8 @@ int fb_register(int display, int plane) goto errout_with_fb; } + fb->vtable->priv = fb; + return OK; errout_with_fb: