<div dir="ltr">On Tue, Apr 30, 2013 at 11:02 AM, David Binderman <span dir="ltr"><<a href="mailto:dcb314@hotmail.com" target="_blank">dcb314@hotmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello there,<br></blockquote><div><br></div><div style>Hi David,</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


I just ran the static analyser "cppcheck" over the source code of libfprint-0.5.0<br></blockquote><div><br></div><div style>Thanks!</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


and it said<br>
<br>
1.<br>
<br>
[drivers/vfs101.c:192]: (warning) Logical conjunction always evaluates to false: result < 0 && result >= 3.<br>
<br>
Suggest replace && with ||<br></blockquote><div><br></div><div style>OK, but I've no vfs101 device to test.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br>
2.<br>
<br>
[img.c:154]: (error) Resource leak: fd<br></blockquote><div><br></div><div style>OK</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
[drivers/aesx660.c:296]: (error) Memory leak: stripdata<br></blockquote><div><br></div><div style>OK</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


[nbis/bozorth3/bz_alloc.c:116]: (error) Memory leak: p<br>
[nbis/bozorth3/bz_io.c:421]: (error) Resource leak: fp<br>
[nbis/mindtct/init.c:593]: (error) Memory leak: powers<br>
[nbis/mindtct/init.c:682]: (error) Memory leak: powmax_dirs<br>
[nbis/mindtct/maps.c:662]: (error) Memory leak: cimage<br>
[nbis/mindtct/maps.c:722]: (error) Memory leak: pmap<br>
[nbis/mindtct/maps.c:729]: (error) Memory leak: pmap<br>
<br></blockquote><div style><br></div><div style>I don't want to touch nbis code until it fails</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I checked the first couple and they did look like real problems, although minor.<br>
<br>
Regards<br>
<span class="HOEnZb"><font color="#888888"><br>
David Binderman<br>
_______________________________________________<br>
fprint mailing list<br>
<a href="mailto:fprint@lists.freedesktop.org">fprint@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/fprint" target="_blank">http://lists.freedesktop.org/mailman/listinfo/fprint</a><br>
</font></span></blockquote></div><br></div></div>