On 8 November 2013 06:22, Pavel Roschin <ros...@scriptumplus.ru> wrote:
> Moving forward to the software source code analysis. Updated more than 300 > alloc-functions for GLib. > > Here are new results (better and better): > > https://gist.github.com/scriptum/7360333 Nice, had a chance to check a couple while sitting in an airport. symbols.c:1093 arrrrrggghh, keeping the buffer between function invocations!!! Why?? But its not a leak. 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 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. utils.c:1844 yes, leaks. 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. 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. Your other assessments of false positives look right. Cheers Lex > > Updated rules: > https://github.com/scriptum/cppcheck-libs > > -- > Best regards, > Pavel Roschin aka RPG > _______________________________________________ > Devel mailing list > Devel@lists.geany.org > https://lists.geany.org/cgi-bin/mailman/listinfo/devel >
_______________________________________________ Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel