|
 |
456be9 |
From 78c066aa4912d8ccea958dffc39a73667c9979c6 Mon Sep 17 00:00:00 2001
|
|
|
456be9 |
From: Cathy Avery <cavery@redhat.com>
|
|
|
456be9 |
Date: Mon, 29 Aug 2022 12:50:45 -0400
|
|
|
456be9 |
Subject: [PATCH] Properly check authorization on incoming guestOps requests.
|
|
|
456be9 |
|
|
|
456be9 |
RH-Author: Cathy Avery <cavery@redhat.com>
|
|
|
456be9 |
RH-MergeRequest: 11: Properly check authorization on incoming guestOps requests.
|
|
|
456be9 |
RH-Bugzilla: 2119283
|
|
|
456be9 |
RH-Acked-by: Mohamed Gamal Morsy <mmorsy@redhat.com>
|
|
|
456be9 |
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
456be9 |
RH-Acked-by: Vitaly Kuznetsov <vkuznets@redhat.com>
|
|
|
456be9 |
RH-Commit: [1/1] 546959f6b3546cc19ff844041ff363ea46793d6c
|
|
|
456be9 |
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2119283
|
|
|
456be9 |
Brew: https://brewweb.engineering.redhat.com/brew/taskinfo?taskID=47491717
|
|
|
456be9 |
Tested: Tested by QE
|
|
|
456be9 |
Upstream Status: origin/master
|
|
|
456be9 |
Conflicts: None
|
|
|
456be9 |
|
|
|
456be9 |
commit 70a74758bfe0042c27f15ce590fb21a2bc54d745
|
|
|
456be9 |
Author: John Wolfe <jwolfe@vmware.com>
|
|
|
456be9 |
Date: Sun Aug 21 07:56:49 2022 -0700
|
|
|
456be9 |
|
|
|
456be9 |
Properly check authorization on incoming guestOps requests.
|
|
|
456be9 |
|
|
|
456be9 |
Fix public pipe request checks. Only a SessionRequest type should
|
|
|
456be9 |
be accepted on the public pipe.
|
|
|
456be9 |
|
|
|
456be9 |
Signed-off-by: Cathy Avery <cavery@redhat.com>
|
|
|
456be9 |
---
|
|
|
456be9 |
vgauth/serviceImpl/proto.c | 6 +++++-
|
|
|
456be9 |
1 file changed, 5 insertions(+), 1 deletion(-)
|
|
|
456be9 |
|
|
|
456be9 |
diff --git a/open-vm-tools/vgauth/serviceImpl/proto.c b/open-vm-tools/vgauth/serviceImpl/proto.c
|
|
|
456be9 |
index db7159ee..6c672601 100644
|
|
|
456be9 |
--- a/open-vm-tools/vgauth/serviceImpl/proto.c
|
|
|
456be9 |
+++ b/open-vm-tools/vgauth/serviceImpl/proto.c
|
|
|
456be9 |
@@ -1,5 +1,5 @@
|
|
|
456be9 |
/*********************************************************
|
|
|
456be9 |
- * Copyright (C) 2011-2016,2019-2021 VMware, Inc. All rights reserved.
|
|
|
456be9 |
+ * Copyright (C) 2011-2016,2019-2022 VMware, Inc. All rights reserved.
|
|
|
456be9 |
*
|
|
|
456be9 |
* This program is free software; you can redistribute it and/or modify it
|
|
|
456be9 |
* under the terms of the GNU Lesser General Public License as published
|
|
|
456be9 |
@@ -1201,6 +1201,10 @@ Proto_SecurityCheckRequest(ServiceConnection *conn,
|
|
|
456be9 |
VGAuthError err;
|
|
|
456be9 |
gboolean isSecure = ServiceNetworkIsConnectionPrivateSuperUser(conn);
|
|
|
456be9 |
|
|
|
456be9 |
+ if (conn->isPublic && req->reqType != PROTO_REQUEST_SESSION_REQ) {
|
|
|
456be9 |
+ return VGAUTH_E_PERMISSION_DENIED;
|
|
|
456be9 |
+ }
|
|
|
456be9 |
+
|
|
|
456be9 |
switch (req->reqType) {
|
|
|
456be9 |
/*
|
|
|
456be9 |
* This comes over the public connection; alwsys let it through.
|
|
|
456be9 |
--
|
|
|
456be9 |
2.35.3
|
|
|
456be9 |
|