Skip to content

bugfix/sign-check-validation: Added changes for validating both cat and sys files for signatures - #120

Merged
Rohit Mishra (rohimish-qc) merged 1 commit into
qualcomm:developfrom
rohimish-qc:bugfix/sign-check-validation
Sep 8, 2026
Merged

Rohit Mishra (rohimish-qc) merged 1 commit into
qualcomm:developfrom
rohimish-qc:bugfix/sign-check-validation

Conversation

@rohimish-qc

@rohimish-qc Rohit Mishra (rohimish-qc) commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Description

Added changes for validating both cat and sys files for signatures

Fixes #

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Hotfix (urgent fix targeted at a release/x.y branch)
  • Refactor (no functional change)
  • Performance improvement
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Test-only change
  • CI / build-pipeline change

How has this been tested?

The script ran on the generated attested files, it validated the signatures of the sys and cat files.

Case 1: When signatures were not valid
image

Case 2: When signatures were valid
image

Checklist

  • All required CI checks are green on the latest commit of this PR
  • Build succeeds following the steps in the README / src/linux/README.md
  • My code follows the Code Style Guidelines of this project
  • My branch follows the naming convention: <branch-prefix>/<area>/<description>
  • PR title follows the Conventional Commits format with our full-word types (feature / bugfix / hotfix / docs)
  • I have performed a self-review of my own code
  • I have added code comments in areas that are complex or hard to understand
  • My changes generate no new compiler warnings
  • I have added tests for my fix or feature (or noted why tests are not feasible)
  • New and existing tests pass locally with my changes
  • My branch is rebased onto the latest develop (or release/x.y for hotfix) - no merge commits
  • Every commit has Signed-off-by: (DCO) - see CONTRIBUTING.md
  • I have linked the relevant issue if any (Fixes #...)
  • Any dependent changes have been merged and published in downstream modules

…nd sys files for signatures

Signed-off-by: rohimish <rohimish@qti.qualcomm.com>
@rohimish-qc
Rohit Mishra (rohimish-qc) merged commit af0dd5a into qualcomm:develop Sep 8, 2026
54 checks passed
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