Use a context manager to clean up after qrexec call

This avoids leaking a subprocess.Popen object.
QubesOS · Jul 10, 2022 · 8bc809e · 8bc809e
1 parent accfebc
commit 8bc809e
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/qvm-tools/qvm-sync-clock b/qvm-tools/qvm-sync-clock
@@ -41,8 +41,8 @@ def main():
             clockvm.name))
         sys.exit(0)
 
-    p = clockvm.run_service('qubes.GetDate+nanoseconds')
-    untrusted_date_out = p.stdout.read(36)
+    with clockvm.run_service('qubes.GetDate+nanoseconds') as p:
+        untrusted_date_out = p.stdout.read(36)
     try:
         untrusted_date_out.decode('ascii', 'strict')
     except UnicodeDecodeError: