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

[Core] Fixing full file system access security vulnerability #2950

Closed
wants to merge 1 commit into from
Closed

[Core] Fixing full file system access security vulnerability #2950

wants to merge 1 commit into from

Conversation

Jkat
Copy link
Contributor

@Jkat Jkat commented Jul 13, 2017

learn2php

@Jkat Jkat added Release: Add to release notes PR whose changes should be highlighted in the release notes Category: Bug PR or issue that aims to report or fix a bug labels Jul 13, 2017
@Jkat Jkat added this to the 17.0 milestone Jul 13, 2017
@Jkat Jkat modified the milestones: 17.1, 17.0 Jul 13, 2017
@johnsaigle johnsaigle mentioned this pull request Jul 17, 2017
Merged
@johnsaigle
Copy link
Contributor

johnsaigle commented Jul 17, 2017 •
edited
Loading

Think this should for sure go to 17.0 along with #2953

johnsaigle
johnsaigle reviewed Jul 17, 2017
View reviewed changes
Copy link
Contributor

@johnsaigle johnsaigle left a comment

Choose a reason for hiding this comment

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

Think this should for sure go to 17.0 along with #2953.

Also @Jkat how is this different from #2948? Should that one be closed?

@Jkat
Copy link
Contributor Author

Jkat commented Jul 17, 2017

It's the same, I just issued this fix for both 17.1 and 17.0

@driusan
Copy link
Collaborator

driusan commented Jul 18, 2017

The 17.0-dev branch gets forward ported to 17.1-dev after each 17.0.x release, so there's no need for 2 PRs.

@driusan driusan closed this Jul 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xlecours xlecours xlecours approved these changes

@johnsaigle johnsaigle johnsaigle approved these changes

Assignees
No one assigned
Labels
Category: Bug PR or issue that aims to report or fix a bug Release: Add to release notes PR whose changes should be highlighted in the release notes
Projects
None yet
Milestone
17.1
Development

Successfully merging this pull request may close these issues.
4 participants
@Jkat @johnsaigle @driusan @xlecours