Changeset 196 in flair-src for trunk/lib/FlairSensorActuator/src/NeoM8N.cpp

Timestamp:

Sep 12, 2017, 11:26:33 AM (8 years ago)

Author:

Sanahuja Guillaume

Message:

avoid buffer overrun

File:

• trunk/lib/FlairSensorActuator/src/NeoM8N.cpp (modified) (3 diffs)

trunk/lib/FlairSensorActuator/src/NeoM8N.cpp

@@ -42 +42 @@
 
 void NeoM8N::Run(void) {
-  char response[200] = {0};
-  int size,written;
-
-  //turn off all mesages
+  
+  //turn off all messages
   {
     char tx[]={0x06, 0x01, 0x03, 0, 0xf0, 0,0};
@@ -82 +80 @@
   
   while (!ToBeStopped()) {
-    //SleepMS(10);??
-    size = 0;
+    char response[1024] = {0};
+    int size = 0;
     while (!ToBeStopped()) {
       ssize_t read = serialport->Read(&response[size], 1);
@@ -90 +88 @@
       }
 
-      if (response[size] == 0x0a)
-        break;
+      if (response[size] == 0x0a ) break;
       size++;
+      if (size==sizeof(response)) break;
     }
-    size++;
-    parseFrame(response, size);
+    if (size!=sizeof(response)) {
+      size++;
+      parseFrame(response, size);
+    } else {
+      Thread::Warn("frame too long for buffer\n");
+    }
   }
   /** fin running loop **/