]> sigrok.org Git - libsigrokdecode.git/commit
eeprom24xx: avoid access to caller's data after validity ends
authorGerhard Sittig <redacted>
Fri, 14 Jul 2023 06:12:13 +0000 (08:12 +0200)
committerGerhard Sittig <redacted>
Tue, 18 Jul 2023 19:09:40 +0000 (21:09 +0200)
commit42fb0f33ff23c1633324bde8b3cf400493237d97
tree07946a0f166bef048600c97e4274da39c79c6464
parentbb6f9c500e4119aa9558dfc66ba82bf7ef3772c5
eeprom24xx: avoid access to caller's data after validity ends

The at24 EEPROM decoder's previous implementation happened to access
caller's data even after the .decode() method invocation ended, and
their content has changed or the data was not valid any longer.

Get deep copies for those details which broke the test suite. Prepare
"generous" deep copies for other data which currently doesn't trigger
exceptions, but might be waiting for an accident to happen. Careful
inspection of the complex implementation and relaxing the current greed
of this commit remains for future commits. Comment heavily for awareness.

It is assumed that the 'databyte' name is misleading. And that much of
this upper layer decoder's complexity would be obsoleted by the lower
layer decoder's providing more useful packets (bytes and their ACK state,
read/write phases of transfers, complete transfers up to STOP, etc).
This commit does not address those readability or layering concerns.
decoders/eeprom24xx/pd.py