How to write a subprocess.Popen terminal execution

Question

I want to kill a process on a remote server with another user, who creted the process via python with the subprocess.Popen command. But there is something I must be doing wrong because nothing happens when I run:

subprocess.Popen(['sudo','kill','-9',str(pid)], stdout=subprocess.PIPE)

In terminal sudo kill -9 pid works fine.

Do you get any error? How are you passing the `sudo` password? — heemayl, Jun 26 '19 at 14:02
@heemayl No error, but you are right I have somehow pass the sudo password' But how? — Varlor, Jun 26 '19 at 14:07
@Legorooj Tried this one too, nothing happens, I think heemayl is right I need to pass the sudo password somehow — Varlor, Jun 26 '19 at 14:08
one may configure to run sudo without password via /etc/sudoers — Sav, Jun 26 '19 at 14:10
It does appear that way. Look at this question - it might help. https://stackoverflow.com/questions/5191878/change-to-sudo-user-within-a-python-script — Legorooj, Jun 26 '19 at 14:11
@Sav Varlor probably doesn't want passwordless sudo as a default on their machine. — Legorooj, Jun 26 '19 at 14:12

score 1 · Answer 1 · answered Aug 08 '19 at 20:28

Your answer works. A slightly more formal way of doing this would be something like

from subprocess import Popen, PIPE


def kill(pid, passwd):
    pipe = Popen(['sudo', '-S', 'kill', '-9', str(pid)], 
                 stdout=PIPE, 
                 stdin=PIPE, 
                 stderr=PIPE)
    pipe.stdin.write(bytes(passwd + '\n', encoding='utf-8'))
    pipe.stdin.flush()
    # at this point, the process is killed, return output and errors
    return (str(pipe.stdout.read()), str(pipe.stderr.read()))

If I were writing this for a production system, I would add some exception handling, so treat this as a proof of concept.

Definitely an improvement - much safer from a security point of view. — Legorooj, Sep 25 '19 at 01:06

score 0 · Accepted Answer · answered Jun 26 '19 at 14:32

0

Ok i got the answer thanks to the comment by Legorooj.

from subprocess import call
import os
command = 'kill -9 '+str(pid)
#p = os.system('echo {}|sudo -S {}'.format(password, command))
call('echo {} | sudo -S {}'.format(password, command), shell=True)

answered Jun 26 '19 at 14:32

Varlor

1,421
3
22
46

Good to know that I helped! Btw you should select this as the correct answer, that way it'll show up as answered. – Legorooj Jun 27 '19 at 16:17

How to write a subprocess.Popen terminal execution

2 Answers2