use utils.DEV_NULL everywhere instead of opening a new /dev/null fd each

time

diff --git a/motioneye/diskctl.py b/motioneye/diskctl.py
index aba8f7b04230bc0b8a86e2503e2115ad0c036d7f..8691474fa840a043bbbf28f1bcc8ae815b9f7432 100644 (file)
--- a/motioneye/diskctl.py
+++ b/motioneye/diskctl.py
@@ -19,6 +19,7 @@ import logging
 import os
 import re
 import subprocess
+import utils
 
 
 def _list_mounts():
@@ -150,7 +151,7 @@ def _list_disks_dev_by_id():
 
 def _list_disks_fdisk():
     try:
-        output = subprocess.check_output(['fdisk', '-l'], stderr=open('/dev/null', 'w'))
+        output = subprocess.check_output(['fdisk', '-l'], stderr=utils.DEV_NULL)
     
     except Exception as e:
         logging.error('failed to list disks using "fdisk -l": %s' % e, exc_info=True)

diff --git a/motioneye/mediafiles.py b/motioneye/mediafiles.py
index 6f2684f58a95bfd4b882b47203b13c63f25d5fa9..28cb1a09d925c11bc3af4aee1b56a9887fc3dc05 100644 (file)
--- a/motioneye/mediafiles.py
+++ b/motioneye/mediafiles.py
@@ -154,7 +154,7 @@ def _remove_older_files(dir, moment, exts):
 
 def find_ffmpeg():
     try:
-        return subprocess.check_output(['which', 'ffmpeg'], stderr=open('/dev/null', 'w')).strip()
+        return subprocess.check_output(['which', 'ffmpeg'], stderr=utils.DEV_NULL).strip()
     
     except subprocess.CalledProcessError: # not found
         return None

diff --git a/motioneye/motionctl.py b/motioneye/motionctl.py
index 0ed7f5cd4e52a35da74b3187380059870a373568..2950770df87eee87977cac6d25ff4e633e66d714 100644 (file)
--- a/motioneye/motionctl.py
+++ b/motioneye/motionctl.py
@@ -51,7 +51,7 @@ def find_motion():
 
     else: # autodetect motion binary path
         try:
-            binary = subprocess.check_output(['which', 'motion'], stderr=open('/dev/null', 'w')).strip()
+            binary = subprocess.check_output(['which', 'motion'], stderr=utils.DEV_NULL).strip()
         
         except subprocess.CalledProcessError: # not found
             return None

diff --git a/motioneye/powerctl.py b/motioneye/powerctl.py
index 403cb4927276f3d8f65af160ba30711c7c50713b..4171fbd8ce2f0e035d600cb4960fc85ce6356a7c 100644 (file)
--- a/motioneye/powerctl.py
+++ b/motioneye/powerctl.py
@@ -18,11 +18,12 @@
 import logging
 import os
 import subprocess
+import utils
 
 
 def _find_prog(prog):
     try:
-        return subprocess.check_output(['which', prog], stderr=open('/dev/null', 'w')).strip()
+        return subprocess.check_output(['which', prog], stderr=utils.DEV_NULL).strip()
     
     except subprocess.CalledProcessError: # not found
         return None

diff --git a/motioneye/smbctl.py b/motioneye/smbctl.py
index 42813789356f1cc2ae175205b75b6ad12992a2a2..668f7d1212d6cdb1f2a4f04b8be781fc4644c73e 100644 (file)
--- a/motioneye/smbctl.py
+++ b/motioneye/smbctl.py
@@ -21,6 +21,7 @@ import os
 import re
 import subprocess
 import time
+import utils
 
 from tornado.ioloop import IOLoop
 
@@ -39,7 +40,7 @@ def stop():
 
 def find_mount_cifs():
     try:
-        return subprocess.check_output(['which', 'mount.cifs'], stderr=open('/dev/null', 'w')).strip()
+        return subprocess.check_output(['which', 'mount.cifs'], stderr=utils.DEV_NULL).strip()
     
     except subprocess.CalledProcessError: # not found
         return None

diff --git a/motioneye/utils.py b/motioneye/utils.py
index 212379fce344616edc0fb1960bc6d51478df7669..bb7b995920f268f74e0343b0ec5f5d50c1341dc4 100644 (file)
--- a/motioneye/utils.py
+++ b/motioneye/utils.py
@@ -43,6 +43,8 @@ except:
 _SIGNATURE_REGEX = re.compile('[^a-zA-Z0-9/?_.=&{}\[\]":, _-]')
 _SPECIAL_COOKIE_NAMES = {'expires', 'domain', 'path', 'secure', 'httponly'}
 
+DEV_NULL = open('/dev/null', 'w')
+
 
 COMMON_RESOLUTIONS = [
     (320, 240),

diff --git a/motioneye/v4l2ctl.py b/motioneye/v4l2ctl.py
index 77bb33f1f7b28c8e5b1fe5a372a4c74e2d09a35b..3e97de5e833260238e7504afdf028f308b321e75 100644 (file)
--- a/motioneye/v4l2ctl.py
+++ b/motioneye/v4l2ctl.py
@@ -35,7 +35,7 @@ _DEV_V4L_BY_ID = '/dev/v4l/by-id/'
 
 def find_v4l2_ctl():
     try:
-        return subprocess.check_output(['which', 'v4l2-ctl'], stderr=open('/dev/null', 'w')).strip()
+        return subprocess.check_output(['which', 'v4l2-ctl'], stderr=utils.DEV_NULL).strip()
     
     except subprocess.CalledProcessError: # not found
         return None