Skip to content

hardened_std: implement Command::* functions#100

Merged
zvonkok merged 1 commit intoNVIDIA:mainfrom
zvonkok:process
Jan 12, 2026
Merged

hardened_std: implement Command::* functions#100
zvonkok merged 1 commit intoNVIDIA:mainfrom
zvonkok:process

Conversation

@zvonkok
Copy link
Copy Markdown
Collaborator

@zvonkok zvonkok commented Jan 12, 2026

Used for executing nvidia-ctk, nvidia-smi, and other helpers.

Copilot AI review requested due to automatic review settings January 12, 2026 03:56
Copy link
Copy Markdown
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 implements process execution functionality for the hardened_std library, enabling the init process to execute whitelisted binaries like nvidia-smi, nvidia-ctk, and other GPU-related utilities. The implementation follows a security-hardened model with compile-time binary whitelisting and static string constraints.

Changes:

  • Implemented Command builder pattern with spawn(), status(), and exec() methods
  • Added Child process management with wait() and kill() functionality
  • Created binary whitelist enforcement with separation of production and test binaries
  • Implemented Stdio redirection supporting Null, Inherit, Piped, and Fd variants

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

Copy link
Copy Markdown
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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 13 comments.


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

@zvonkok zvonkok changed the title hardened_std: implement Command::* fucntions hardened_std: implement Command::* functions Jan 12, 2026
Signed-off-by: Zvonko Kaiser <zkaiser@nvidia.com>
@zvonkok zvonkok merged commit 1fb15ce into NVIDIA:main Jan 12, 2026
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Ok to test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants