7u83/stcgal
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

protocols: fix off-by-one in payload extraction

Browse Source 
Drop all of the checksum bytes. This didn't take the end-of-frame marker
into account.
This commit is contained in:
Grigori Goronzy 2021-01-03 16:10:05 +01:00
parent b46d81a184
commit 0e020f2aa4
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
stcgal/protocols.py
View File

@ -417,7 +417,7 @@ class Stc89Protocol(StcBaseProtocol):
raise StcFramingException("packet checksum mismatch")
payload = StcBaseProtocol.extract_payload(self, packet)
return payload[:-1]
return payload[:-2]
def write_packet(self, packet_data):
"""Send packet to MCU.
@ -812,7 +812,7 @@ class Stc12BaseProtocol(StcBaseProtocol):
raise StcFramingException("packet checksum mismatch")
payload = StcBaseProtocol.extract_payload(self, packet)
return payload[:-2]
return payload[:-3]
def write_packet(self, packet_data):
"""Send packet to MCU.