From 5ff1e144b27dee487c8820a2d98d0befdded6298 Mon Sep 17 00:00:00 2001
From: patacongo <patacongo@42af7a65-404d-4744-a932-0658087f49c3>
Date: Tue, 19 Apr 2011 01:21:55 +0000
Subject: [PATCH] Minor fix to OLED logic
git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@3524 42af7a65-404d-4744-a932-0658087f49c3
---
drivers/lcd/ug-9664hswag01.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/lcd/ug-9664hswag01.c b/drivers/lcd/ug-9664hswag01.c
index b68aef3b8b..bb49f20e6b 100755
--- a/drivers/lcd/ug-9664hswag01.c
+++ b/drivers/lcd/ug-9664hswag01.c
@@ -296,7 +296,7 @@ static inline void up_clear(FAR struct ug_dev_s *priv);
* if there are multiple LCD devices, they must each have unique run buffers.
*/
-static uint8_t g_runbuffer[UG_XSTRIDE];
+static uint8_t g_runbuffer[UG_XSTRIDE+1];
/* This structure describes the overall LCD video controller */
@@ -701,12 +701,16 @@ static int ug_getrun(fb_coord_t row, fb_coord_t col, FAR uint8_t *buffer,
*buffer |= usrmask;
}
- /* Inc/Decrement to the next destination pixel */
+ /* Inc/Decrement to the next destination pixel. Hmmmm. It looks like
+ * this logic could write past the end of the user buffer. Revisit
+ * this!
+ */
#ifdef CONFIG_NX_PACKEDMSFIRST
if (usrmask == LS_BIT)
{
buffer++;
+ *buffer = 0;
usrmask = MS_BIT;
}
else
--
GitLab