Thread: pgsql: Small tidyup for commit d41a178b.
Small tidyup for commit d41a178b. A comment was left behind claiming that we needed to use malloc() rather than palloc() because the corresponding free would run in another thread, but that's not true anymore. Remove that comment. And, with the reason being gone, we might as well actually use palloc(). Back-patch to supported releases, like d41a178b. Discussion: https://postgr.es/m/CA%2BhUKG%2BpdM9v3Jv4tc2BFx2jh_daY3uzUyAGBhtDkotEQDNPYw%40mail.gmail.com Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/6a9229da65c9553a1268764d46fb28043b83bfd6 Modified Files -------------- src/backend/postmaster/postmaster.c | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-)
Thomas Munro <tmunro@postgresql.org> writes: > Small tidyup for commit d41a178b. > A comment was left behind claiming that we needed to use malloc() rather > than palloc() because the corresponding free would run in another > thread, but that's not true anymore. Remove that comment. And, with > the reason being gone, we might as well actually use palloc(). Hm, doesn't the change to palloc make the following error check redundant? childinfo = palloc(sizeof(win32_deadchild_waitinfo)); - if (!childinfo) - ereport(FATAL, - (errcode(ERRCODE_OUT_OF_MEMORY), - errmsg("out of memory"))); - childinfo->procHandle = pi.hProcess; childinfo->procId = pi.dwProcessId; regards, tom lane
On Fri, Mar 17, 2023 at 1:36 PM Tom Lane <tgl@sss.pgh.pa.us> wrote: > Thomas Munro <tmunro@postgresql.org> writes: > > Small tidyup for commit d41a178b. > > A comment was left behind claiming that we needed to use malloc() rather > > than palloc() because the corresponding free would run in another > > thread, but that's not true anymore. Remove that comment. And, with > > the reason being gone, we might as well actually use palloc(). > > Hm, doesn't the change to palloc make the following error check redundant? > > childinfo = palloc(sizeof(win32_deadchild_waitinfo)); > - if (!childinfo) > - ereport(FATAL, > - (errcode(ERRCODE_OUT_OF_MEMORY), > - errmsg("out of memory"))); > - > childinfo->procHandle = pi.hProcess; > childinfo->procId = pi.dwProcessId; True. Will fix.