Re: forgotten initalization of a variable

2020-04-21 Thread Kyotaro Horiguchi
At Wed, 22 Apr 2020 08:13:02 +0900, Michael Paquier wrote in > On Tue, Apr 21, 2020 at 06:09:30PM +0900, Kyotaro Horiguchi wrote: > > At Tue, 21 Apr 2020 17:34:26 +0900, Michael Paquier > > wrote in > >> a checkpoint record now, but this routine could be called elsewhere in > >> the future.

Re: forgotten initalization of a variable

2020-04-21 Thread Michael Paquier
On Tue, Apr 21, 2020 at 06:09:30PM +0900, Kyotaro Horiguchi wrote: > At Tue, 21 Apr 2020 17:34:26 +0900, Michael Paquier > wrote in >> a checkpoint record now, but this routine could be called elsewhere in >> the future. Please see the attached. > > It looks fine to me. Fixed this way, then.

Re: forgotten initalization of a variable

2020-04-21 Thread Kyotaro Horiguchi
At Tue, 21 Apr 2020 17:34:26 +0900, Michael Paquier wrote in > On Tue, Apr 21, 2020 at 03:08:30PM +0900, Kyotaro Horiguchi wrote: > > The commit a7e8ece41c adds a new member restoreCommand to > > XLogPageReadPrivate. readOneRecord doesn't make use of it but forgets > > to set NULL. That can lead

Re: forgotten initalization of a variable

2020-04-21 Thread Michael Paquier
On Tue, Apr 21, 2020 at 03:08:30PM +0900, Kyotaro Horiguchi wrote: > The commit a7e8ece41c adds a new member restoreCommand to > XLogPageReadPrivate. readOneRecord doesn't make use of it but forgets > to set NULL. That can lead to illegal pointer access. That's an oversight of the original commit.