Fixing pending write cache management

quartiq / minimq

Minimal no_std MQTT v5.0 client implementation

MIT License

143 stars 16 forks source link

Fixing pending write cache management #149

Closed ryan-summers closed 1 year ago

ryan-summers commented 1 year ago

This PR fixes an issue I encountered with pending writes when integrating miniconf/minimq onto Stabilizer.

The cached pending write needs to use the start and len parameters when refreshing the cache indices, otherwise we accidentally cause malformed packets due to incorrect indices.

ryan-summers commented 1 year ago

Very potentially. I had noticed that as well.