<div dir="ltr">Hi Eric,<div style="font-size:12.8px"> </div><div style="font-size:12.8px">I used smatch (<a href="http://smatch.sourceforge.net/" target="_blank">http://smatch.sourceforge.net/</a>). It is mainly used for Linux kernel.</div></div><div class="gmail_extra"> <div class="gmail_quote">2017-12-04 18:52 GMT+02:00 Eric Engestrom <<a href="mailto:eric.engestrom@imgtec.com" target="_blank">eric.engestrom@imgtec.com</a>>: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Monday, 2017-12-04 12:48:55 +0200, Vadim Shovkoplias wrote: > Hi Eric, Hey, sorry, I forgot to hit "send" on the reply I wrote on friday :] > > Mostly by a static analysis tool. It found at least 7 issues with useless > free() calls and other problems that probably should be fixed. What tool? It would be interesting for others to know what tools exist, especially when they find issues other tools didn't :) > Suggest please should I create one cumulative commit for this or it should > be a separate commits ? Same kind of issues in the same module should be grouped, whereas different kind of issues or different modules should be separate. Don't worry too much about it though, if people ask you to merge or split commits, it's not that complicated to do for a v2 :) <div class="HOEnZb"><div class="h5"> > > 2017-12-01 17:41 GMT+02:00 Eric Engestrom <<a href="mailto:eric.engestrom@imgtec.com">eric.engestrom@imgtec.com</a>>: > > > On Friday, 2017-12-01 17:08:53 +0200, <a href="mailto:vadim.shovkoplias@gmail.com">vadim.shovkoplias@gmail.com</a> wrote: > > > From: Vadym Shovkoplias <<a href="mailto:vadym.shovkoplias@globallogic.com">vadym.shovkoplias@globallogic.com</a>> > > > > > > In this condition dri2_dpy->driver_name string always equals > > > NULL, so call to free() is useless > > > > > > Signed-off-by: Vadym Shovkoplias <<a href="mailto:vadym.shovkoplias@globallogic.com">vadym.shovkoplias@globallogic.com</a>> > > > > Reviewed and pushed :) > > > > Are you finding all of these by inspection, or are you using a tool? > > > > > --- > > > src/egl/drivers/dri2/platform_x11.c | 1 - > > > 1 file changed, 1 deletion(-) > > > > > > diff --git a/src/egl/drivers/dri2/platform_x11.c b/src/egl/drivers/dri2/ > > platform_x11.c > > > index c49cb1f..8ede590b 100644 > > > --- a/src/egl/drivers/dri2/platform_x11.c > > > +++ b/src/egl/drivers/dri2/platform_x11.c > > > @@ -704,7 +704,6 @@ dri2_x11_connect(struct dri2_egl_display *dri2_dpy) > > > > > > if (dri2_dpy->driver_name == NULL) { > > > close(dri2_dpy->fd); > > > - free(dri2_dpy->driver_name); > > > free(connect); > > > return EGL_FALSE; > > > } > > > -- > > > 2.7.4 > > > > > </div></div></blockquote></div> </div>