DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org F34763858C53
To: Takashi Yano <takashi DOT yano AT nifty DOT ne DOT jp>,
        "cygwin AT cygwin DOT com"
 <cygwin AT cygwin DOT com>
Subject: RE: [EXTERNAL] Re: scp stalls on uploading in cygwin 3.5 current
 master.
Thread-Topic: [EXTERNAL] Re: scp stalls on uploading in cygwin 3.5 current
 master.
Thread-Index: AQHZ1juS/PjVpg8vtUumO/YZ+jANLa/5JiKAgAGPQQCAACIyAIAAkNwAgAE1/+A=
Date: Sat, 26 Aug 2023 14:08:34 +0000
Deferred-Delivery: Sat, 26 Aug 2023 14:08:12 +0000
Message-ID: <DM8PR09MB70951B220E8E94793560EB8EA5E2A@DM8PR09MB7095.namprd09.prod.outlook.com>
References: <20230824060502 DOT c4798062cb19d4d35a5633ae AT nifty DOT ne DOT jp>
 <20230824123131 DOT 390b4471915c963425c77608 AT nifty DOT ne DOT jp>
 <ZOcb9SKtMPTQfj/m AT calimero DOT vinschen DOT de>
 <20230825174832 DOT 9ebae8112667d5d5411cb8db AT nifty DOT ne DOT jp>
 <ZOiHkCWY7PK3livD AT calimero DOT vinschen DOT de>
 <20230826042924 DOT 53b49a9f8372a9196a151425 AT nifty DOT ne DOT jp>
In-Reply-To: <20230826042924.53b49a9f8372a9196a151425@nifty.ne.jp>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?bpHsFQEX3BRqeJ4rCG2A8eAWRMA8/UUXSGMi2zRVSCXFDSpG2yLRNP5RbBK3?=
 =?us-ascii?Q?fJy8Jg5mjyL6G3h2CgIy6Sa8kKGnjVr2AjMFiIYWKQZ4vmOLZrAVzVMg+l6A?=
 =?us-ascii?Q?8LpM6C621xzPstE5ma8W9cNqdsdkDmxOgw2qMxLB3u3m+aWi1w6Tgfkbe7OO?=
 =?us-ascii?Q?UBmMozJ8N6SP2qo0fTKQCv6SaFFmRp3hyQiZbB1vcNQuQE0vI+DthaGPnZ1H?=
 =?us-ascii?Q?YAGgyBc3TEDKXGQ90bL//ZmmUQOrjVC6Hl6Kfoe66mgbwBFHkqvyws0WI7wX?=
 =?us-ascii?Q?+ad82ZXfhxmA90DJALH8JGFCVWaHqN3+lVhzQvA40SVyHq3q13ab8sLvtStn?=
 =?us-ascii?Q?/M0vlEuPMgMCHALkEnAZXv9nKcboZ2A6yEhFb5haHvViu2EELZHI6WDwcjaa?=
 =?us-ascii?Q?3TqzV6wwLjFO5d4suf8OdJIuWedhuNV2WxSjn4YeMrY1sTLHeBDo+8qtq09m?=
 =?us-ascii?Q?wYzTqGqB5pEIt8xXeQyBVGiKmzkcb47uiRWZByInXL4LLVwkH3QMT4k61Aiu?=
 =?us-ascii?Q?oj3/zvvxV8f/5l1m7N5RxFVNVD3i6KRF80ol8hnsSuNThOrCmz6glOVazCjF?=
 =?us-ascii?Q?L2pN5LsBqyCLuHykXpoZ3hCqWEOsVYZeK6Q6ZlPCT+YA0kLcsJur3zDheVNZ?=
 =?us-ascii?Q?leClRc01/Jdn4MA1yVDpmFZVj7OQhv9++ZI6eU1dTa2+Yp6nW/mvoYNw4720?=
 =?us-ascii?Q?Sr6FrgYHE1CufQkzT3wjJsupEVqt89Cz26BeWJaS/riAV75LRZgfg7Y5wCMW?=
 =?us-ascii?Q?ve5D8vQI02MuLxOwePbVRWzegqDHgrw6klq6uHhKSSzPTdQwKWFJRgLecVTc?=
 =?us-ascii?Q?mVf4OhX/Y5iDWoYx/g+8Ah/XD/kBAO5l9Cg2J3tTbQJKW7q2e/vvifzIgRug?=
 =?us-ascii?Q?sLv5In2oaldnm/u5OB6wCNkfYsJzRtO8fAQ1IWQvPc0qDgLu0uGslhp6j0+F?=
 =?us-ascii?Q?vf2QtIlTimGnF6aI7tYVUazFad68H+CNmrJAcHa2jeYmwfhb3GM5/1Qh27gM?=
 =?us-ascii?Q?/Qd7p3o4cwz0dQyihBEnTSAFrNjgPWpc5dGLxvExuYnmx+STkRh/jf8i8UiU?=
 =?us-ascii?Q?tw3T7ZPCP6XzaK9cgyxG1UKnqSZ7J1eSYAEdTdkENq1DLN+g5ZztKrSCrzFU?=
 =?us-ascii?Q?OqEYZQJpovPrXigtiB1L2z3SUXrUgv4k3cTY8Ny6EEAvFfNalosdknuRUpkd?=
 =?us-ascii?Q?k1tiaXMwsMo1BJrmGYpJtW7R0gvmDp8zZ1ZXT09bZ36LwwZT09hdRWeUYRr8?=
 =?us-ascii?Q?BByPwCOuU97FrEMn9lt6TJYV7I+7vVVcZetEYdbJ7ziRlrm3PgvJ0vH5JzGh?=
 =?us-ascii?Q?M2Ttc/ptd9YW2duPunk4T8g4TPvwSBESR9vGMc3ST6Irzo+e4fVwKZPFBR/I?=
 =?us-ascii?Q?onWdpDPjxi04znqxD/M8Go6nsMIVf33k3XAUlKqaCwOW1BcTVdk8w9j9PbSo?=
 =?us-ascii?Q?rCSkugAnbPT46/vBrJ9PnNYJMILOe/iiVJT3HqvyrhOqTMcJqCtldULQw3MW?=
 =?us-ascii?Q?kefNUBK71i4d3zBoaRaXn6uJtYTZd1lGeYB8p5NC67Pfy0m/V+CHfddIUHFt?=
 =?us-ascii?Q?7R1MCkhKOYMNGRFgZX8=3D?=
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: tj2nUifn+030BJFMMGwzPTAKfPwnUihqSrJyLa0g2xBLIeGirhmepWRPbwBu9SB0jALheRH78afU8nRefSebUwnUSdPycKhoSlF7NqCTpfwyqxueGeM2rCmkmSQWuXzir6A92Cq8AwNDOAEp7/GZX+lrxGZ/jQDBLpvvJDVzTwXCAok9ndIWqX6l0fMwwPul8ryP5gi0rlNYry3cTBPaHIXS+6k4mzeI8NGSbM0UC5MV6B2I9T/PELBhWQfKxpe+GI8adpkejfLoLBuoGlX2FxLzSmmkG9iot6ZHvRpQwhYzm0WWkCaccdYY75Up8jnG4QW/hsnL4vCC3X0/Svs3huqOMo4G4W+wn9z08XYTEQ5d4i87n/u3WiVpZgOVtHgPLA74BCqsPKGRS7p/g/9t6tICSQsK2TXHIqO1A3ldTHYigcAzas+hIicToyaQPATU/YZ262J8hXIlrqRXqxYnRHbOpzC+bs0T5T/RBT4Xyg/2XdFDeYsqphjJNEpbTorFj3+4mU0Ts0NayXcpjb+/qq1HS6F6QrQfuwt2IYDKfupyFMF87OaGfJQn+VQ4KqdXnuPJKNZqYm+doZ6n/LE+SF6t8ecVA1hsQdBQnFBe3mYw6aYlb99DIU5TztvuKAw+WnXALNVAA/FgOVIBg5T2vhMppRtAKOkmhn5JMwXNHn3nW7JwMG/byNZiBBjtfLweUe9SsXyCtFtLZzHZWoi3Mj5eJ5EfE+JhNHrud2db+lAQiNtoetowcUU4Rzdvz4Cgt71K6o6abmRIhuGpPCUbTlrUw1ZKBvobMPtakLbjjKUs8zM2OT+SzTRcpyQmhDovjOPlJqIKYjzxpAYtFDiX+L1RVXvZy8qmp7vGYCRRdGc=
X-OriginatorOrg: ncbi.nlm.nih.gov
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM8PR09MB7095.namprd09.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 79ee5834-cdfe-4b15-b9b1-08dba63dfdde
X-MS-Exchange-CrossTenant-originalarrivaltime: 26 Aug 2023 14:08:58.5409 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 14b77578-9773-42d5-8507-251ca2dc2b06
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR09MB10930
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00, DKIM_SIGNED,
 DKIM_VALID, DKIM_VALID_EF, SPF_PASS, TXREP,
 T_SPF_HELO_PERMERROR autolearn=ham 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
List-Id: General Cygwin discussions and problem reports <cygwin.cygwin.com>
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>
From: "Lavrentiev, Anton \(NIH/NLM/NCBI\) \[C\] via Cygwin" <cygwin AT cygwin DOT com>
Reply-To: "Lavrentiev, Anton \(NIH/NLM/NCBI\) \[C\]" <lavr AT ncbi DOT nlm DOT nih DOT gov>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Sender: "Cygwin" <cygwin-bounces+archive-cygwin=delorie DOT com AT cygwin DOT com>

> > Can we use send(sock, "", 0) to reenable FD_WRITE, perhaps?
> 
> Your idea seems to work. The following patch looks to solve the issue.
> Is it supposed to be any side effect()?

IMO you're triggering an undefined (or not well-defined) behavior, because of the murky status
of the byte count of 0.  It may work now, but it may not work in the future...

RETURN VALUE
       On  success,  the  number of bytes written is returned (zero indicates nothing was written).  On error, -1 is returned, and
       errno is set appropriately.

       If count is zero and fd refers to a regular file, then write() may return a failure status if one of the  errors  below  is
       detected.   If no errors are detected, 0 will be returned without causing any other effect.  If count is zero and fd refers
       to a file other than a regular file, the results are not specified.

Anton Lavrentiev
Contractor NIH/NLM/NCBI

P.S.

Think of the effects of "recv(sock, buf, 0)" along the lines: the only valid return values in this
case would be -1 and 0, whereas 0 [usually] means EOF, but would it be really meant in this case?


-- 
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