Thou shalt not trust Adafruit GFX

Especially when using the co-ordinates it generates as a means to access memory.
mrcodetastic · Aug 14, 2020 · db198d7 · db198d7
1 parent b504f62
commit db198d7
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/ESP32-RGB64x32MatrixPanel-I2S-DMA.cpp b/ESP32-RGB64x32MatrixPanel-I2S-DMA.cpp
@@ -442,7 +442,7 @@ void RGB64x32MatrixPanel_I2S_DMA::updateMatrixDMABuffer(int16_t x_coord, int16_t
 
     // Check that the co-ordinates are within range, or it'll break everything big time.
     // Valid co-ordinates are from 0 to (MATRIX_XXXX-1)
-	if ( x_coord >= MATRIX_WIDTH || y_coord >= MATRIX_HEIGHT) {
+	if ( x_coord < 0 || y_coord < 0 || x_coord >= MATRIX_WIDTH || y_coord >= MATRIX_HEIGHT) {
 		  return;
     }
 

diff --git a/library.json b/library.json
@@ -10,7 +10,7 @@
     "name": "Faptastic",
     "url": "https://github.com/mrfaptastic/"
   },
-  "version": "1.2.1",
+  "version": "1.2.2",
   "frameworks": "arduino",
   "platforms": "esp32",
   "examples": [

diff --git a/library.properties b/library.properties
@@ -1,5 +1,5 @@
 name=ESP32 64x32 LED MATRIX HUB75 DMA Display
-version=1.2.1
+version=1.2.2
 author=Faptastic
 maintainer=Faptastic 
 sentence=Experimental DMA based LED Matrix HUB75 Library