Skip to content

tests/int: add hooks argv[0] test#4593

Merged
cyphar merged 1 commit into
opencontainers:mainfrom
kolyshkin:hooks-argv0-test
Feb 25, 2025
Merged

tests/int: add hooks argv[0] test#4593
cyphar merged 1 commit into
opencontainers:mainfrom
kolyshkin:hooks-argv0-test

Conversation

@kolyshkin

Copy link
Copy Markdown
Contributor

Looking into old opened runc issues, I noticed #1663 is there without any resolution, and wrote this simple test checking if we mangle hook's argv[0] in any way.

Apparently we're good, but the test actually makes sense to have.

@kolyshkin

Copy link
Copy Markdown
Contributor Author

ci / cross-i386 failure is unrelated (being fixed by #4595)

@kolyshkin
kolyshkin marked this pull request as ready for review January 16, 2025 20:29

@rata rata left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Looking into old opened runc issues, I noticed opencontainers#1663 is there without
any resolution, and wrote this simple test checking if we mangle hook's
argv[0] in any way.

Apparently we're good, but the test actually makes sense to have.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>

@cyphar cyphar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@cyphar
cyphar merged commit 58a599d into opencontainers:main Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants