Skip to content

Commit

Permalink
Prevent clean_plugins from deleting root
Browse files Browse the repository at this point in the history
If tmux was not yet configured to initialize the Tmux Plugin Manager,
the variable for the plugin home was not being set. This was resulting
in a `rm -rf /` which is pretty nuclear.

Fixes: #58
  • Loading branch information
Graham42 committed Dec 3, 2016
1 parent cfaf15b commit 1579534
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 0 deletions.
13 changes: 13 additions & 0 deletions scripts/helpers/plugin_functions.sh
Expand Up @@ -31,9 +31,22 @@ _sourced_files() {
awk '/^[ \t]*source(-file)? +/ { gsub(/'\''/,""); gsub(/'\"'/,""); print $2 }'
}

# Want to be able to abort in certain cases
trap "exit 1" TERM
export TOP_PID=$$

_fatal_error_abort() {
echo >&2 "Aborting."
kill -s TERM $TOP_PID
}

# PUBLIC FUNCTIONS BELOW

tpm_path() {
if [ "$_CACHED_TPM_PATH" == "/" ]; then
echo >&2 "FATAL: Tmux Plugin Manager not configured in tmux.conf"
_fatal_error_abort
fi
echo "$_CACHED_TPM_PATH"
}

Expand Down
1 change: 1 addition & 0 deletions scripts/update_plugin.sh
Expand Up @@ -60,6 +60,7 @@ update_plugins() {
}

main() {
ensure_tpm_path_exists
if [ "$1" == "all" ]; then
update_all
else
Expand Down

0 comments on commit 1579534

Please sign in to comment.