Revert "qrexec: fix "yes to all" for qrexec calls with custom argument"

Do not copy policy file at arbitrary time.
This reverts commit 1dff6361b7.
This commit is contained in:
Marek Marczykowski-Górecki 2016-10-31 14:17:47 +01:00
parent c15841c828
commit 610902a5c1
No known key found for this signature in database
GPG Key ID: 063938BA42CFA724

View File

@ -9,7 +9,6 @@ import qubes.guihelpers
import libvirt
from optparse import OptionParser
import fcntl
import shutil
POLICY_FILE_DIR="/etc/qubes-rpc/policy"
# XXX: Backward compatibility, to be removed soon
@ -137,12 +136,7 @@ def confirm_execution(domain, target, service_name):
def add_always_allow(domain, target, service_name, options):
policy_file=POLICY_FILE_DIR+"/"+service_name
if not os.path.isfile(policy_file):
# if we add "always allow" for specifc argument value, base the new
# file on the generic one
policy_file_source = os.path.join(POLICY_FILE_DIR, service_name.split("+")[0])
if not os.path.isfile(policy_file_source):
return None
shutil.copy2(policy_file_source, policy_file)
f = open(policy_file, 'r+')
fcntl.flock(f, fcntl.LOCK_EX)
lines = []