audk/NetworkPkg/IScsiDxe
Laszlo Ersek b8649cf2a3 NetworkPkg/IScsiDxe: check IScsiHexToBin() return values
IScsiDxe (that is, the initiator) receives two hex-encoded strings from
the iSCSI target:

- CHAP_C, where the target challenges the initiator,

- CHAP_R, where the target answers the challenge from the initiator (in
  case the initiator wants mutual authentication).

Accordingly, we have two IScsiHexToBin() call sites:

- At the CHAP_C decoding site, check whether the decoding succeeds. The
  decoded buffer ("AuthData->InChallenge") can accommodate 1024 bytes,
  which is a permissible restriction on the target, per
  <https://tools.ietf.org/html/rfc7143#section-12.1.3>. Shorter challenges
  from the target are acceptable.

- At the CHAP_R decoding site, enforce that the decoding both succeed, and
  provide exactly ISCSI_CHAP_RSP_LEN bytes. CHAP_R contains the digest
  calculated by the target, therefore it must be of fixed size. We may
  only call IScsiCHAPAuthTarget() if "TargetRsp" has been fully populated.

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-11-lersek@redhat.com>
2021-06-09 17:25:03 +00:00
..
ComponentName.c NetworkPkg/IScsiDxe: Fix various typos 2020-02-10 22:30:07 +00:00
IScsiAuthenticationInfo.c NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiCHAP.c NetworkPkg/IScsiDxe: check IScsiHexToBin() return values 2021-06-09 17:25:03 +00:00
IScsiCHAP.h NetworkPkg/IScsiDxe: clean up "ISCSI_CHAP_AUTH_DATA.OutChallengeLength" 2021-06-09 17:25:03 +00:00
IScsiConfig.c NetworkPkg/IScsiDxe: Fix various typos 2020-02-10 22:30:07 +00:00
IScsiConfig.h NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiConfigNVDataStruc.h NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiConfigStrings.uni NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiConfigVfr.vfr NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiDhcp.c NetworkPkg/IScsiDxe: Enhance the check for array boundary 2020-04-26 06:20:32 +00:00
IScsiDhcp.h NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiDhcp6.c NetworkPkg/IScsiDxe: Enhance the check for array boundary 2020-04-26 06:20:32 +00:00
IScsiDhcp6.h NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiDns.c NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiDns.h NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiDriver.c NetworkPkg/IScsiDxe: Fix various typos 2020-02-10 22:30:07 +00:00
IScsiDriver.h NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiDxe.inf NetworkPkg/IScsiDxe: fix potential integer overflow in IScsiBinToHex() 2021-06-09 17:25:03 +00:00
IScsiDxe.uni NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiDxeExtra.uni NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiExtScsiPassThru.c NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiIbft.c NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiIbft.h NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiImpl.h NetworkPkg/IScsiDxe: fix potential integer overflow in IScsiBinToHex() 2021-06-09 17:25:03 +00:00
IScsiInitiatorName.c NetworkPkg: Replace BSD License with BSD+Patent License 2019-04-09 10:58:15 -07:00
IScsiMisc.c NetworkPkg/IScsiDxe: fix IScsiHexToBin() buffer overflow 2021-06-09 17:25:03 +00:00
IScsiMisc.h NetworkPkg/IScsiDxe: fix IScsiHexToBin() buffer overflow 2021-06-09 17:25:03 +00:00
IScsiProto.c NetworkPkg/IScsiDxe: Fix various typos 2020-02-10 22:30:07 +00:00
IScsiProto.h NetworkPkg/IScsiDxe: Fix various typos 2020-02-10 22:30:07 +00:00