Re: [edk2] [PATCH 3/3] MdeModulePkg/FrameBufferBltLib: Fix copying of unaligned memory

[Zeng, Star]

Reviewed-by: Star Zeng 

Thanks,
Star
-Original Message-
From: Ni, Ruiyu 
Sent: Monday, January 15, 2018 11:46 AM
To: edk2-devel@lists.01.org
Cc: Christian Ehrhardt ; Zeng, Star 
Subject: [PATCH 3/3] MdeModulePkg/FrameBufferBltLib: Fix copying of unaligned 
memory

Contributed-under: TianoCore Contribution Agreement 1.1
Reported-by: Christian Ehrhardt 
Signed-off-by: Christian Ehrhardt 
Cc: Star Zeng 
Reviewed-by: Ruiyu Ni 
---
 MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c 
b/MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c
index c88469859b..78dc0c0b51 100644
--- a/MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c
+++ b/MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c
@@ -280,6 +280,7 @@ FrameBufferBltLibVideoFill (
 SizeInBytes = WidthInBytes * Height;
 if (SizeInBytes >= 8) {
   SetMem32 (Destination, SizeInBytes & ~3, (UINT32) WideFill);
+  Destination += SizeInBytes & ~3;
   SizeInBytes &= 3;
 }
 if (SizeInBytes > 0) {
@@ -297,6 +298,7 @@ FrameBufferBltLibVideoFill (
 SizeInBytes = WidthInBytes;
 if (SizeInBytes >= 8) {
   SetMem64 (Destination, SizeInBytes & ~7, WideFill);
+  Destination += SizeInBytes & ~7;
   SizeInBytes &= 7;
 }
 if (SizeInBytes > 0) {
-- 
2.15.1.windows.2

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

[edk2] [PATCH 3/3] MdeModulePkg/FrameBufferBltLib: Fix copying of unaligned memory

[Ruiyu Ni]

Contributed-under: TianoCore Contribution Agreement 1.1
Reported-by: Christian Ehrhardt 
Signed-off-by: Christian Ehrhardt 
Cc: Star Zeng 
Reviewed-by: Ruiyu Ni 
---
 MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c 
b/MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c
index c88469859b..78dc0c0b51 100644
--- a/MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c
+++ b/MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.c
@@ -280,6 +280,7 @@ FrameBufferBltLibVideoFill (
 SizeInBytes = WidthInBytes * Height;
 if (SizeInBytes >= 8) {
   SetMem32 (Destination, SizeInBytes & ~3, (UINT32) WideFill);
+  Destination += SizeInBytes & ~3;
   SizeInBytes &= 3;
 }
 if (SizeInBytes > 0) {
@@ -297,6 +298,7 @@ FrameBufferBltLibVideoFill (
 SizeInBytes = WidthInBytes;
 if (SizeInBytes >= 8) {
   SetMem64 (Destination, SizeInBytes & ~7, WideFill);
+  Destination += SizeInBytes & ~7;
   SizeInBytes &= 7;
 }
 if (SizeInBytes > 0) {
-- 
2.15.1.windows.2

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