Skip to content

[Spec Impl] Add checksum to all writes #41

@krishnakumar4a4

Description

@krishnakumar4a4

Background

Every write from/to Wallet has a possibility to have corruption. Hence all writes should include a checksum at the last.

What to do

  1. Modify all the structures chunk write, transfer report request, transfer summary report to include checksum at the end.
  2. Finalise on the CRC to be used. The suggestion is to use CRC16-CCITT-True which is used in Bluetooth 5.3
  3. Update Transfer report structure to match with the specification. Also investigate if the mentioned structure is okay for a success and failure case. Since it will have CRC, on a success case, it will still have CRC value to differentiate between a success report Vs a spurious empty characteristic read.

References for finalising the CRC

https://reveng.sourceforge.io/crc-catalogue/16.htm

Metadata

Metadata

Labels

enhancementNew feature or request

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions