[PATCH] fhandler_pipe: add sanity limit to handle loops

Ken Brown kbrown@cornell.edu
Sun Dec 26 03:04:00 GMT 2021


On 12/25/2021 6:00 PM, Jeremy Drake via Cygwin-patches wrote:
> On Sat, 25 Dec 2021, Jeremy Drake via Cygwin-patches wrote:
> 
>> On Sun, 26 Dec 2021, Takashi Yano wrote:
>>
>>> Could you please check the result of the following test case
>>> in that ARM64 platform?
>>>
>>
> 
> OK, on Windows 11 ARM64 (same machine as I was testing the assert on):
> per_process: n_handle=52, NumberOfHandles=52
> per_system: n_handle=65536, NumberOfHandles=35331
> 
> On GitHub "windows-2022" runner:
> per_process: n_handle=63, NumberOfHandles=63
> per_system: n_handle=65536, NumberOfHandles=34077
> 
> On Windows 10 x86_64 (can't remember if it was 21H1 or 21H2):
> per_process: n_handle=37, NumberOfHandles=37
> per_system: n_handle=131072, NumberOfHandles=76069

This completely shoots down the speculation in my last email.  Nevertheless, the 
results you posted earlier do indicate that *sometimes* 
NtQueryInformationProcess(ProcessHandleInformation) returns STATUS_SUCCESS even 
if the buffer it's passed is too small.  I hope Takashi has an idea what's going on.

Ken


More information about the Cygwin-patches mailing list