On 12/10/2019 1:57 PM, Corinna Vinschen wrote:
> On Dec 10 13:36, Ken Brown wrote:
>> On 12/10/2019 8:22 AM, Wilfed Olaf Sulla via cygwin wrote:
>>>>> Done. I'll document it once Wilfed verifies that it fixes his problem.
>>>>
>>>> Great! I created new snapshots.
>>>>
>>>> Wilfed, please test if the latest snapshot from
>>>> https://cygwin.com/snapshots/ fixes your problem.
>>>>
>>>
>>> Hi,
>>>
>>> Have install the just the modified DLL.
>>>
>>> Problem appears to be resolved; instead of an attempted core dump I
>>> receive the following message if the remote share is off-line:
>>>
>>> shackleton:sulla:$ ls -la /cygdrive/
>>> ls: cannot access '/cygdrive/z': No such file or directory
>>> total 24
>>> dr-xr-xr-x 1 sulla None 0 Dec 10 13:21 .
>>> drwxr-xr-x+ 1 sulla None 0 Dec 2 20:01 ..
>>> drwxr-xr-x 1 sulla None 0 Nov 16 15:52 c
>>> drwxr-xr-x 1 sulla None 0 Jan 1 1980 e
>>> drwxr-xr-x 1 sulla None 0 Nov 16 15:52 g
>>> drwxr-xr-x 1 sulla None 0 Dec 9 02:25 v
>>> drwxr-xr-x 1 sulla None 0 Dec 9 02:25 w
>>> drwxr-xr-x 1 sulla None 0 Dec 9 02:25 x
>>> drwxr-xr-x 1 sulla None 0 Dec 9 02:25 y
>>> ??????????? ? ? ? ? ? z
>>>
>>>
>>> Many thanks to all concerned.
>>
>> Thanks for testing.
>
> Thanks for testing, Wilfed, and thanks for the patch, Ken!
>
> About time for the 3.1.0 release later this week, I guess...
Sounds good to me. One question I have is whether to leave the assertion that
we've been discussing. When you introduced it in May 2018, the commit message
said
Cygwin: TEST only: Add a buffer underrun assertion to symlink_info::check
[...]
Revert before release.
On the other hand, the assertion has caught two bugs in the last few months,
and
who knows whether there are more. So I'd probably be inclined to leave it for
a
while longer.
Ken
--
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
