Message-ID: <805c0496-d60d-802b-3d61-73fed38dea16@enterprisedb.com>
Date: Fri, 4 Mar 2022 09:50:20 +0100
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0)
 Gecko/20100101 Thunderbird/91.6.0
Subject: Re: psql - add SHOW_ALL_RESULTS option
Content-Language: en-US
To: Fabien COELHO <coelho@cri.ensmp.fr>
Cc: "pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>
References: <18c78a01-4a34-9dd4-f78b-6860f1420c8e@enterprisedb.com>
 <alpine.DEB.2.22.394.2107221655020.25741@pseudo>
 <6954328d-96f2-77f7-735f-7ce493a40949@enterprisedb.com>
 <alpine.DEB.2.22.394.2109251058000.3216312@pseudo>
 <alpine.DEB.2.22.394.2110021622530.2817726@pseudo>
 <2570e2ae-fa0f-aac9-f72f-bb59a9983a20@enterprisedb.com>
 <alpine.DEB.2.22.394.2112230703530.2668598@pseudo>
 <46259d13-5f90-1475-3c85-d7a597a2b862@enterprisedb.com>
 <alpine.DEB.2.22.394.2201040851220.3857182@pseudo>
 <alpine.DEB.2.22.394.2201081931170.731324@pseudo>
 <20220113054433.wdjhidtm7gvyhfts@alap3.anarazel.de>
 <alpine.DEB.2.22.394.2201150938470.3378943@pseudo>
From: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
In-Reply-To: <alpine.DEB.2.22.394.2201150938470.3378943@pseudo>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Archived-At: <https://www.postgresql.org/message-id/805c0496-d60d-802b-3d61-73fed38dea16%40enterprisedb.com>
Precedence: bulk

On 15.01.22 10:00, Fabien COELHO wrote:
>> The reason this test constantly fails on cfbot windows is a 
>> use-after-free
>> bug.
> 
> Indeed! Thanks a lot for the catch and the debug!
> 
> The ClearOrSaveResult function is quite annoying because it may or may 
> not clear the result as a side effect.
> 
> Attached v14 moves the status extraction before the possible clear. I've 
> added a couple of results = NULL after such calls in the code.

Are you planning to send a rebased patch for this commit fest?