cygin 3.6.2-1 under Win11 24H2: Cannot launch: no available terminals

[Roumen Petrov]

На 5.06.25 г. в 15:49 ч., Johannes Schindelin via Cygwin написа:
[SNIP]
>> and bisecting my problem to
>> https://github.com/cygwin/cygwin/commit/48e7d632689313106d16b0de035fb955c54492b2,
>> the following diff (which partially reverts that commit) fixes it for me:


Dunno what is correction in v3.6.3 but functionality is restored.


>>
>> ```diff
>> diff --git a/winsup/cygwin/uinfo.cc b/winsup/cygwin/uinfo.cc
>> index 83883f9f65..795a00cc1a 100644
>> --- a/winsup/cygwin/uinfo.cc
>> +++ b/winsup/cygwin/uinfo.cc
>> @@ -1996,10 +1996,12 @@ pwdgrp::fetch_account_from_windows (fetch_user_arg_t &arg, cyg_ldap *pldap)
>>         if (sid_id_auth (sid) == 5 /* SECURITY_NT_AUTHORITY */
>>   	  && sid_sub_auth (sid, 0) == SECURITY_APPPOOL_ID_BASE_RID)
>>   	break;
>> +#if 0
>>         /* AzureAD SIDs */
>>         if (sid_id_auth (sid) == 12 /* AzureAD ID */
>>   	  && sid_sub_auth (sid, 0) == 1 /* Azure ID base RID */)
>>   	break;
>> +#endif
>>         /* Samba user/group SIDs */
>>         if (sid_id_auth (sid) == 22)
>>   	break;
>> ```
>>
>> Now, the commit message of that commit includes the name of the reporter,
>> which helped me find
>> https://inbox.sourceware.org/cygwin/pbli6jecn5go2h7heebwivkwz6xf5h2stvq5fxna4u6pql6uht@7xpentpxjiak/
>> as the original reason for that commit. That request only talked about
>> `S-1-15-3-*`, though, and not about `S-1-12-1-*`. And the commit message
>> makes it sound as if my SID would never resolve by `LookupAccountSid()`,
>> but it clearly does.
>>
>> So I am not quite sure what to make of this finding... Should the
>> `S-1-12-1-*` part of 48e7d63268 (Cygwin: fetch_account_from_windows: skip
>> LookupAccountSid for SIDs known to fail, 2025-04-10) simply be dropped? Or
>> was there another clever idea to exploit a shortcut that would side-step
>> the need to call `LookupAccountSid()` for those SIDs and set `ret = TRUE`?
>>
>> Puzzled,
>> Johannes

[SNIP]

Thanks for all your efforts.


Regards,
Roumen Petrov