Skip to content

Release 2.9 1.6 - #237

Closed
Junior Martinez (jmartinez-silabs) wants to merge 10 commits into
mainfrom
release_2.9-1.6
Closed

Release 2.9 1.6#237
Junior Martinez (jmartinez-silabs) wants to merge 10 commits into
mainfrom
release_2.9-1.6

Conversation

@jmartinez-silabs

Copy link
Copy Markdown
Contributor

Problem / Feature

What is being fixed? What is the feature being added? Examples:

  • Fix crash on startup
  • Fixes #12345 Frobnozzle is leaky (exactly like that, so GitHub will auto-close the issue).

Change overview

What's in this PR

Testing

How was this tested?

  • If unit tests were added, how do they cover this issue?
  • If unit tests existed, how were they fixed/modified to prevent this in future?
  • If new unit tests are not added, why not?
  • If integration tests were added, how do they verify this change?
  • If new integration tests are not added, why not?
  • If manually tested, what platforms controller and device platforms were manually tested, and how?
  • If no testing is required, why not?

silabs-matter-ci-bot Bot and others added 10 commits April 17, 2026 12:32
Co-authored-by: silabs-matter-ci-bot[bot] <silabs-matter-ci-bot[bot]@users.noreply.github.com>
Update provisioning libraries and tooling

Co-authored-by: silabs-matter-ci-bot[bot] <silabs-matter-ci-bot[bot]@users.noreply.github.com>
Update provisioning libraries and tooling

Co-authored-by: silabs-matter-ci-bot[bot] <silabs-matter-ci-bot[bot]@users.noreply.github.com>
Update provisioning libraries and tooling

Co-authored-by: silabs-matter-ci-bot[bot] <silabs-matter-ci-bot[bot]@users.noreply.github.com>
Update provisioning libraries and tooling

Co-authored-by: silabs-matter-ci-bot[bot] <silabs-matter-ci-bot[bot]@users.noreply.github.com>
Update provisioning libraries and tooling

Co-authored-by: silabs-matter-ci-bot[bot] <silabs-matter-ci-bot[bot]@users.noreply.github.com>
Update provisioning libraries and tooling

Co-authored-by: silabs-matter-ci-bot[bot] <silabs-matter-ci-bot[bot]@users.noreply.github.com>
Update provisioning libraries and tooling

Co-authored-by: silabs-matter-ci-bot[bot] <silabs-matter-ci-bot[bot]@users.noreply.github.com>
@cursor

cursor Bot commented Jul 20, 2026

Copy link
Copy Markdown

PR Summary

High Risk
Changes manufacturing-date handling, attestation CSR subject limits, and Series 3 credentials flash layout—errors can break factory provisioning or device attestation for new builds.

Overview
Adds provisioning release 2.9 aligned with release_2.9-1.6, including new per-chip *_mfd generator firmware entries in devices.yaml and a 2.9 row in versions.yaml documenting YYYYMMDD manufacturing dates and a Series 3 credentials base address change to avoid manufacturing-token collisions.

Firmware/storage API updates support that behavior: ProvisionStorage tightens kCommonNameMax to 64 (RFC 5280) and adds GetManufacturingDateSuffix alongside the existing date getters; AttestationKey documents a fixed kSubjectNameLengthMax buffer for CSR subjects that include Matter VID/PID OIDs. bluet.py now resolves modules.util via client/python on sys.path.

The PR also introduces bundled provision config (parameters.yaml, OpenSSL csa_openssl.cnf with Matter VID OID, silabs_creds.tmpl) used by the provisioning toolchain.

Reviewed by Cursor Bugbot for commit 4df04a1. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

Reviewed by Cursor Bugbot for commit 4df04a1. Configure here.

static constexpr size_t kCsrLengthMax = 512;
static constexpr size_t kCommonNameMax = 128;
// X.509 (RFC 5280, Appendix A.1): CommonName attribute value is limited to 64 characters
static constexpr size_t kCommonNameMax = 64;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Common name length mismatch

Medium Severity

kCommonNameMax is reduced to 64 on-device, but common_name in the added parameters config still has no max, so the host can accept names longer than storage and CSR subject limits allow.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 4df04a1. Configure here.

@jmartinez-silabs

Copy link
Copy Markdown
Contributor Author

This was just a test. Only changes coming from 2.9 are coming from provision artifact. They should come from provision repo directly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants