[PATCH 2/2] ntdll: Don't set socket IO status after queuing async.

[Paul Gofman]

Thanks for spotting all that.

On 12/16/21 22:43, Zebediah Figura (she/her) wrote:
>
>
>> @@ -1666,7 +1669,8 @@ NTSTATUS sock_ioctl( HANDLE handle, HANDLE 
>> event, PIO_APC_ROUTINE apc, void *apc
>>                   io->Information = sizeof(*ws_linger);
>>               }
>>   -            return ret ? sock_errno_to_status( errno ) : 
>> STATUS_SUCCESS;
>> +            status = ret ? sock_errno_to_status( errno ) : 
>> STATUS_SUCCESS;
>> +            break;
>>           }
>>             case IOCTL_AFD_WINE_SET_SO_LINGER:
>
> And this might close "fd" twice.


I must be missing something but not sure how? 
IOCTL_AFD_WINE_SET_SO_LINGER and other similar cases return 
do_setsockopt() and fd is opened and closed inside do_setsockopt()?