From 96301aadc199e3dcd4dec39a8e6257862d1f92c5 Mon Sep 17 00:00:00 2001 From: Gris87 Date: Mon, 16 Sep 2019 22:21:47 +0800 Subject: [PATCH] MdeModulePkg: Add missing sourceX for Blt There is no sourceX offset in case when Configure->PixelFormat == PixelBlueGreenRedReserved8BitPerColor. We are copying most left pixels instead of copying required rectangle. Signed-off-by: Gris87 Cc: Ruiyu Ni Tested-by: Zhichao Gao Reviewed-by: Zhichao Gao --- MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c b/MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c index 47c5326e99..ff7979c962 100644 --- a/MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c +++ b/MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c @@ -504,7 +504,7 @@ FrameBufferBltLibBufferToVideo ( Destination = Configure->FrameBuffer + Offset; if (Configure->PixelFormat == PixelBlueGreenRedReserved8BitPerColor) { - Source = (UINT8 *) BltBuffer + (SrcY * Delta); + Source = (UINT8 *) BltBuffer + (SrcY * Delta) + SourceX * sizeof (EFI_GRAPHICS_OUTPUT_BLT_PIXEL); } else { for (IndexX = 0; IndexX < Width; IndexX++) { Blt =