In ec2-instance-connect/eic_harvest_hostkeys line 21:
elif [ $(cat /sys/hypervisor/uuid | cut -c1-3) != "ec2" ] ; then
       ^-- SC2046: Quote this to prevent word splitting.
             ^-- SC2002: Useless cat. Consider 'cmd < file | ..' or 'cmd file | 
..' instead.


In ec2-instance-connect/eic_harvest_hostkeys line 35:
    printf "$2" | openssl dgst -binary -hex -sha256 -mac HMAC -macopt hexkey:$1 
| sed 's/.* //'
           ^-- SC2059: Don't use variables in the printf format string. Use 
printf "..%s.." "$foo".
                                                                             
^-- SC2086: Double quote to prevent globbing and word splitting.


In ec2-instance-connect/eic_harvest_hostkeys line 41:
    local base=$(echo -n "AWS4${1}" | od -A n -t x1 | sed ':a;N;$!ba;s/[\n 
]//g')
          ^-- SC2155: Declare and assign separately to avoid masking return 
values.


In ec2-instance-connect/eic_harvest_hostkeys line 42:
    local kdate=$(sign "${base}" $2)
          ^-- SC2155: Declare and assign separately to avoid masking return 
values.
                                 ^-- SC2086: Double quote to prevent globbing 
and word splitting.


In ec2-instance-connect/eic_harvest_hostkeys line 43:
    local kregion=$(sign "${kdate}" $3)
          ^-- SC2155: Declare and assign separately to avoid masking return 
values.
                                    ^-- SC2086: Double quote to prevent 
globbing and word splitting.


In ec2-instance-connect/eic_harvest_hostkeys line 44:
    local kservice=$(sign "${kregion}" $4)
          ^-- SC2155: Declare and assign separately to avoid masking return 
values.
                                       ^-- SC2086: Double quote to prevent 
globbing and word splitting.


In ec2-instance-connect/eic_harvest_hostkeys line 51:
if [ $(eval "${curl_cmd} -o /dev/null -I -w %{http_code} 
http://169.254.169.254/latest/meta-data/identity-credentials/ec2/security-credentials/ec2-instance/";)
 -ne 200 ]
     ^-- SC2046: Quote this to prevent word splitting.


In ec2-instance-connect/eic_harvest_hostkeys line 60:
        key=$(cat $file | awk '{$1=$1};1')
                  ^-- SC2086: Double quote to prevent globbing and word 
splitting.
                  ^-- SC2002: Useless cat. Consider 'cmd < file | ..' or 'cmd 
file | ..' instead.


In ec2-instance-connect/eic_harvest_hostkeys line 61:
        keys="${keys:+${keys},}\"${key}\""
                               ^-- SC2089: Quotes/backslashes will be treated 
literally. Use an array.


In ec2-instance-connect/eic_harvest_hostkeys line 66:
trap "rm -rf '${userpath}'" EXIT
              ^-- SC2064: Use single quotes, otherwise this expands now rather 
than when signalled.


In ec2-instance-connect/eic_harvest_hostkeys line 69:
chmod 700 $userpath
          ^-- SC2086: Double quote to prevent globbing and word splitting.


In ec2-instance-connect/eic_harvest_hostkeys line 73:
if [ $? -ne 0 ]
     ^-- SC2181: Check exit code directly with e.g. 'if mycmd;', not indirectly 
with $?.


In ec2-instance-connect/eic_harvest_hostkeys line 86:
if [ $? -ne 0 ]
     ^-- SC2181: Check exit code directly with e.g. 'if mycmd;', not indirectly 
with $?.


In ec2-instance-connect/eic_harvest_hostkeys line 107:
hostkeys=$(echo ${keys:?})
         ^-- SC2116: Useless echo? Instead of 'cmd $(echo foo)', just use 'cmd 
foo'.
                ^-- SC2090: Quotes/backslashes in this variable will not be 
respected.
                ^-- SC2086: Double quote to prevent globbing and word splitting.


In ec2-instance-connect/eic_harvest_hostkeys line 118:
if [ $? -ne 0 ] ; then
     ^-- SC2181: Check exit code directly with e.g. 'if mycmd;', not indirectly 
with $?.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1835114

Title:
  [MIR] ec2-instance-connect

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/ec2-instance-connect/+bug/1835114/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to