InfiniTimeOrg / InfiniTimeOrg/InfiniTime

Restrict DFU updates to a specific SoftDevice version for safety

Open
#725 10 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
C
Stars
3.4k
Forks
1.1k
PR merge metrics
No merged PRs in 30d

Description

  • I searched for similar bug reports and found none was relevant.
What happened?

some users brick their devices by installing the DFU ZIP files of InfiniTime

What should happen instead?

FW metadata in ZIP should disallow

Reproduction steps

Currently DFU zip is produced without --sd-req option: https://github.com/InfiniTimeOrg/InfiniTime/blob/develop/src/CMakeLists.txt#L903
This makes dfu.zip harmful for some bootloaders used on nRF52 devices.

A simple solution here is to add --sd-req with pseudo SD code.

More details?

No response

Version

HEAD

Companion app

No response

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at src/CMakeLists.txt around line 903 and inspect how the DFU ZIP is generated and how SoftDevice metadata is currently handled. Confirm the appropriate pseudo SoftDevice requirement from the surrounding build configuration or project history, then verify that generated ZIP metadata restricts installation to compatible nRF52 bootloaders.

Written by the indexing model from the issue text.

Assessment

Tech stack
cmake
Domain
build-system, embedded-iot
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.