[checkmk-commits] Fixed another subprocess call that involves a shell

Module: check_mk Branch: master Commit: 207151d0732ce8a72237b194361ed3a08f9063ed URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=207151d0732ce8… Author: Lars Michelsen <lm(a)mathias-kettner.de> Date: Wed Nov 9 17:10:58 2016 +0100 Fixed another subprocess call that involves a shell --- cmk_base/localize.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmk_base/localize.py b/cmk_base/localize.py index a2a3b11..9e1517e 100644 --- a/cmk_base/localize.py +++ b/cmk_base/localize.py @@ -130,7 +130,7 @@ def write_alias(alias): def check_binaries(): # Are the xgettext utils available? for b in [ 'xgettext', 'msgmerge', 'msgfmt' ]: - if os.system('which %s >/dev/null 2>&1' % b): + if subprocess.call(['which', b], stdout=open(os.devnull , "wb")) != 0: raise LocalizeException('%s binary not found in PATH\n' % b)