fix crash when printing to pclm supporting printers

Account for the margin when copying data. Prevents
writing to memory that has not been allocated.

Bug: 187662332
Test: Print to a printer that supports pclm
Signed-off-by: Kurtis Schlepp <mopriadevteam@gmail.com>
Change-Id: Idb703f65f6641ddc3a477c3077c53c6c6f61c30b

1 files changed, 2 insertions, 3 deletions

diff --git a/jni/plugins/genPCLm/src/genPCLm.cpp b/jni/plugins/genPCLm/src/genPCLm.cpp
index 0aab701..a9a6ac4 100644
--- a/jni/plugins/genPCLm/src/genPCLm.cpp
+++ b/jni/plugins/genPCLm/src/genPCLm.cpp
@@ -60,9 +60,8 @@ static ubyte *shiftStripByLeftMargin(ubyte *ptrToStrip, sint32 currSourceWidth,
 
     if (destColorSpace == grayScale) {
         scanLineWidth = currMediaWidth;
-
         // Allocate a full strip
-        newStrip = (ubyte *) malloc(scanLineWidth * currStripHeight);
+        newStrip = (ubyte *) malloc((scanLineWidth * currStripHeight) + leftMargin);
         memset(newStrip, 0xff, scanLineWidth * currStripHeight);
         for (int i = 0; i < numLinesThisCall; i++) {
             toPtr = newStrip + leftMargin + (i * currMediaWidth);
@@ -73,7 +72,7 @@ static ubyte *shiftStripByLeftMargin(ubyte *ptrToStrip, sint32 currSourceWidth,
         scanLineWidth = currMediaWidth * 3;
         sint32 srcScanlineWidth = currSourceWidth * 3;
         sint32 shiftAmount = leftMargin * 3;
-        newStrip = (ubyte *) malloc(scanLineWidth * currStripHeight);
+        newStrip = (ubyte *) malloc((scanLineWidth * currStripHeight) + shiftAmount);
         memset(newStrip, 0xff, scanLineWidth * currStripHeight);
         for (int i = 0; i < numLinesThisCall; i++) {
             toPtr = newStrip + shiftAmount + (i * scanLineWidth);