73 lines
2.8 KiB
Diff
73 lines
2.8 KiB
Diff
|
From 543362e185edf822b9832b1953e78548ab42a0c5 Mon Sep 17 00:00:00 2001
|
||
|
|
From: Laszlo Ersek <lersek@redhat.com>
|
||
|
|
Date: Tue, 8 Jun 2021 14:12:51 +0200
|
||
|
|
Subject: [PATCH 02/11] NetworkPkg/IScsiDxe: simplify
|
||
|
|
"ISCSI_CHAP_AUTH_DATA.InChallenge" size
|
||
|
|
MIME-Version: 1.0
|
||
|
|
Content-Type: text/plain; charset=UTF-8
|
||
|
|
Content-Transfer-Encoding: 8bit
|
||
|
|
|
||
|
|
RH-Author: Laszlo Ersek <lersek@redhat.com>
|
||
|
|
RH-MergeRequest: 1: NetworkPkg/IScsiDxe: fix IScsiHexToBin() security and functionality bugs [RHEL-9, c9s]
|
||
|
|
RH-Commit: [2/10] d1c332767a87d87274e5ff68cb0c0f630ec095e1
|
||
|
|
RH-Bugzilla: 1961100
|
||
|
|
RH-Acked-by: Philippe Mathieu-Daudé <philmd@redhat.com>
|
||
|
|
|
||
|
|
The ISCSI_CHAP_AUTH_MAX_LEN macro is defined with value 1024.
|
||
|
|
|
||
|
|
The usage of this macro currently involves a semantic (not functional)
|
||
|
|
bug, which we're going to fix in a subsequent patch, eliminating
|
||
|
|
ISCSI_CHAP_AUTH_MAX_LEN altogether.
|
||
|
|
|
||
|
|
For now, remove the macro's usage from all
|
||
|
|
"ISCSI_CHAP_AUTH_DATA.InChallenge" contexts. This is doable without
|
||
|
|
duplicating open-coded constants.
|
||
|
|
|
||
|
|
No changes in functionality.
|
||
|
|
|
||
|
|
Cc: Jiaxin Wu <jiaxin.wu@intel.com>
|
||
|
|
Cc: Maciej Rabeda <maciej.rabeda@linux.intel.com>
|
||
|
|
Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
|
||
|
|
Cc: Siyuan Fu <siyuan.fu@intel.com>
|
||
|
|
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3356
|
||
|
|
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
|
||
|
|
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
|
||
|
|
Reviewed-by: Maciej Rabeda <maciej.rabeda@linux.intel.com>
|
||
|
|
Message-Id: <20210608121259.32451-3-lersek@redhat.com>
|
||
|
|
(cherry picked from commit 29cab43bb7912a12efa5a78dac15394aee866e4c)
|
||
|
|
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
||
|
|
---
|
||
|
|
NetworkPkg/IScsiDxe/IScsiCHAP.c | 2 +-
|
||
|
|
NetworkPkg/IScsiDxe/IScsiCHAP.h | 2 +-
|
||
|
|
2 files changed, 2 insertions(+), 2 deletions(-)
|
||
|
|
|
||
|
|
diff --git a/NetworkPkg/IScsiDxe/IScsiCHAP.c b/NetworkPkg/IScsiDxe/IScsiCHAP.c
|
||
|
|
index cbbc56ae5b..df3c2eb120 100644
|
||
|
|
--- a/NetworkPkg/IScsiDxe/IScsiCHAP.c
|
||
|
|
+++ b/NetworkPkg/IScsiDxe/IScsiCHAP.c
|
||
|
|
@@ -289,7 +289,7 @@ IScsiCHAPOnRspReceived (
|
||
|
|
}
|
||
|
|
|
||
|
|
AuthData->InIdentifier = (UINT32) Result;
|
||
|
|
- AuthData->InChallengeLength = ISCSI_CHAP_AUTH_MAX_LEN;
|
||
|
|
+ AuthData->InChallengeLength = (UINT32) sizeof (AuthData->InChallenge);
|
||
|
|
IScsiHexToBin (
|
||
|
|
(UINT8 *) AuthData->InChallenge,
|
||
|
|
&AuthData->InChallengeLength,
|
||
|
|
diff --git a/NetworkPkg/IScsiDxe/IScsiCHAP.h b/NetworkPkg/IScsiDxe/IScsiCHAP.h
|
||
|
|
index 5e59fb678b..1fc1d96ea3 100644
|
||
|
|
--- a/NetworkPkg/IScsiDxe/IScsiCHAP.h
|
||
|
|
+++ b/NetworkPkg/IScsiDxe/IScsiCHAP.h
|
||
|
|
@@ -49,7 +49,7 @@ typedef struct _ISCSI_CHAP_AUTH_CONFIG_NVDATA {
|
||
|
|
typedef struct _ISCSI_CHAP_AUTH_DATA {
|
||
|
|
ISCSI_CHAP_AUTH_CONFIG_NVDATA *AuthConfig;
|
||
|
|
UINT32 InIdentifier;
|
||
|
|
- UINT8 InChallenge[ISCSI_CHAP_AUTH_MAX_LEN];
|
||
|
|
+ UINT8 InChallenge[1024];
|
||
|
|
UINT32 InChallengeLength;
|
||
|
|
//
|
||
|
|
// Calculated CHAP Response (CHAP_R) value.
|
||
|
|
--
|
||
|
|
2.27.0
|
||
|
|
|