DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org F34763858C53
To: Takashi Yano <takashi.yano@nifty.ne.jp>,
        "cygwin@cygwin.com"
 <cygwin@cygwin.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.c4798062cb19d4d35a5633ae@nifty.ne.jp>
 <20230824123131.390b4471915c963425c77608@nifty.ne.jp>
 <ZOcb9SKtMPTQfj/m@calimero.vinschen.de>
 <20230825174832.9ebae8112667d5d5411cb8db@nifty.ne.jp>
 <ZOiHkCWY7PK3livD@calimero.vinschen.de>
 <20230826042924.53b49a9f8372a9196a151425@nifty.ne.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@cygwin.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@cygwin.com>
List-Help: <mailto:cygwin-request@cygwin.com?subject=help>
List-Subscribe: <https://cygwin.com/mailman/listinfo/cygwin>,
 <mailto:cygwin-request@cygwin.com?subject=subscribe>
From: "Lavrentiev, Anton \(NIH/NLM/NCBI\) \[C\] via Cygwin" <cygwin@cygwin.com>
Reply-To: "Lavrentiev, Anton \(NIH/NLM/NCBI\) \[C\]" <lavr@ncbi.nlm.nih.gov>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Sender: "Cygwin" <cygwin-bounces+archive-cygwin=delorie.com@cygwin.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