<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On 8 November 2013 06:22, Pavel Roschin <span dir="ltr"><<a href="mailto:roshin@scriptumplus.ru" target="_blank">roshin@scriptumplus.ru</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Moving forward to the software source code analysis. Updated more than 300<br>
alloc-functions for GLib.<br>
<br>
Here are new results (better and better):<br>
<br>
<a href="https://gist.github.com/scriptum/7360333" target="_blank">https://gist.github.com/scriptum/7360333</a></blockquote><div><br></div><div>Nice, had a chance to check a couple while sitting in an airport.</div><div>
<br></div><div>symbols.c:1093 arrrrrggghh, keeping the buffer between function invocations!!! Why??  But its not a leak.</div><div><br></div><div>build.c:2416 build.c:2633 technical leak, doesn't happen in any real Geany, needs a menu of >100 items to trigger, replace the return by break to fix</div>
<div><br></div><div>build.c printfs are a debugging facility that is never used AFAICT and are always compiled out, but are safe anyway since the small unsigned ints have the same values as small signed ints.</div><div><br>
</div><div>utils.c:1844 yes, leaks.</div><div>All the g_return_if_fail() calls in that function s/b replaced, the returns are not errors, there is no need to print a message if they occur, the tests are required, but g_return_if_fail() can be compiled out.</div>
<div><br></div><div>document.c:528 false pos, the doc allocated at 522 is added to the array at 525 not leaked, could be silenced by using a different GeanyDocument* temporary variable.</div><div><br></div><div>Your other assessments of false positives look right.</div>
<div><br></div><div>Cheers</div><div>Lex</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
Updated rules:<br>
<a href="https://github.com/scriptum/cppcheck-libs" target="_blank">https://github.com/scriptum/cppcheck-libs</a><br>
<br>
--<br>
Best regards,<br>
Pavel Roschin aka RPG<br>
_______________________________________________<br>
Devel mailing list<br>
<a href="mailto:Devel@lists.geany.org">Devel@lists.geany.org</a><br>
<a href="https://lists.geany.org/cgi-bin/mailman/listinfo/devel" target="_blank">https://lists.geany.org/cgi-bin/mailman/listinfo/devel</a><br>
</blockquote></div><br></div></div>