Added "node id" test

If a float is passed as a node id it can fail. added int 0 to 31 check to first value in payload. http://openenergymonitor.org/emon/node/6048
emilm · Dec 9, 2014 · 366b3db · 366b3db
1 parent 30bd975
commit 366b3db
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/emonhub_interfacer.py b/src/emonhub_interfacer.py
@@ -147,6 +147,11 @@ def _validate_frame(self, ref, received):
         except Exception:
             self._log.warning(str(ref) + " Discarded RX frame 'non-numerical content' : " + str(received))
             return False
+
+        # Discard if first value is not a valid node id
+        if received[0] % 1 != 0 or received[0] < 0 or received[0] > 31
+            self._log.warning(str(ref) + " Discarded RX frame 'node id outside scope' : " + str(received))
+            return False
 
         # If it passes all the checks return
         return received