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.

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

Dangerous!,need param check #6

Closed
oxsean opened this issue Mar 19, 2014 · 3 comments
Closed

Dangerous!,need param check #6

oxsean opened this issue Mar 19, 2014 · 3 comments

Comments

@oxsean
Copy link

oxsean commented Mar 19, 2014

I guess and run 'cooler export /' to backup all,and my mac nearly be killed
line 100:
rm -rf "${2%.*}" >/dev/null

@alebcay
Copy link
Owner

alebcay commented Mar 19, 2014

Where did you run it from and what did you name your bundle?

@oxsean
Copy link
Author

oxsean commented Mar 19, 2014

My home,I don't look the usage carefully
when i run: cooler export /
it become to: rm -rf /

@alebcay
Copy link
Owner

alebcay commented Mar 19, 2014

cooler export is supposed to receive a file as a parameter, not a directory. Fortunately, there are safeguards built in against rm -rf /. Granted, there should be some additional safety checks on parameters. Expect that in the next release. Thanks for bringing this up!

@oxsean oxsean closed this as completed Mar 20, 2014
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

No branches or pull requests

2 participants