From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id 9CF4BD8118F for ; Tue, 19 Mar 2024 08:49:23 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=azIZ17ahiqd/aIvPA3o9ANc6r22Qa5RLTfk0uEIXuBc=; c=relaxed/simple; d=groups.io; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Resent-Date:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Transfer-Encoding; s=20240206; t=1710838162; v=1; b=NA6beMzvW2gwEC8Beapr3ByhwviCwgmAPxLC1zzxvVxHDLzJfepAcDl0gzWecvFHqITB/aSL RZ0IFtMIEHn0lOEnIr5cxocN5vmalgrPh/UP5FSnxgoz+desgh+F1vqpxtqmQzXmkDB1yT0pf8j Gor4DXZR7hUUVyAMVju20bb9LKIQYf5IKgswiDNMg5EQ51VPsuDw5nRC53K0hQ3Vm688AxUuC/L 0ozSonaG8Hig+WBfoo7WnoT6OCr783lUTZJsHOm/lizHxXPGFjhZBdtKgXxcJhPxinhxik+4Ynh r6JFNlp7p3vUpA82+uZh9CKxNt0oa4Aw5GeZn182brAKg== X-Received: by 127.0.0.2 with SMTP id ZASCYY7687511xSMvY0LHa58; Tue, 19 Mar 2024 01:49:22 -0700 X-Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.10]) by mx.groups.io with SMTP id smtpd.web10.10306.1710838161400299109 for ; Tue, 19 Mar 2024 01:49:21 -0700 X-IronPort-AV: E=McAfee;i="6600,9927,11017"; a="23149673" X-IronPort-AV: E=Sophos;i="6.07,136,1708416000"; d="scan'208";a="23149673" X-Received: from orviesa006.jf.intel.com ([10.64.159.146]) by orvoesa102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Mar 2024 01:49:21 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.07,136,1708416000"; d="scan'208";a="14128364" X-Received: from babiosdev005.gar.corp.intel.com ([10.223.9.180]) by orviesa006-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Mar 2024 01:49:20 -0700 From: "Nayana Patel" To: devel@edk2.groups.io Cc: bhavana.s@intel.com, tabassum.yasmin@intel.com, Nayana Patel Subject: [edk2-devel] [PATCH] REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4645 Date: Tue, 19 Mar 2024 14:19:03 +0530 Message-Id: MIME-Version: 1.0 Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Resent-Date: Tue, 19 Mar 2024 01:49:21 -0700 Reply-To: devel@edk2.groups.io,nayana.patel@intel.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: fGf65Z9xYPlYokXy6oflLXl9x7686176AA= Content-Transfer-Encoding: 8bit X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240206 header.b=NA6beMzv; spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=intel.com (policy=none) Clear out the variable SmmCommunicateSetPassword which contains password before goto Exit. To avoid vulnerability. Signed-off-by: Nayana Patel --- .../UserAuthenticationDxeSmm/UserAuthenticationSmm.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Features/Intel/UserInterface/UserAuthFeaturePkg/UserAuthenticationDxeSmm/UserAuthenticationSmm.c b/Features/Intel/UserInterface/UserAuthFeaturePkg/UserAuthenticationDxeSmm/UserAuthenticationSmm.c index 98f40c1812..ba01d599e0 100644 --- a/Features/Intel/UserInterface/UserAuthFeaturePkg/UserAuthenticationDxeSmm/UserAuthenticationSmm.c +++ b/Features/Intel/UserInterface/UserAuthFeaturePkg/UserAuthenticationDxeSmm/UserAuthenticationSmm.c @@ -555,6 +555,7 @@ SmmPasswordHandler ( if (PasswordLen == sizeof(SmmCommunicateVerifyPassword.Password)) { DEBUG ((DEBUG_ERROR, "SmmPasswordHandler: Password invalid!\n")); Status = EFI_INVALID_PARAMETER; + ZeroMem (&SmmCommunicateVerifyPassword, sizeof (SmmCommunicateVerifyPassword)); goto EXIT; } if (!IsPasswordVerified (UserGuid, SmmCommunicateVerifyPassword.Password, PasswordLen + 1)) { @@ -565,6 +566,7 @@ SmmPasswordHandler ( } else { Status = EFI_SECURITY_VIOLATION; } + ZeroMem (&SmmCommunicateVerifyPassword, sizeof (SmmCommunicateVerifyPassword)); goto EXIT; } mPasswordVerified = TRUE; -- 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#116880): https://edk2.groups.io/g/devel/message/116880 Mute This Topic: https://groups.io/mt/105020497/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=-