Skip to content

Introduce restricted mode bash in building#103

Merged
HarithaIBM merged 5 commits intozopencommunity:mainfrom
sabi789:patch-1
Apr 8, 2026
Merged

Introduce restricted mode bash in building#103
HarithaIBM merged 5 commits intozopencommunity:mainfrom
sabi789:patch-1

Conversation

@sabi789
Copy link
Copy Markdown
Member

@sabi789 sabi789 commented Apr 6, 2026

Enable rbash restricted mode

Enable rbash restricted mode

Signed-off-by: sabi789 <sabithac6298@gmail.com>
sabi789 added 3 commits April 7, 2026 12:28
Signed-off-by: sabi789 <sabithac6298@gmail.com>
Signed-off-by: sabi789 <sabithac6298@gmail.com>
Comment thread buildenv Outdated

zopen_post_install() {
echo "enable rbash"
ln -s "$1/bin/bash" "$1/bin/rbash"
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.

I think we need to change this to:

ln -s bash "$1/bin/rbash"

Otherwise rbash will point to an absolute path

Signed-off-by: sabi789 <sabithac6298@gmail.com>
Copy link
Copy Markdown
Member

@IgorTodorovskiIBM IgorTodorovskiIBM left a comment

Choose a reason for hiding this comment

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

LGTM

@HarithaIBM
Copy link
Copy Markdown
Member

can you create a task to add test case for this ?

Copy link
Copy Markdown
Member

@HarithaIBM HarithaIBM left a comment

Choose a reason for hiding this comment

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

LGTM

@HarithaIBM HarithaIBM merged commit 1440cd5 into zopencommunity:main Apr 8, 2026
1 of 2 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.

3 participants