Pan Nengyuan <pannengy...@huawei.com> 于2020年8月14日周五 下午6:15写道: > > Receiving error in local variable err, and forgot to free it. > Considering that there is no place to deal with it. Clean up. > > Reported-by: Euler Robot <euler.ro...@huawei.com> > Signed-off-by: Pan Nengyuan <pannengy...@huawei.com> > --- > Cc: Gerd Hoffmann <kra...@redhat.com> > --- > ui/gtk-gl-area.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/ui/gtk-gl-area.c b/ui/gtk-gl-area.c > index 85f9d14c51..c740a7eb14 100644 > --- a/ui/gtk-gl-area.c > +++ b/ui/gtk-gl-area.c > @@ -142,15 +142,14 @@ QEMUGLContext > gd_gl_area_create_context(DisplayChangeListener *dcl, > VirtualConsole *vc = container_of(dcl, VirtualConsole, gfx.dcl); > GdkWindow *window; > GdkGLContext *ctx; > - GError *err = NULL; > > gtk_gl_area_make_current(GTK_GL_AREA(vc->gfx.drawing_area)); > window = gtk_widget_get_window(vc->gfx.drawing_area); > - ctx = gdk_window_create_gl_context(window, &err); > + ctx = gdk_window_create_gl_context(window, NULL); > gdk_gl_context_set_required_version(ctx, > params->major_ver, > params->minor_ver); > - gdk_gl_context_realize(ctx, &err); > + gdk_gl_context_realize(ctx, NULL); > return ctx; > }
Maybe we should check the return value of 'gdk_window_create_gl_context' and 'gdk_gl_context_realize' instead of omitting it? Thanks, Li Qiang > > -- > 2.18.2 > >