|
 |
c4e3b2 |
From 618ba71beb3f848660c8c95187d92f2c8f277143 Mon Sep 17 00:00:00 2001
|
|
|
c4e3b2 |
From: Laszlo Ersek <lersek@redhat.com>
|
|
|
c4e3b2 |
Date: Tue, 8 Jun 2021 14:12:56 +0200
|
|
|
c4e3b2 |
Subject: [PATCH 07/11] NetworkPkg/IScsiDxe: reformat IScsiHexToBin() leading
|
|
|
c4e3b2 |
comment block
|
|
|
c4e3b2 |
MIME-Version: 1.0
|
|
|
c4e3b2 |
Content-Type: text/plain; charset=UTF-8
|
|
|
c4e3b2 |
Content-Transfer-Encoding: 8bit
|
|
|
c4e3b2 |
|
|
|
c4e3b2 |
RH-Author: Laszlo Ersek <lersek@redhat.com>
|
|
|
c4e3b2 |
RH-MergeRequest: 1: NetworkPkg/IScsiDxe: fix IScsiHexToBin() security and functionality bugs [RHEL-9, c9s]
|
|
|
c4e3b2 |
RH-Commit: [7/10] ea7e41e567759e461777094ae2049a29eb5c3826
|
|
|
c4e3b2 |
RH-Bugzilla: 1961100
|
|
|
c4e3b2 |
RH-Acked-by: Philippe Mathieu-Daudé <philmd@redhat.com>
|
|
|
c4e3b2 |
|
|
|
c4e3b2 |
We'll need further return values for IScsiHexToBin() in a subsequent
|
|
|
c4e3b2 |
patch; make room for them in the leading comment block of the function.
|
|
|
c4e3b2 |
While at it, rewrap the comment block to 80 characters width.
|
|
|
c4e3b2 |
|
|
|
c4e3b2 |
No functional changes.
|
|
|
c4e3b2 |
|
|
|
c4e3b2 |
Cc: Jiaxin Wu <jiaxin.wu@intel.com>
|
|
|
c4e3b2 |
Cc: Maciej Rabeda <maciej.rabeda@linux.intel.com>
|
|
|
c4e3b2 |
Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
|
|
|
c4e3b2 |
Cc: Siyuan Fu <siyuan.fu@intel.com>
|
|
|
c4e3b2 |
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3356
|
|
|
c4e3b2 |
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
c4e3b2 |
Reviewed-by: Maciej Rabeda <maciej.rabeda@linux.intel.com>
|
|
|
c4e3b2 |
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
|
|
|
c4e3b2 |
Message-Id: <20210608121259.32451-8-lersek@redhat.com>
|
|
|
c4e3b2 |
(cherry picked from commit dc469f137110fe79704b8b92c552972c739bb915)
|
|
|
c4e3b2 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
c4e3b2 |
---
|
|
|
c4e3b2 |
NetworkPkg/IScsiDxe/IScsiMisc.c | 16 ++++++++--------
|
|
|
c4e3b2 |
NetworkPkg/IScsiDxe/IScsiMisc.h | 16 ++++++++--------
|
|
|
c4e3b2 |
2 files changed, 16 insertions(+), 16 deletions(-)
|
|
|
c4e3b2 |
|
|
|
c4e3b2 |
diff --git a/NetworkPkg/IScsiDxe/IScsiMisc.c b/NetworkPkg/IScsiDxe/IScsiMisc.c
|
|
|
c4e3b2 |
index 42988e15cb..014700e87a 100644
|
|
|
c4e3b2 |
--- a/NetworkPkg/IScsiDxe/IScsiMisc.c
|
|
|
c4e3b2 |
+++ b/NetworkPkg/IScsiDxe/IScsiMisc.c
|
|
|
c4e3b2 |
@@ -370,14 +370,14 @@ IScsiBinToHex (
|
|
|
c4e3b2 |
/**
|
|
|
c4e3b2 |
Convert the hexadecimal string into a binary encoded buffer.
|
|
|
c4e3b2 |
|
|
|
c4e3b2 |
- @param[in, out] BinBuffer The binary buffer.
|
|
|
c4e3b2 |
- @param[in, out] BinLength Length of the binary buffer.
|
|
|
c4e3b2 |
- @param[in] HexStr The hexadecimal string.
|
|
|
c4e3b2 |
-
|
|
|
c4e3b2 |
- @retval EFI_SUCCESS The hexadecimal string is converted into a binary
|
|
|
c4e3b2 |
- encoded buffer.
|
|
|
c4e3b2 |
- @retval EFI_BUFFER_TOO_SMALL The binary buffer is too small to hold the converted data.
|
|
|
c4e3b2 |
-
|
|
|
c4e3b2 |
+ @param[in, out] BinBuffer The binary buffer.
|
|
|
c4e3b2 |
+ @param[in, out] BinLength Length of the binary buffer.
|
|
|
c4e3b2 |
+ @param[in] HexStr The hexadecimal string.
|
|
|
c4e3b2 |
+
|
|
|
c4e3b2 |
+ @retval EFI_SUCCESS The hexadecimal string is converted into a
|
|
|
c4e3b2 |
+ binary encoded buffer.
|
|
|
c4e3b2 |
+ @retval EFI_BUFFER_TOO_SMALL The binary buffer is too small to hold the
|
|
|
c4e3b2 |
+ converted data.
|
|
|
c4e3b2 |
**/
|
|
|
c4e3b2 |
EFI_STATUS
|
|
|
c4e3b2 |
IScsiHexToBin (
|
|
|
c4e3b2 |
diff --git a/NetworkPkg/IScsiDxe/IScsiMisc.h b/NetworkPkg/IScsiDxe/IScsiMisc.h
|
|
|
c4e3b2 |
index 231413993b..28cf408cd5 100644
|
|
|
c4e3b2 |
--- a/NetworkPkg/IScsiDxe/IScsiMisc.h
|
|
|
c4e3b2 |
+++ b/NetworkPkg/IScsiDxe/IScsiMisc.h
|
|
|
c4e3b2 |
@@ -165,14 +165,14 @@ IScsiBinToHex (
|
|
|
c4e3b2 |
/**
|
|
|
c4e3b2 |
Convert the hexadecimal string into a binary encoded buffer.
|
|
|
c4e3b2 |
|
|
|
c4e3b2 |
- @param[in, out] BinBuffer The binary buffer.
|
|
|
c4e3b2 |
- @param[in, out] BinLength Length of the binary buffer.
|
|
|
c4e3b2 |
- @param[in] HexStr The hexadecimal string.
|
|
|
c4e3b2 |
-
|
|
|
c4e3b2 |
- @retval EFI_SUCCESS The hexadecimal string is converted into a binary
|
|
|
c4e3b2 |
- encoded buffer.
|
|
|
c4e3b2 |
- @retval EFI_BUFFER_TOO_SMALL The binary buffer is too small to hold the converted data.
|
|
|
c4e3b2 |
-
|
|
|
c4e3b2 |
+ @param[in, out] BinBuffer The binary buffer.
|
|
|
c4e3b2 |
+ @param[in, out] BinLength Length of the binary buffer.
|
|
|
c4e3b2 |
+ @param[in] HexStr The hexadecimal string.
|
|
|
c4e3b2 |
+
|
|
|
c4e3b2 |
+ @retval EFI_SUCCESS The hexadecimal string is converted into a
|
|
|
c4e3b2 |
+ binary encoded buffer.
|
|
|
c4e3b2 |
+ @retval EFI_BUFFER_TOO_SMALL The binary buffer is too small to hold the
|
|
|
c4e3b2 |
+ converted data.
|
|
|
c4e3b2 |
**/
|
|
|
c4e3b2 |
EFI_STATUS
|
|
|
c4e3b2 |
IScsiHexToBin (
|
|
|
c4e3b2 |
--
|
|
|
c4e3b2 |
2.27.0
|
|
|
c4e3b2 |
|