Skip to content

Commit

Permalink
Removing unused tmpfile variable. Also...
Browse files Browse the repository at this point in the history
- `local` is undefined by POSIX (http://mywiki.wooledge.org/Bashism)
- declaration and assigment should be separated (https://github.com/koalaman/shellcheck/wiki/SC2155)
- and using $() rather than `code` (https://github.com/koalaman/shellcheck/wiki/SC2006)
  • Loading branch information
PedroSFreitas committed Aug 9, 2017
1 parent d573028 commit c66fd16
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion sploitctl.sh
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,9 @@ search_web()
# search exploit(s) using given string pattern
search_archive()
{
local tmpfile="`mktemp`"
# `local` is undefined by POSIX and `tmpfile` seems to be unused. Read full commit.
# local tmpfile
# tmpfile="$(mktemp)"

msg "searching exploit for '${srch_str}'"

Expand Down

0 comments on commit c66fd16

Please sign in to comment.