From: | Ted Yu <yuzhihong(at)gmail(dot)com> |
---|---|
To: | Japin Li <japinli(at)hotmail(dot)com> |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: closing file in adjust_data_dir |
Date: | 2022-11-16 02:52:38 |
Message-ID: | CALte62yKXx8P+=q=_YLNF1tb2Q8KHJwjpUTZzythpyi=skbqxA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Nov 15, 2022 at 6:35 PM Japin Li <japinli(at)hotmail(dot)com> wrote:
>
> On Wed, 16 Nov 2022 at 10:06, Ted Yu <yuzhihong(at)gmail(dot)com> wrote:
> >> Hi,
> > That check is a few line above:
> >
> > + if (fd == NULL || fgets(filename, sizeof(filename), fd) == NULL)
> > {
> >
> > Cheers
>
> Thanks for the explanation. Comment on v2 patch.
>
> fd = popen(cmd, "r");
> - if (fd == NULL || fgets(filename, sizeof(filename), fd) == NULL ||
> pclose(fd) != 0)
> + if (fd == NULL || fgets(filename, sizeof(filename), fd) == NULL)
> {
> + pclose(fd);
> write_stderr(_("%s: could not determine the data directory
> using command \"%s\"\n"), progname, cmd);
> exit(1);
> }
>
> Here, segfault maybe occurs if fd is NULL. I think we can remove pclose()
> safely since the process will exit.
>
> --
> Regrads,
> Japin Li.
> ChengDu WenWu Information Technology Co.,Ltd.
>
That means we're going back to v1 of the patch.
Cheers
From | Date | Subject | |
---|---|---|---|
Next Message | Julien Rouhaud | 2022-11-16 02:53:02 | Re: Allow file inclusion in pg_hba and pg_ident files |
Previous Message | Simon Riggs | 2022-11-16 02:40:47 | Re: Slow standby snapshot |