The awesome tool https://www.shellcheck.net/ ("finds bugs in your shell scripts") finds 13 suggestions when checking 1.6.5, see attached.

Nothing critical, but still would be cool to have fixed.

Thanks, and keep up the good work!

--
ilf

Über 80 Millionen Deutsche benutzen keine Konsole. Klick dich nicht weg!
                -- Eine Initiative des Bundesamtes für Tastaturbenutzung
In /usr/bin/pass line 86:
        local groups="$($GPG --list-config --with-colons | grep 
"^cfg:group:.*")"
              ^-- SC2155: Declare and assign separately to avoid masking return 
values.


In /usr/bin/pass line 98:
                                local group="$(sed -n "s/^cfg:group:$(sed 
's/[\/&]/\\&/g' <<<"${GPG_RECIPIENTS[$index]}"):\\(.*\\)\$/\\1/p" <<<"$groups" 
| head -n 1)"
                                      ^-- SC2155: Declare and assign separately 
to avoid masking return values.


In /usr/bin/pass line 101:
                                unset GPG_RECIPIENTS[$index]
                                      ^-- SC2184: Quote arguments to unset so 
they're not glob expanded.


In /usr/bin/pass line 109:
                        $GPG -d "${GPG_OPTS[@]}" "$passfile" | $GPG -e 
"${GPG_RECIPIENT_ARGS[@]}" -o "$passfile_temp" "${GPG_OPTS[@]}" &&
                                                                                
                                                       ^-- SC2015: Note that A 
&& B || C is not if-then-else. C may run when A is true.


In /usr/bin/pass line 136:
        local before="$(xclip -o -selection "$X_SELECTION" 2>/dev/null | 
base64)"
              ^-- SC2155: Declare and assign separately to avoid masking return 
values.


In /usr/bin/pass line 140:
                local now="$(xclip -o -selection "$X_SELECTION" | base64)"
                      ^-- SC2155: Declare and assign separately to avoid 
masking return values.


In /usr/bin/pass line 178:
                        find "$SECURE_TMPDIR" -type f -exec $SHRED {} +
                                                            ^-- SC2086: Double 
quote to prevent globbing and word splitting.


In /usr/bin/pass line 286:
                local id_print="$(printf "%s, " "$@")"
                      ^-- SC2155: Declare and assign separately to avoid 
masking return values.


In /usr/bin/pass line 314:
                        local pass="$($GPG -d "${GPG_OPTS[@]}" "$passfile" | 
head -n 1)"
                              ^-- SC2155: Declare and assign separately to 
avoid masking return values.


In /usr/bin/pass line 335:
        local terms="*$(printf '%s*|*' "$@")"
              ^-- SC2155: Declare and assign separately to avoid masking return 
values.


In /usr/bin/pass line 344:
                [ $? -ne 0 ] && continue
                  ^-- SC2181: Check exit code directly with e.g. 'if mycmd;', 
not indirectly with $?.


In /usr/bin/pass line 413:
        local tmp_file="$(mktemp -u "$SECURE_TMPDIR/XXXXX")-${path//\//-}.txt"
              ^-- SC2155: Declare and assign separately to avoid masking return 
values.


In /usr/bin/pass line 454:
        local pass="$(pwgen -s $symbols $length 1)"
              ^-- SC2155: Declare and assign separately to avoid masking return 
values.
                                        ^-- SC2086: Double quote to prevent 
globbing and word splitting.

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Password-Store mailing list
[email protected]
https://lists.zx2c4.com/mailman/listinfo/password-store

Reply via email to