Allocate memory for _saraRXBuffer, _pruneBuffer & _saraResponseBacklog in begin

Author: PaulZC

src/SparkFun_u-blox_SARA-R5_Arduino_Library.cpp

@@ -44,16 +44,39 @@ SARA_R5::SARA_R5(int powerPin, int resetPin, uint8_t maxInitDepth)
   _bufferedPollReentrant = false;
   _pollReentrant = false;
   _currentInitDepth = 0;
-
-  memset(_saraRXBuffer, 0, _RXBuffSize);
-  memset(_pruneBuffer, 0, _RXBuffSize);
-  memset(_saraResponseBacklog, 0, _RXBuffSize);
   _saraResponseBacklogLength = 0;
 }
 
 #ifdef SARA_R5_SOFTWARE_SERIAL_ENABLED
 bool SARA_R5::begin(SoftwareSerial &softSerial, unsigned long baud)
 {
+  _saraRXBuffer = new char[_RXBuffSize];
+  if (_saraRXBuffer == NULL)
+  {
+    if (_printDebug == true)
+      _debugPort->println(F("begin: not enough memory for _saraRXBuffer!"));
+    return false;
+  }
+  memset(_saraRXBuffer, 0, _RXBuffSize);
+
+  _pruneBuffer = new char[_RXBuffSize];
+  if (_pruneBuffer == NULL)
+  {
+    if (_printDebug == true)
+      _debugPort->println(F("begin: not enough memory for _pruneBuffer!"));
+    return false;
+  }
+  memset(_pruneBuffer, 0, _RXBuffSize);
+
+  _saraResponseBacklog = new char[_RXBuffSize];
+  if (_saraResponseBacklog == NULL)
+  {
+    if (_printDebug == true)
+      _debugPort->println(F("begin: not enough memory for _saraResponseBacklog!"));
+    return false;
+  }
+  memset(_saraResponseBacklog, 0, _RXBuffSize);
+
   SARA_R5_error_t err;
 
   _softSerial = &softSerial;
@@ -69,6 +92,33 @@ bool SARA_R5::begin(SoftwareSerial &softSerial, unsigned long baud)
 
 bool SARA_R5::begin(HardwareSerial &hardSerial, unsigned long baud)
 {
+  _saraRXBuffer = new char[_RXBuffSize];
+  if (_saraRXBuffer == NULL)
+  {
+    if (_printDebug == true)
+      _debugPort->println(F("begin: not enough memory for _saraRXBuffer!"));
+    return false;
+  }
+  memset(_saraRXBuffer, 0, _RXBuffSize);
+
+  _pruneBuffer = new char[_RXBuffSize];
+  if (_pruneBuffer == NULL)
+  {
+    if (_printDebug == true)
+      _debugPort->println(F("begin: not enough memory for _pruneBuffer!"));
+    return false;
+  }
+  memset(_pruneBuffer, 0, _RXBuffSize);
+
+  _saraResponseBacklog = new char[_RXBuffSize];
+  if (_saraResponseBacklog == NULL)
+  {
+    if (_printDebug == true)
+      _debugPort->println(F("begin: not enough memory for _saraResponseBacklog!"));
+    return false;
+  }
+  memset(_saraResponseBacklog, 0, _RXBuffSize);
+
   SARA_R5_error_t err;
 
   _hardSerial = &hardSerial;

src/SparkFun_u-blox_SARA-R5_Arduino_Library.h

@@ -832,9 +832,9 @@ class SARA_R5 : public Print
 
   #define _RXBuffSize 2056
   const unsigned long _rxWindowMillis = 2; // 1ms is not quite long enough for a single char at 9600 baud. millis roll over much less often than micros.
-  char _saraRXBuffer[_RXBuffSize];
-  char _pruneBuffer[_RXBuffSize];
-  char _saraResponseBacklog[_RXBuffSize];
+  char *_saraRXBuffer; // Allocated in SARA_R5::begin
+  char *_pruneBuffer;
+  char *_saraResponseBacklog;
   int _saraResponseBacklogLength = 0; // The backlog could contain binary data so we can't use strlen to find its length
 
   void (*_socketListenCallback)(int, IPAddress, unsigned int, int, IPAddress, unsigned int);