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

Cake.Core.Tooling.ToolRepository.Register(FilePath) path not validated #933

Closed
devlead opened this issue May 31, 2016 · 0 comments
Closed

Cake.Core.Tooling.ToolRepository.Register(FilePath) path not validated #933

devlead opened this issue May 31, 2016 · 0 comments
Assignees
@devlead
Labels
Bug
Milestone
v0.13.0

Comments

@devlead
Copy link
Member

devlead commented May 31, 2016

#848 / 7dd9118#diff-b2db3067153e940008b4cd0f0fe76632R34

Code analyzer triggers warning:
Warning CA1062 In externally visible method 'ToolRepository.Register(FilePath)', validate parameter 'path' before using it.

I'll submit PR shortly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@devlead devlead

Labels
Bug
Projects
None yet
Milestone
v0.13.0
Development

No branches or pull requests
2 participants
@gep13 @devlead