From faca89875bbfe0bf92e3598a2a5ac2086c2ca3e4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marek=20Marczykowski-G=C3=B3recki?= Date: Thu, 29 Mar 2018 00:53:17 +0200 Subject: [PATCH] storage/lvm: force default locale for lvm command Scripts do parse its output sometimes (especially `lvs`), so make sure we always gets the same format, regardless of the environment. Including decimal separator. Fixes QubesOS/qubes-issues#3753 --- qubes/storage/lvm.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/qubes/storage/lvm.py b/qubes/storage/lvm.py index 1ff08811..3cece332 100644 --- a/qubes/storage/lvm.py +++ b/qubes/storage/lvm.py @@ -566,8 +566,10 @@ def qubes_lvm(cmd, log=logging.getLogger('qubes.storage.lvm')): cmd = ['sudo', 'lvm'] + lvm_cmd else: cmd = ['lvm'] + lvm_cmd + environ = os.environ.copy() + environ['LC_ALL'] = 'C.utf8' p = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE, - close_fds=True) + close_fds=True, env=environ) out, err = p.communicate() return_code = p.returncode if out: