Fix for pymodbus-dev#356

Reset frame when checkframe fails as it is impossible to know what is a valid start packet if CRC or mid frame is in buffer.
pazzarpj · Dec 3, 2018 · 77abfa9 · 77abfa9
1 parent f0edef3
commit 77abfa9
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/pymodbus/framer/rtu_framer.py b/pymodbus/framer/rtu_framer.py
@@ -217,10 +217,11 @@ def processIncomingPacket(self, data, callback, unit, **kwargs):
             if self.checkFrame():
                 if self._validate_unit_id(unit, single):
                     self._process(callback)
+                    return
                 else:
                     _logger.debug("Not a valid unit id - {}, "
                                   "ignoring!!".format(self._header['uid']))
-                    self.resetFrame()
+                self.resetFrame()
         else:
             _logger.debug("Frame - [{}] not ready".format(data))