weizhouapache commented on issue #6049:
URL: https://github.com/apache/cloudstack/issues/6049#issuecomment-1054422439


   @GabrielBrascher 
   oh, I have found the issue and fixed in #5769  , but never created PR for it
   
   
   
[server/src/main/java/com/cloud/vm/UserVmManagerImpl.java](https://github.com/apache/cloudstack/pull/5769/files#diff-865f710fe60b9b62ff8c2cbce8a8342e4f6fca658223119762285fd5f7857f08)[](https://github.com/apache/cloudstack/pull/5769/files#diff-865f710fe60b9b62ff8c2cbce8a8342e4f6fca658223119762285fd5f7857f08)
 
   
   ```
   diff --git a/server/src/main/java/com/cloud/vm/UserVmManagerImpl.java 
b/server/src/main/java/com/cloud/vm/UserVmManagerImpl.java
   index 8b266b9481..58bba1e393 100644
   --- a/server/src/main/java/com/cloud/vm/UserVmManagerImpl.java
   +++ b/server/src/main/java/com/cloud/vm/UserVmManagerImpl.java
   @@ -1329,7 +1329,7 @@ public class UserVmManagerImpl extends ManagerBase 
implements UserVmManager, Vir
            }
    
            // Perform account permission check on network
   -        _accountMgr.checkAccess(caller, AccessType.UseEntry, false, 
network);
   +        _accountMgr.checkAccess(vmOwner, AccessType.UseEntry, false, 
network);
    
            //ensure network belongs in zone
            if (network.getDataCenterId() != vmInstance.getDataCenterId()) {
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to