Bug 116546 fix

[Derek Atkins]

Matthew:

1) you should use kvp_frame_delete(), not g_free().
2) You should never have to set data to NULL when you free it.
3) It sounds like the REAL problem is that the postgres backend is
   referencing a deleted book -- why is it doing that?  You should
   try to fix that, rather than trying to fix the symbols of accessing
   freed data.

-derek

Matthew Vanecek <mevanecek at yahoo.com> writes:

> This patch corrects Bug 116546.  There is still a problem in qofid.c
> that was exposed by the fix--the PG backend doesn't fully support the
> entity type lookups yet, or something.  Too late to diagnose now, must
> sleep, but here's the patch for the bug fix.
> 
> The patch is agains head.  I'll submit the 1.8.x branch patch tomorrow
> after I've tested it.
> 
> Thanks, 
> -- 
> Matthew Vanecek
> perl -e 'print $i=pack(c5,(41*2),sqrt(7056),(unpack(c,H)-2),oct(115),10);'
> ********************************************************************************
> For 93 million miles, there is nothing between the sun and my shadow except me.
> I'm always getting in the way of something...
> 
> _______________________________________________
> gnucash-patches mailing list
> gnucash-patches at lists.gnucash.org
> https://lists.gnucash.org/mailman/listinfo/gnucash-patches

-- 
       Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
       Member, MIT Student Information Processing Board  (SIPB)
       URL: http://web.mit.edu/warlord/    PP-ASEL-IA     N1NWH
       warlord at MIT.EDU                        PGP key available