* bug#1112: 23.0.60; Child process not cleaned up properly
@ 2008-10-07 15:15 Brent Goodrick
0 siblings, 0 replies; 4+ messages in thread
From: Brent Goodrick @ 2008-10-07 15:15 UTC (permalink / raw)
To: emacs-pretest-bug
Please write in English if possible, because the Emacs maintainers
usually do not have translators to read other languages for them.
Your bug report will be posted to the emacs-pretest-bug@gnu.org mailing list.
Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:
1. M-x compile
2. Enter in: sudo apt-get install gimp-help-en
3. See the apt-get prompt:
Reading package lists... 0%
<snip>
The following extra packages will be installed:
gimp-help-common
The following NEW packages will be installed:
gimp-help-common gimp-help-en
0 upgraded, 2 newly installed, 0 to remove and 1 not upgraded.
Need to get 15.9MB of archives.
After this operation, 27.5MB of additional disk space will be used.
Do you want to continue [Y/n]?
4. Kill the buffer, and expect the underlying process to die, just
like you would have if you had typed the above command in a shell
buffer.
5. Open up a shell, and type ps to see that the apt-get process still
exists
6. Go through step 1 again and notice now that a lock is being
reported by the second apt-get session because the first process
was not properly torn down by the act of killing the previous
compilation buffer.
My assessment: The shell mode somehow works differently than the
compilation mode since the compilation mode does not allow user
input. Fair enough, but the two modes should work the same in terms of
tearing down the two processes if the buffers are killed, and should
not ever leave dormant child processes.
If Emacs crashed, and you have the Emacs process in the gdb debugger,
please include the output from the following gdb commands:
`bt full' and `xbacktrace'.
If you would like to further debug the crash, please read the file
/home/brentg/emacs_from_source/install/share/emacs/23.0.60/etc/DEBUG
for instructions.
Emacs did not crash.
In GNU Emacs 23.0.60.1 (x86_64-unknown-linux-gnu, GTK+ Version 2.12.11)
of 2008-10-03 on hungover
Windowing system distributor `The X.Org Foundation', version 11.0.10402000
configured using `configure '--with-x-toolkit' '--with-xft'
'--prefix=/home/brentg/emacs_from_source/install''
Important settings:
value of $LC_ALL: nil
value of $LC_COLLATE: nil
value of $LC_CTYPE: nil
value of $LC_MESSAGES: nil
value of $LC_MONETARY: nil
value of $LC_NUMERIC: nil
value of $LC_TIME: nil
value of $LANG: en_US.UTF-8
value of $XMODIFIERS: nil
locale-coding-system: utf-8-unix
default-enable-multibyte-characters: t
Major mode: Apropos
Minor modes in effect:
desktop-save-mode: t
iswitchb-mode: t
erc-services-mode: t
erc-networks-mode: t
display-time-mode: t
shell-dirtrack-mode: t
delete-selection-mode: t
mouse-wheel-mode: t
menu-bar-mode: t
file-name-shadow-mode: t
global-font-lock-mode: t
font-lock-mode: t
blink-cursor-mode: t
global-auto-composition-mode: t
auto-composition-mode: t
auto-encryption-mode: t
auto-compression-mode: t
line-number-mode: 1
transient-mark-mode: t
Recent input:
o m p i <return> C-1 d C-x h <backspace> <return> M-P
<return> C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p
C-p C-a C-n C-f C-f C-f C-f C-f C-f C-f C-f C-f C-f
C-f C-b C-b C-M-SPC C-z M-> C-4 M-P <return> k i l
l C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p C-a
C-n C-n C-n C-n C-p C-p C-p C-p C-n C-f C-f C-f C-f
C-f C-f C-f C-f C-f C-M-SPC C-z M-> SPC C-v C-a <return>
M-P C-a s u d i <backspace> o SPC <return> C-4 M-P
C-k C-g C-v <return> C-1 C-c d p s f <return> C-r a
p t - C-g C-g M-> a p t = g e t <backspace> <backspace>
<backspace> <backspace> <C-backspace> C-r a p t - g
e t C-g C-f M-> M-x a p r o <tab> <return> p t y <backspace>
t y <backspace> <backspace> y <return> C-x o C-s c
o m i n t C-b <return> C-x o C-M-SPC C-z C-c d C-x
h <backspace> M-P SPC | SPC g r e p SPC a p t - g e
t <return> M-x M-P C-g C-4 M-p <return> C-x o M-: C-v
<return> <down-mouse-1> <mouse-1> C-1 M-P <return>
s u d o SPC k i l l SPC a p t = <backspace> - g e t
<backspace> <backspace> <backspace> <backspace> <backspace>
<backspace> <backspace> 1 2 2 5 <backspace> 6 0 <return>
M-x a p r o <tab> <return> e m a c s . * f <backspace>
b u g <return> C-x o C-n C-n C-n C-n C-n C-n C-n C-n
C-n C-n <return> M-x r e p o r t - e m a c s = <backspace>
- b u g <tab> <return>
Recent messages:
Not applicable. It is repeatable.
^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#1112: 23.0.60; Child process not cleaned up properly
@ 2008-10-07 19:23 Chong Yidong
0 siblings, 0 replies; 4+ messages in thread
From: Chong Yidong @ 2008-10-07 19:23 UTC (permalink / raw)
To: Brent Goodrick; +Cc: 1112
> 1. M-x compile
> 2. Enter in: sudo apt-get install gimp-help-en
> 4. Kill the buffer, and expect the underlying process to die
> 5. Open up a shell, and type ps to see that the apt-get process still
> exists
I suspect Emacs may fail to kill the process due to insufficient
permissions: it's a sudo process.
If you run Emacs with sudo, does the same problem appear?
^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#1112: 23.0.60; Child process not cleaned up properly
[not found] <mailman.474.1223398646.25473.bug-gnu-emacs@gnu.org>
@ 2008-10-07 19:24 ` Sven Joachim
2008-10-08 14:53 ` Brent Goodrick
0 siblings, 1 reply; 4+ messages in thread
From: Sven Joachim @ 2008-10-07 19:24 UTC (permalink / raw)
To: Brent Goodrick; +Cc: 1112
On 2008-10-07 17:15 +0200, Brent Goodrick wrote:
> 1. M-x compile
> 2. Enter in: sudo apt-get install gimp-help-en
> 3. See the apt-get prompt:
> Reading package lists... 0%
> <snip>
> The following extra packages will be installed:
> gimp-help-common
> The following NEW packages will be installed:
> gimp-help-common gimp-help-en
> 0 upgraded, 2 newly installed, 0 to remove and 1 not upgraded.
> Need to get 15.9MB of archives.
> After this operation, 27.5MB of additional disk space will be used.
> Do you want to continue [Y/n]?
> 4. Kill the buffer, and expect the underlying process to die, just
> like you would have if you had typed the above command in a shell
> buffer.
Won't work for processes run under sudo, see below.
> 5. Open up a shell, and type ps to see that the apt-get process still
> exists
> 6. Go through step 1 again and notice now that a lock is being
> reported by the second apt-get session because the first process
> was not properly torn down by the act of killing the previous
> compilation buffer.
>
> My assessment: The shell mode somehow works differently than the
> compilation mode since the compilation mode does not allow user
> input. Fair enough, but the two modes should work the same in terms of
> tearing down the two processes if the buffers are killed, and should
> not ever leave dormant child processes.
The real problem is that sudo is suid root and thus the compilation
process runs with superuser rights. Emacs is simply lacking the
privileges to kill it.
You can try something similar in your shell:
,----
| % sudo sleep 1000 &
| [1] 2186
| % kill %1
| kill: kill %1 failed: operation not permitted
| % sudo kill $(pidof sleep)
| [1] + 2186 terminated sudo sleep 1000
| %
`----
Sven
^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#1112: 23.0.60; Child process not cleaned up properly
2008-10-07 19:24 ` Sven Joachim
@ 2008-10-08 14:53 ` Brent Goodrick
0 siblings, 0 replies; 4+ messages in thread
From: Brent Goodrick @ 2008-10-08 14:53 UTC (permalink / raw)
To: Sven Joachim; +Cc: 1112
Hi Sven,
I agree about the part about the process permissions: Emacs definitely
cannot kill the process because it is a sudo process. However, if
Emacs fails to kill the process, Emacs should emit a warning message
as to the reason why the process could not be killed (e.g., "process
id 12345 could not be killed: operation not permitted"). This would
be appropriate in the case where the user kills the process buffer on
a long running process (a process that is not attempting to read from
standard input as this one is; see next paragraph). This warning is
appropriate because otherwise, the user thinks that the process died
on its own and will have to discover the hard way via some degree of
head-scratching that the process could not be killed.
There is more to this problem than process permissions. The process
has opened up standard input for the Y/N prompt. When I run
compilation mode on some process, I expect no user prompts (if I
wanted to handle prompts,I would execute it under the ever so handy
shell mode). But, when that process does read standard input for
prompting, I want that process to exit immediately (and most well
behaved apps do this these days, with the exception of Bourne shell
scripts using the "read" operator which doesn't behave correctly IMO).
As a proof of concept, when I type "sudo apt-get some_package <
/dev/null" into a M-x compile prompt, I see reasonable behavior:
<snip>
After this operation, 27.5MB of additional disk space will be used.
Do you want to continue [Y/n]? Abort.
Compilation exited abnormally with code 1 at Wed Oct 8 07:18:54
So, is there any harm in having compilation mode close standard input
when it spawns any and all processes (perhaps only under the direction
of a customizable and properly documented defvar variable so as to
avoid backlash from users expecting the existing behavior)? If so,
this would fix this issue in the majority of cases. Actually, this
closing of standard input should be an option on the lowest level
Elisp command that compile uses to start processes, if it is not
already exposed as such: I imagine that having to hack this into the
compile mode Elisp code itself would be problematic given that the "<
/dev/null" construct would have to vary quite a bit on the two axes of
type of shell and execution platform.
Thanks,
Brent
On Tue, Oct 7, 2008 at 12:24 PM, Sven Joachim <svenjoac@gmx.de> wrote:
>
> On 2008-10-07 17:15 +0200, Brent Goodrick wrote:
>
> > 1. M-x compile
> > 2. Enter in: sudo apt-get install gimp-help-en
> > 3. See the apt-get prompt:
> > Reading package lists... 0%
> > <snip>
> > The following extra packages will be installed:
> > gimp-help-common
> > The following NEW packages will be installed:
> > gimp-help-common gimp-help-en
> > 0 upgraded, 2 newly installed, 0 to remove and 1 not upgraded.
> > Need to get 15.9MB of archives.
> > After this operation, 27.5MB of additional disk space will be used.
> > Do you want to continue [Y/n]?
> > 4. Kill the buffer, and expect the underlying process to die, just
> > like you would have if you had typed the above command in a shell
> > buffer.
>
> Won't work for processes run under sudo, see below.
>
> > 5. Open up a shell, and type ps to see that the apt-get process still
> > exists
> > 6. Go through step 1 again and notice now that a lock is being
> > reported by the second apt-get session because the first process
> > was not properly torn down by the act of killing the previous
> > compilation buffer.
> >
> > My assessment: The shell mode somehow works differently than the
> > compilation mode since the compilation mode does not allow user
> > input. Fair enough, but the two modes should work the same in terms of
> > tearing down the two processes if the buffers are killed, and should
> > not ever leave dormant child processes.
>
> The real problem is that sudo is suid root and thus the compilation
> process runs with superuser rights. Emacs is simply lacking the
> privileges to kill it.
>
> You can try something similar in your shell:
>
> ,----
> | % sudo sleep 1000 &
> | [1] 2186
> | % kill %1
> | kill: kill %1 failed: operation not permitted
> | % sudo kill $(pidof sleep)
> | [1] + 2186 terminated sudo sleep 1000
> | %
> `----
>
> Sven
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2008-10-08 14:53 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-10-07 15:15 bug#1112: 23.0.60; Child process not cleaned up properly Brent Goodrick
-- strict thread matches above, loose matches on Subject: below --
2008-10-07 19:23 Chong Yidong
[not found] <mailman.474.1223398646.25473.bug-gnu-emacs@gnu.org>
2008-10-07 19:24 ` Sven Joachim
2008-10-08 14:53 ` Brent Goodrick
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.