Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

KRNL-5820: Accept ulimit with -H parameter also #1364

Merged
merged 1 commit into from
May 16, 2024
Merged

KRNL-5820: Accept ulimit with -H parameter also #1364

merged 1 commit into from
May 16, 2024

Conversation

pyllyukko
Copy link
Contributor 

ulimit: ulimit [-SHabcdefiklmnpqrstuvxPT] [limit]

@mboelen
Copy link
Member

mboelen commented May 14, 2024

Trying to understand this one. Wouldn't this alter the check or mix things up between multiple parameters?

@pyllyukko
Copy link
Contributor Author

Trying to understand this one. Wouldn't this alter the check or mix things up between multiple parameters?

So the modification would allow (explicit) ulimit -Hc ("use the `hard' resource limit") instead of only ulimit -c.

@mboelen mboelen merged commit 672d1fc into CISOfy:master May 16, 2024
@mboelen mboelen self-assigned this May 16, 2024
@mboelen
Copy link
Member

mboelen commented May 16, 2024

Thanks, merged.

@pyllyukko pyllyukko deleted the ulimit-fix branch May 16, 2024 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mboelen mboelen

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@pyllyukko @mboelen