Re: [PATCH 1/1] SecurityPkg: Add missing break in Tpm2TestParms

["Yao, Jiewen" <jiewen.yao@intel.com>]

Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>

Can we add this in 202305 stable tag?


> -----Original Message-----
> From: Rebecca Cran <rebecca@bsdio.com>
> Sent: Wednesday, May 10, 2023 12:45 AM
> To: devel@edk2.groups.io; Yao, Jiewen <jiewen.yao@intel.com>; Wang,
> Jian J <jian.j.wang@intel.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>
> Cc: Rebecca Cran <rebecca@bsdio.com>
> Subject: [PATCH 1/1] SecurityPkg: Add missing break in Tpm2TestParms
> 
> Add missing break in Tpm2TestParms.
> 
> Reported in https://github.com/tianocore/edk2/issues/4073
> 
> Signed-off-by: Rebecca Cran <rebecca@bsdio.com>
> ---
>  SecurityPkg/Library/Tpm2CommandLib/Tpm2Capability.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/SecurityPkg/Library/Tpm2CommandLib/Tpm2Capability.c
> b/SecurityPkg/Library/Tpm2CommandLib/Tpm2Capability.c
> index 9db301ec9a90..f0e6019a47be 100644
> --- a/SecurityPkg/Library/Tpm2CommandLib/Tpm2Capability.c
> +++ b/SecurityPkg/Library/Tpm2CommandLib/Tpm2Capability.c
> @@ -743,6 +743,7 @@ Tpm2TestParms (
>            return EFI_INVALID_PARAMETER;
>        }
> 
> +      break;
>      case TPM_ALG_SYMCIPHER:
>        WriteUnaligned16 ((UINT16 *)Buffer, SwapBytes16 (Parameters-
> >parameters.symDetail.algorithm));
>        Buffer += sizeof (UINT16);
> --
> 2.39.2