Don't assume that the user creating a VM has a locker with the same

name.

This isn't true, for example, for people using root instance tickets
with SPNEGO

svn path=/trunk/packages/invirt-web/; revision=2022

diff --git a/code/validation.py b/code/validation.py
index dd3278b..bd3be91 100644 (file)
--- a/code/validation.py
+++ b/code/validation.py
@@ -241,8 +241,6 @@ def testOwner(user, owner, machine=None):
 
     If machine is None, this is the owner of a new machine.
     """
 
     If machine is None, this is the owner of a new machine.
     """

-    if owner == user:
-        return owner

     if machine is not None and owner in (machine.owner, None):
         return machine.owner
     if owner is None:
     if machine is not None and owner in (machine.owner, None):
         return machine.owner
     if owner is None: