On Fri, Apr 15, 2016 at 11:30 AM, Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp> wrote: > > At Fri, 15 Apr 2016 08:52:56 +0530, Amit Kapila <amit.kapila16@gmail.com> wrote : > > > > How about if we do all the parsing stuff in temporary context and then copy > > the results using TopMemoryContext? I don't think it will be a leak in > > TopMemoryContext, because next time we try to check/assign s_s_names, it > > will free the previous result. > > I agree with you. A temporary context for the parser seems > reasonable. TopMemoryContext is created very early in main() so > palloc on it is effectively the same with malloc. > > One problem is that only the top memory block is assumed to be > free()'d, not pfree()'d by guc_set_extra. It makes this quite > ugly.. >
Is there a reason to use malloc here, can't we use palloc directly? Also for both the functions SyncRepCopyConfig() and SyncRepFreeConfig(), if we directly use TopMemoryContext inside the function (if required) rather than taking it as argument, then it will simplify the code a lot.
