X-Recipient: archive-cygwin AT delorie DOT com DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:date:from:to:subject:message-id:in-reply-to :references:mime-version:content-type:content-transfer-encoding; q=dns; s=default; b=rcoYHpslp9B0vWWSx0Ts65SHsfn3TVq1jpKyoWq2rYc cdpWLefaUeOF/Id0Efluh87LZiA+mBwJj/k5C61HNNLAHbzEey0TVjZKmjYUHiea ijcSo6aljKMOR39hPOYmTQQ+9NUvVD+fPWyH3B26XYgxfDkweuQxGXNP9V2Q1j0U = DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:date:from:to:subject:message-id:in-reply-to :references:mime-version:content-type:content-transfer-encoding; s=default; bh=lX2CboXxO1XCHtlWJFxcHYy6aKI=; b=uW1SuNkd01byl6RGl U37Hn9lP+21hCAATrsDmrY96OmiCW9x4clFOBQP6A2wt4EeowfpU2Iaq9bQj0Snx iP8u1llKNbiA5L3JnsILa1SRKlx8PdmilRHl5Vxbg/BCoz++kcZjdJFJgSa5HZ2Y v99o3dci5GY6LHLo8Jt92o1v/c= Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-owner AT cygwin DOT com Mail-Followup-To: cygwin AT cygwin DOT com Delivered-To: mailing list cygwin AT cygwin DOT com Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-5.4 required=5.0 tests=AWL,BAYES_00,KAM_NUMSUBJECT,RCVD_IN_DNSWL_NONE autolearn=no version=3.3.1 spammy=revised, HDKIM-Filter:v2.10.3, culprit, HX-Spam-Relays-External:Sendmail X-HELO: conssluserg-04.nifty.com DKIM-Filter: OpenDKIM Filter v2.10.3 conssluserg-04.nifty.com x7UIK2VX021466 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nifty.ne.jp; s=dec2015msa; t=1567189202; bh=5JOuEFqMcZWRHDIoZOn02ol+93vzokMflDs6FMw4xuU=; h=Date:From:To:Subject:In-Reply-To:References:From; b=buoSyswXU4TLz4gcEoAU4p0Ryl/djatT5i8Va6wfj1/chgu+EzoCzWjeuH4PcdI+r yB02Qvdi5Rh+eplofeIBamO2w6ZxAorIINcxc7yoi6xhMatcNBzjIfg0+1t6fAAwqu AbVOiP1d+4FCl+vYPyc0WG03Wdw7p9ov5uupm6/G4TO9C8mngGBWqRPzIp6O4lSt8q xyyEk+r/uvOwzMB89blWJPle7fBSXJMHO+I0CVAyYay13rZy+SyH9q7QOkowMWxykl pvebv0TNBk2+XoiEyqE0acB4cUjSWKX8OOEIHmG3XH1fm/e+RFNSrLiem9D0VsnVYO 2HkTiVhRwuPZg== Date: Sat, 31 Aug 2019 03:20:03 +0900 From: Takashi Yano To: cygwin AT cygwin DOT com Subject: Re: [ANNOUNCEMENT] TEST: Cygwin 3.1.0-0.3 Message-Id: <20190831032003.d19fb7060c632bb51777c0de@nifty.ne.jp> In-Reply-To: <20190830075523.GB27273@calimero.vinschen.de> References:

<20190830075523 DOT GB27273 AT calimero DOT vinschen DOT de> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-IsSubscribed: yes Hi Biswapriyo and Corinna, Thank you very much for testing. On Fri, 30 Aug 2019 09:55:23 +0200 Corinna Vinschen wrote: > On Aug 29 22:15, Biswapriyo Nath wrote: > > On Thursday, August 29, 2019, Corinna Vinschen > > 1a. In fhandler_pty_mater::ioctl function, shouldn't the function pointer > > be checked before use? Also what the FIXME says, isn't clear. Any hint? > > Yeah, right. What happens on pre-W10 1809 systems? They probably > crash on TIOCSWINSZ. See below. This code is protected by if (getPseudoConsole () && ... that is, pseudo console handle is set only if CreatePseudoConsole() successes. In pre-W10 1809, since pseudo console handle is not set, this code is not reached. However, it is better to check before call as you suggest. What is meant in FIXME comment: ResizePseudoConsole() needs handle to pseudo console. This handle is valid only in the process which created it. If ioctl(TIOCSWINSZ, ...) is called from other process, it fails. I had tried DuplicateHandle(), but it did not work. Therefore, ResizePseudoConsole() is called only if ioctl() is called from PTY master process. Similarly, ClosePseudoConsole() can work only in the master process. > > 1b. GetModuleHandle and GetProcessHeap is called several times. Wouldn't be > > it easier to get all the pseudo console function pointers in one > > constructor? > > In terms of GetModuleHandle/GetProcAddress the right thing to do is to > use the autoload feature, i.e., add the functions to autoload.cc like > this: > > LoadDLLfuncEx (ClosePseudoConsole, 4, kernel32, 1) > LoadDLLfuncEx (CreatePseudoConsole, 20, kernel32, 1) > LoadDLLfuncEx (ResizePseudoConsole, 8, kernel32, 1) > > If the function call returns FALSE with GetLastError() == > ERROR_PROC_NOT_FOUND, then the function is not available. > > Takashi, I didn't actually notice the usage of the Windows heap here. > The usual method is to use a tls_pbuf buffer, and rather than > using MultiByteToWideChar/WideCharToMultiByte, use sys_mbstowcs/ > sys_wcstombs. > > Also, can you please change the camelback names in class tty_min to > underscored writing, e.g., term_codepage rather than TermCodePage? OK. I will revised the code followed to your advice. First, I would like to fix some bugs and will post a patch. Second, I will revise the coding style. May I post them as indivisual patches? I also suppose the patch should be against the v9 patch, right? > > 3. I can't reproduce this issue with exact steps. But when I zoom in/out + > > resize mintty window + execute cmd.exe in mintty in some random order it > > crashed. Here is the error: > > > > [main] D:\Cygwin64\bin\bash 1129 > > fhandler_pty_slave::push_to_pcon_screenbuffer: pty0: pcon_attach > > mismatch?????? (0x18035DBD0) > > Takashi? This most likely is caused by a bug. Now, I am looking into this problem. I have found the culprit, maybe. -- Takashi Yano -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documentation: http://cygwin.com/docs.html Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple