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

Minor fix for xdebug_unauth_exec #12017

Merged
merged 1 commit into from
Jun 26, 2019
Merged

Minor fix for xdebug_unauth_exec #12017

merged 1 commit into from
Jun 26, 2019

Conversation

aushack
Copy link
Contributor

@aushack aushack commented Jun 26, 2019

Avoid triggering error where res.headers may not exist.

@aushack
Minor fix for xdebug_unauth_exec
8d6f36e 
Avoid triggering error where res.headers may not exist.
@wvu
Copy link
Contributor

wvu commented Jun 26, 2019

Legit. I thought this was similar to #12008 (comment) at first, but it's not, since res isn't checked beforehand. Good fix! Would get converted to empty string in the interpolation if nil, anyway.

@wvu wvu self-assigned this Jun 26, 2019
@wvu wvu merged commit 8d6f36e into rapid7:master Jun 26, 2019
wvu added a commit that referenced this pull request Jun 26, 2019
@wvu
Land #12017, xdebug_unauth_exec nil fix
61f09d0
@wvu
Copy link
Contributor

wvu commented Jun 26, 2019 •
edited by tdoan-r7
Loading

Release Notes

This fix resolves a nil bug in exploit/unix/http/xdebug_unauth_exec.

msjenkins-r7 pushed a commit that referenced this pull request Jun 26, 2019
@wvu @msjenkins-r7
Land #12017, xdebug_unauth_exec nil fix
e55ed68
@tdoan-r7 tdoan-r7 added the rn-fix release notes fix label Jul 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@wvu wvu

Labels
bug module rn-fix release notes fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@aushack @wvu @tdoan-r7