lyp-bobi commented on issue #2046: URL: https://github.com/apache/age/issues/2046#issuecomment-2292642489
> @lyp-bobi The problem is that there are places where the value isn't copied, for whatever reason. Using pfree ends up freeing stuff it's not supposed to. This is making it difficult to see what actually needs to be done to correct it. Maybe just create a new function other than `pfree_agtype_value_content` that is only used in `compare_agtype_containers_orderability` so it won't pfree unsupposed memories? > @lyp-bobi Unfortunately, your fix doesn't resolve the memory issue. I've made modifications to fix the code and the regression tests that fail due to the modifications added. However, it still gobbles up memory. There is likely more to this than just that section of code. I have no idea about that. In my case most memory seems to be used by `pnstrdup`, but I'm not sure if there are other memory leaks or other problems :P I'm glad to see you can view this problem at a higher level perspective. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@age.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
