On Thu, Nov 18, 2021 at 12:33 PM Peter Smith <smithpb2...@gmail.com> wrote: > > PSA new set of v40* patches. >
Another thing I noticed was in the 0004 patch, list_free_deep() should be used instead of list_free() in the following code block, otherwise the rfnodes themselves (allocated by stringToNode()) are not freed: src/backend/replication/pgoutput/pgoutput.c + if (rfnodes) + { + list_free(rfnodes); + rfnodes = NIL; + } Regards, Greg Nancarrow Fujitsu Australia