Skip to content

Fix volumes on host#3184

Merged
vieux merged 2 commits intomoby:masterfrom
creack:fix-volumes-on-host
Dec 12, 2013
Merged

Fix volumes on host#3184
vieux merged 2 commits intomoby:masterfrom
creack:fix-volumes-on-host

Conversation

@creack
Copy link
Copy Markdown
Contributor

@creack creack commented Dec 12, 2013

Prevent the user to be able to create arbitrary directory on the host FS

@tianon
Copy link
Copy Markdown
Member

tianon commented Dec 12, 2013

Super massive +1s - was just discussing similar issues with @vieux earlier today, what a "coincidence." :)

@vieux
Copy link
Copy Markdown
Contributor

vieux commented Dec 12, 2013

":)"

On Wed, Dec 11, 2013 at 5:18 PM, Tianon Gravi [email protected]:

Super massive +1s - was just discussing similar issues with @vieuxhttps://github.com/vieuxearlier today, what a "coincidence." :)


Reply to this email directly or view it on GitHubhttps://github.com//pull/3184#issuecomment-30381538
.

Victor VIEUX
http://vvieux.com

@crosbymichael
Copy link
Copy Markdown
Contributor

@tianon yep we got it taken care of.

@crosbymichael
Copy link
Copy Markdown
Contributor

LGTM

@tianon
Copy link
Copy Markdown
Member

tianon commented Dec 12, 2013

@crosbymichael that's because you guys are so awesome :D

Seriously, nice work.

@vieux
Copy link
Copy Markdown
Contributor

vieux commented Dec 12, 2013

LGTM

vieux pushed a commit that referenced this pull request Dec 12, 2013
@vieux vieux merged commit 621523a into moby:master Dec 12, 2013
@creack creack deleted the fix-volumes-on-host branch December 12, 2013 19:35
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.

4 participants