From: Oleg Broytman <phd@phdru.name>
Date: Wed, 23 Aug 2017 19:22:54 +0000 (+0300)
Subject: Use subprocess.Popen instead of os.popen
X-Git-Tag: 2.8.0~32
X-Git-Url: https://git.phdru.name/?p=mimedecode.git;a=commitdiff_plain;h=bf2931c9491ed9d9155dbd97feb48ac5fb0bde49

Use subprocess.Popen instead of os.popen
---

diff --git a/mimedecode.py b/mimedecode.py
index 3b1df3e..9433b0d 100755
--- a/mimedecode.py
+++ b/mimedecode.py
@@ -2,6 +2,7 @@
 """Decode MIME message"""
 
 import sys, os
+import subprocess
 from mimedecode_version import __version__, \
     __author__, __copyright__, __license__
 if sys.version_info[0] >= 3:
@@ -250,9 +251,10 @@ def decode_body(msg, s):
     outfile.write(s)
     outfile.close()
 
-    pipe = os.popen(command, 'r')
-    new_s = pipe.read()
-    if pipe.close() is None: # result=0, Ok
+    pipe = subprocess.Popen(command, shell=True, stdout=subprocess.PIPE)
+    new_s = pipe.stdout.read()
+    pipe.stdout.close()
+    if pipe.wait() == 0: # result=0, Ok
         s = new_s
     os.remove(filename)