Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[Process] Returnfalse whenopen_basedir prevents access to/dev/tty#54863

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.

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

Already on GitHub?Sign in to your account

Merged
xabbuh merged 1 commit intosymfony:6.4frommjauvin:6.4
May 15, 2024
Merged

Conversation

@mjauvin
Copy link
Contributor

@mjauvinmjauvin commentedMay 7, 2024•
edited
Loading

QA
Branch?6.4
Bug fix?yes
New feature?no
Deprecations?no

If open_basedir restrictions are in effect, checking if the file /dev/tty is writable will prevent setting tty mode on the process, and avoid failing to create a Process.

Fixes#54874

jaxwilko reacted with thumbs up emoji
…nt access to /dev/tty.If open_basedir restrictions are in effect, checking if the file /dev/tty is writable will prevent setting tty mode on the process, and avoid failing to create a Process.
@xabbuh
Copy link
Member

Symfony 5.4 would suffer from this too, right?

@carsonbotcarsonbot changed the titleReturn false when open_basedir prevents access to /dev/tty.[Process] Return false when open_basedir prevents access to /dev/tty.May 7, 2024
@mjauvin
Copy link
ContributorAuthor

Symfony 5.4 would suffer from this too, right?

I suppose, yes.

@OskarStarkOskarStark changed the title[Process] Return false when open_basedir prevents access to /dev/tty.[Process] Returnfalse whenopen_basedir prevents access to/dev/ttyMay 7, 2024
@OskarStarkOskarStark modified the milestones:6.4,5.4May 7, 2024
Copy link
Member

@chalasrchalasr left a comment

Choose a reason for hiding this comment

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

On 5.4

@xabbuh
Copy link
Member

looks like this only affects 6.4+ because of the changes done in#46971

@xabbuhxabbuh modified the milestones:5.4,6.4May 15, 2024
@xabbuh
Copy link
Member

Thank you@mjauvin.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@chalasrchalasrchalasr approved these changes

Assignees

No one assigned

Projects

None yet

Milestone

6.4

Development

Successfully merging this pull request may close these issues.

5 participants

@mjauvin@xabbuh@chalasr@OskarStark@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp