From: Andrei M. <and...@gm...> - 2013-08-06 04:04:43
|
2013/8/6 Koichi Suzuki <koi...@gm...> > Okay, the patch looks nice. > > Andrei, did you find any similar (wrong) memory handling? > > No, if I did, I would come up with a patch or bug report. > Regards; > --- > Koichi Suzuki > > > 2013/8/6 Andrei Martsinchyk <and...@gm...> > >> GTM never pfrees pstrdup'ed value of gti_coordname, and therefore leaks >> memory in the TopMostMemoryContext. >> Please find fix attached. The patch is made against the master branch, >> but applies to STABLE branches, to 1_0 with offsets. >> There were two places where GTM_TransactionInfo was cleaned up, so the >> code was pulled out into a separate function, for easier maintenance. >> >> -- >> Andrei Martsinchyk >> >> StormDB - http://www.stormdb.com >> The Database Cloud >> >> >> >> ------------------------------------------------------------------------------ >> Get your SQL database under version control now! >> Version control is standard for application code, but databases havent >> caught up. So what steps can you take to put your SQL databases under >> version control? Why should you start doing it? Read more to find out. >> >> http://pubads.g.doubleclick.net/gampad/clk?id=49501711&iu=/4140/ostg.clktrk >> _______________________________________________ >> Postgres-xc-developers mailing list >> Pos...@li... >> https://lists.sourceforge.net/lists/listinfo/postgres-xc-developers >> >> > -- Andrei Martsinchyk StormDB - http://www.stormdb.com The Database Cloud |