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