Skip to content

Conversation

@tjchadaga
Copy link
Contributor

Why I did it

To pick up latest fixes and enhancements

Work item tracking
  • Microsoft ADO (number only): 36110346

How I did it

Updated sai-dnx.mk

How to verify it

Image bringup and basic sanity on dnx device

Which release branch to backport (provide reason below if selected)

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Tejaswini Chadaga <tchadaga@microsoft.com>
Copilot AI review requested due to automatic review settings January 8, 2026 22:33
@tjchadaga tjchadaga requested a review from lguohan as a code owner January 8, 2026 22:33
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@tjchadaga tjchadaga requested a review from vmittal-msft January 8, 2026 22:33
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Broadcom DNX SAI library version from 14.1.0.1.0.0.0.0 to 14.1.0.1.0.0.0.1 to pick up the latest fixes and enhancements from the vendor.

Key changes:

  • Incremental version bump of the DNX SAI library (last octet incremented by 1)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

LIBSAIBCM_DNX_VERSION = 14.1.0.1.0.0.0.1
LIBSAIBCM_DNX_BRANCH_NAME = SAI_14.1.0_GA

LIBSAIBCM_DNX_URL_PREFIX = "https://packages.trafficmanager.net/public/sai/sai-broadcom/$(LIBSAIBCM_DNX_BRANCH_NAME)/$(LIBSAIBCM_DNX_VERSION)/dnx"
Copy link

Copilot AI Jan 8, 2026

Choose a reason for hiding this comment

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

LIBSAIBCM_DNX_URL_PREFIX points to a remote .deb host that is consumed via $(BRCM_DNX_SAI)_URL and SONIC_ONLINE_DEBS, which the generic build logic fetches with curl and installs without any checksum or signature verification. If packages.trafficmanager.net (or its TLS/DNS) is compromised, an attacker could serve a malicious libsaibcm_dnx package that is transparently pulled into SONiC images and executed with full privileges during build and at runtime. To harden the supply chain, ensure this downloaded package is protected by strong integrity verification (for example, pinned content hashes or verified signatures) or is sourced from a repository/mechanism that enforces such verification.

Copilot uses AI. Check for mistakes.
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