On Mon, Jun 23, 2008 at 11:17 AM, Casey Cichon <[EMAIL PROTECTED]> wrote:
> Charles Day wrote: > >> On Mon, Jun 23, 2008 at 10:23 AM, Casey Cichon < >> [EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>> wrote: >> >> After a few weeks away from using the development gnucash version to >> upgrade my machine from Fedora 5 to Ubuntu gutsy (soon to be >> hardy). I >> get this when running .... >> >> >> This definitely seems like a bug in the code, since msg could get passed >> to g_free() at line 1351 before it is ever assigned. Shall I go ahead and >> commit a fix? >> > The fix is easy enough, but I guess what I mean is: Since the code exists in the "lib" part of the tree, it might not be maintained by GnuCash developers. Should we be fixing this code ourselves, or reporting this bug to the authors? -Charles >> BEFORE: >> char *msg, *book_id; >> >> AFTER: >> char *msg = NULL; >> char *book_id; >> >> -Charles >> >> I would think the char *msg = NULL; would work ... since the message says > that it might never get assigned. > > Casey > _______________________________________________ gnucash-devel mailing list [email protected] https://lists.gnucash.org/mailman/listinfo/gnucash-devel
