DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org D946B3858C2C
Message-ID: <7a999f10-eda4-c9e5-725a-2d3e3524eef8@cornell.edu>
Date: Mon, 20 Jun 2022 13:50:23 -0400
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.9.1
Subject: Re: Bug in Control-d handling?
Content-Language: en-US
To: moss AT cs DOT umass DOT edu, cygwin AT cygwin DOT com
References: <827e2006-2aae-9f7f-9c3f-eef3a7c6e793 AT cornell DOT edu>
 <20220620175935 DOT 924a49feb13156f1a3cf3da4 AT nifty DOT ne DOT jp>
 <20220620192223 DOT 2e0a084a7c093939b1c7c8d6 AT nifty DOT ne DOT jp>
 <5c548a4c-b7ea-b7e0-6ab6-ea0e94ce66a1 AT cornell DOT edu>
 <0ce1465e-b9fd-8794-8707-ee4b2ff4cbb4 AT cs DOT umass DOT edu>
From: Ken Brown <kbrown AT cornell DOT edu>
In-Reply-To: <0ce1465e-b9fd-8794-8707-ee4b2ff4cbb4@cs.umass.edu>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 2
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?c1kxdG9qVFg1MVRmS0J2V0pDcGwycHdiL0tqODU3Ulp2Q1BIemplSk9qTGdZ?=
 =?utf-8?B?ZldwMU1DbVNlWDJva0tRdUMyVloxRnZ4bDJ2TmM4Zm1EbVB5YTltejdnMmpL?=
 =?utf-8?B?SGRSNjhRdlIxSnhva3UrcGlKamFCUTY5MGo1NWdxVzdFRHNTeUZqV2hZeWY5?=
 =?utf-8?B?azhHTGI0b2FMTVg3QStrMUErSG5GbUxZaUVKcHg4ZzhQQThwK2x3eXRocE9n?=
 =?utf-8?B?cWlydE9RMzVTakxZQnArR01ZU2ZYVzk1b1lmZHozazlrV1JVeThHcHBKd0Fp?=
 =?utf-8?B?cUFDWktiWXJERjd0QkhOMzFhS1g0Q3BYTysvb25TZTM4Ym8yN0toeGxjbS9i?=
 =?utf-8?B?MmhoM1JyMlg4UEYvZStXMmg4WEZYTXkrUmJtemRZM1pyeUtiYTQ2SkhDaFRS?=
 =?utf-8?B?dHVyWitGeWRuZFFMcko0ZFZoNE1EbVJBcEw0cG5yNmcyN0E2VUxkU2QxMjFv?=
 =?utf-8?B?TjdzbGJtWHhrcjBoOE54WUZ5R2Vwb0gwbDlBeEZYMm1yS3RZbS9JaTRhNmpr?=
 =?utf-8?B?MDgybWl4NUszSzNoQjJkTkdKWjJhMXNraFVVMzdQWUhXdGs1TGZ0cFhmSWxB?=
 =?utf-8?B?cEdNVTZ4TlhSbWg1OFhWUGwyTnphUnk0b0p5bGJsU0p6dCtzNkR6bHJ1OUZ6?=
 =?utf-8?B?Q3JDU3NYOHFHZW0wb0R0NmwrcFkrS3pKWk9YNTdOcDNJQUNTOFR6YWdiZVlT?=
 =?utf-8?B?TWpZeE9oZzY5MlhIbzhid2ZZbGFWMFNLVkZ1eEhFVkE1VjN0alo0Y0Npb3Fo?=
 =?utf-8?B?cmh3STJyVU5HOGtNYjRRby9VUDBCcklCa0NmNnk5ZFg2UW5RNFZaa2hKOXlz?=
 =?utf-8?B?dGF3REtSOENHdSt0ckNzK3dWdy9TTDdRdE9LSEt6czdMSFVMNnA1cmoydHRZ?=
 =?utf-8?B?MGlyeExma2lHY29pRTNYR2lNQzhpSmZUWVZnMzFHQmZ2MHNlY3RmbjQwTTJ1?=
 =?utf-8?B?cUx2RklRYUVSWStNVC9kQSs4WXZRT0FVMU9WT2MwZFlMbzBOWjE5Rkk3MGFj?=
 =?utf-8?B?S1VsMlhMaGo0MTRvRnBMWTZOc0YrcDdZTkozcUR2aWtmTGdDTmNmbVhPM1dE?=
 =?utf-8?B?RUVSUHBhbllXYlFmUFl0Z1p2dTN0L3NuUUgvK2RxNnZzWVJEeUZpT1l1bDlo?=
 =?utf-8?B?bGFVSHZ5R1I5VG5UVWVsV3RnZmZrTEFZVXVIRXpoTTk0cVE2eXNJSCtqbHIz?=
 =?utf-8?B?dWNQNHJLSEZ0cStpbTlNT1JUSEVOQmUzVEkvMlJyNExibkxUODRzNlI0WUp1?=
 =?utf-8?B?dFBYQlZwY24wMzZTKzAwOWxqS2c4YndTTnNIcFpySENrbWlWU3ExL1N2ODRG?=
 =?utf-8?B?K1J5MkVzMzY1eGJxcHEzbDg2Q3JZUjJYMDc2b0NUaXdybVlKMDRtcFJ0U1pQ?=
 =?utf-8?B?ME1IWVRaWmpOdFhWa2dBbHZNckxRUUFXL1QyTXNlMVB4VFc2Wk40Q0JLQlBE?=
 =?utf-8?B?NVpKUDBIQ0dxSmhabStKTVkwWXBnQVoxeWE4M2dWM1B5YlVPSG5kQ1ZnZTk1?=
 =?utf-8?B?YjI4cVR4V1BJZDFzM0FtVUZ3THFhSEVERkZlamZyTGhHYmxYOWp0SlVWby9V?=
 =?utf-8?B?TituK25sWXRYc1Z4ak9iaktwa2xtTm1Qemx4Qm1JdWpJM05LK09WNWF5V2ds?=
 =?utf-8?B?c09xZ0NudFVMdVo3KzZQZGM5eEhtREM4TXNZSHFqUytWK2hhblJsM2xqa2dh?=
 =?utf-8?B?T3FQaitIUXFuZ3hDeXkyL0kxbzBXQWQvSHQxMEdIT2R3eWpEeFQwd0N0OVBO?=
 =?utf-8?B?UDFJSWt5MCtaRlhYVUtxQVhVS2gvc1dQUm5LNEtXMWJGTE1uNWc4bDcxUUEv?=
 =?utf-8?B?QlFpcWJ5bnU1ZlcvcFgxdHRZZW52YWFvSFZadkdmZFB0bTRDR0dZZ1dCenNE?=
 =?utf-8?B?eW5Xa3RxQ1BMajM3S0Rudzc1NlhydmFXNkxBWWg1d2pJMnBxYndHTjhLcUNx?=
 =?utf-8?B?aWZnTWVNSHNkcWpXcFBlSXBrbUZQTFNZK0h5VjRSWG9uUFlaUnZiVVNLd0ZD?=
 =?utf-8?B?L2NCZjJta24vOHhEQ1pPeFlPYkx2YXcxVkdFcHhRbWlKMFg5dEZ2VzhjUTBL?=
 =?utf-8?B?ZzlJMENHYzJZRVB2NHI0a1lMTjV6TWlCa1E0bXBJRE12aTJoVnFXK2xjWCtV?=
 =?utf-8?B?aVpOSUY3ajZNVUtaL1BKcjR6VUxXMjNTdUxqQmlvdUszYlQ0cERzNmowNUdH?=
 =?utf-8?B?aStvTjEwWXRRUGdySmI0alpKdUtycnFNNWpISmJVaXNhN2ZFY2hTeGhRTE1V?=
 =?utf-8?B?MmxFUmFIYm5VN2JKU0Z6U2VtZDNOMkdqRGVVa3orUi9COUpKemVXRjQ4TWNJ?=
 =?utf-8?B?b0FNWGYyczA5ZHJkeWxpaGlGV2UzWXRyajlRMGY0cTdLMCtWSHdyMHh5M0Nj?=
 =?utf-8?Q?eAMGBPE+bK2MarhJbPna1qbmUKxD4pNSy0tf4K5m9YIj7?=
X-MS-Exchange-AntiSpam-MessageData-1: SSszdGWrMmdwag==
X-OriginatorOrg: cornell.edu
X-MS-Exchange-CrossTenant-Network-Message-Id: 392b8176-9771-4091-739c-08da52e55abc
X-MS-Exchange-CrossTenant-AuthSource: BN7PR04MB4388.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Jun 2022 17:50:25.0728 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 5d7e4366-1b9b-45cf-8e79-b14b27df46e1
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: OZL3tfi/NuxxBG6O5GntkDDF2lc8lYgjrtG4cKjkudFOGIyZy9lTFvEKMlG6fn8qdYwh6KxM0fwsmOtowJiesg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR04MB0501
X-Spam-Status: No, score=-3.2 required=5.0 tests=BAYES_00, DKIM_SIGNED,
 DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, JMQ_SPF_NEUTRAL, NICE_REPLY_A,
 SPF_HELO_PASS, SPF_PASS, TXREP,
 T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
 server2.sourceware.org
X-BeenThere: cygwin AT cygwin DOT com
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: General Cygwin discussions and problem reports <cygwin.cygwin.com>
List-Unsubscribe: <https://cygwin.com/mailman/options/cygwin>,
 <mailto:cygwin-request AT cygwin DOT com?subject=unsubscribe>
List-Archive: <https://cygwin.com/pipermail/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-request AT cygwin DOT com?subject=help>
List-Subscribe: <https://cygwin.com/mailman/listinfo/cygwin>,
 <mailto:cygwin-request AT cygwin DOT com?subject=subscribe>
Content-Type: text/plain; charset="utf-8"; Format="flowed"
Errors-To: cygwin-bounces+archive-cygwin=delorie DOT com AT cygwin DOT com
Sender: "Cygwin" <cygwin-bounces+archive-cygwin=delorie DOT com AT cygwin DOT com>
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from base64 to 8bit by delorie.com id 25KHp8NY014122

On 6/20/2022 9:53 AM, Eliot Moss wrote:
> Good idea to wait for Corinna.  I did a smidge of research, looking through
> the Wikipedia page on the POSIX terminal interface.  It suggests that this
> part was largely borrowed from System-V, so that may explain what we see in
> newlib.  And looking up the POSIX spec, it says this:
> 
> "Special character on input, which is recognized if the ICANON flag is
> set. When received, all the bytes waiting to be read are immediately passed to
> the process without waiting for a <newline>, and the EOF is discarded. Thus,
> if there are no bytes waiting (that is, the EOF occurred at the beginning of a
> line), a byte count of zero shall be returned from the read(), representing an
> end-of-file indication. If ICANON is set, the EOF character shall be discarded
> when processed."
> 
> This suggests that indeed control-D will need to be typed twice before a
> blocking read on a terminal with ICANON will appear to be at eof.  I guess we
> can consider getchar to be (almost) like a read of size 1, except where read
> would return 0, getchar returns -1.
> 
> The quoted text above implies that the second control-D will set the
> end-of-file indicator.  So I agree that it looks like the ifndef should be
> removed.  And of course, wherever we said control-D, EOF should be implied (it
> is normally control-D, but the user can change which character is associated
> with the EOF semantics).

I would frame the issue slightly differently, since this is not just about 
terminals; the code we're discussing applies to any file.  Suppose EOF has been 
seen in a call to one of the stdio input functions.  Prior to commit 1f8f7e2d, 
any further calls to those functions would immediately return EOF.  Since that 
commit, however, another attempt is made to read, and the EOF indicator for the 
file is turned off if input is found.  In many cases (e.g., an ordinary disk 
file), this second read will still return EOF.  In other cases (e.g., a 
terminal), the second read might block if there's still no input available.

Ken

-- 
Problem reports:      https://cygwin.com/problems.html
FAQ:                  https://cygwin.com/faq/
Documentation:        https://cygwin.com/docs.html
Unsubscribe info:     https://cygwin.com/ml/#unsubscribe-simple