Reviewed-by: Ryszard Knop <ryszard.k...@linux.intel.com>

On Tue, 2018-11-06 at 18:58 +0100, ard.biesheuvela wrote:
> E1000MemCopy () takes UINT8 pointers not INT8 pointers, so cast the
> arguments to the correct type.
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel at linaro.org>
> ---
>  IntelUndiPkg/GigUndiDxe/e1000.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/IntelUndiPkg/GigUndiDxe/e1000.c
> b/IntelUndiPkg/GigUndiDxe/e1000.c
> index 1f08a5d67b2d..4c9a06b8cf73 100644
> --- a/IntelUndiPkg/GigUndiDxe/e1000.c
> +++ b/IntelUndiPkg/GigUndiDxe/e1000.c
> @@ -674,8 +674,8 @@ E1000Receive (
>  
>        // Copy the packet from our list to the EFI buffer.
>        E1000MemCopy (
> -        (INT8 *) (UINTN) CpbReceive->BufferAddr,
> -        (INT8 *) (UINTN) ReceiveDescriptor->buffer_addr,
> +        (UINT8 *) (UINTN) CpbReceive->BufferAddr,
> +        (UINT8 *) (UINTN) ReceiveDescriptor->buffer_addr,
>          TempLen
>        );
>  

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to