Re: Avoid memory leaks during base backups
Robert Haas <robertmhaas@gmail.com>
From: Robert Haas <robertmhaas@gmail.com>
To: Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>
Cc: Michael Paquier <michael@paquier.xyz>, Cary Huang <cary.huang@highgo.ca>, pgsql-hackers@lists.postgresql.org
Date: 2022-10-20T18:51:21Z
Lists: pgsql-hackers
On Thu, Oct 20, 2022 at 1:35 PM Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> wrote: > I think elsewhere in the code we reset dangling pointers either ways - > before or after deleting/resetting memory context. But placing them > before would give us extra safety in case memory context > deletion/reset fails. Not sure what's the best way. I think it's OK to assume that deallocating memory will always succeed, so it doesn't matter whether you do it just before or just after that. But it's not OK to assume that *allocating* memory will always succeed. -- Robert Haas EDB: http://www.enterprisedb.com
Commits
-
Improve memory handling across SQL-callable backup functions
- 6cc66197ff9a 16.0 landed