GNU bug report logs - #55832
28.1; Emacs crashes when using tramp from helm in emacs-29

Previous Next

Package: emacs;

Reported by: Thierry Volpiatto <thievol <at> posteo.net>

Date: Tue, 7 Jun 2022 15:24:02 UTC

Severity: normal

Found in version 28.1

Fixed in version 29.1

Done: Michael Albinus <michael.albinus <at> gmx.de>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Thierry Volpiatto <thievol <at> posteo.net>
To: Michael Albinus <michael.albinus <at> gmx.de>
Cc: Eli Zaretskii <eliz <at> gnu.org>, larsi <at> gnus.org, 55832 <at> debbugs.gnu.org
Subject: bug#55832: 28.1; Emacs crashes when using tramp from helm in emacs-29
Date: Fri, 17 Jun 2022 17:10:38 +0000
[Message part 1 (text/plain, inline)]
Hi Michael,

Michael Albinus <michael.albinus <at> gmx.de> writes:

> Thierry Volpiatto <thievol <at> posteo.net> writes:
>
>> Hi Michael,
>
> Hi Thierry,
>
>>> I made a code review of using property "process-buffer" in Tramp, and
>>> there is indeed a case it behaves incorrectly: after spawning an
>>> asynchronous process. Recipe:
>>>
>>> # emacs -Q /sudo::
>>>
>>> M-: (tramp-list-connections)
>>> => ((tramp-file-name "sudo" #("root" 0 4 (tramp-default t)) nil #("gandalf" 0 7 (tramp-default t)) nil nil nil))
>>>
>>> M-x async-shell-command RET ls
>>
>> Probably there is something else than this but couldn't figure out what, I
>> will let you know if I find other use cases.
>
> According to the code review, out-of-band methods (like "scp") are
> suspicious, too. And there might be some corner cases with the "smb" method.

Ok, I sometimes use scp but more rarely now that I have a rsync command
in helm, also last time I tried, scp method was not supporting more than
three file at the time (marked files), see
https://github.com/emacs-helm/helm/issues/1945.  
For smb I never used it. 

> I haven't tried to compose further recipes for problematic cases. I have
> simply changed the handling of the "process-buffer" and "process-name"
> properties in all Tramp files. This shall be good enough.

Ok thanks.

>>> M-: (tramp-list-connections)
>>> nil
>>>
>>> I'm working on a fix.
>>
>> Great thanks.
>
> I've pushed the fix to master.

Thanks, will try as soon as possible.

-- 
Thierry
[signature.asc (application/pgp-signature, inline)]

This bug report was last modified 2 years and 332 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.