From 3664b1a1991103337f4eb9b253f9955f5afcfbbc Mon Sep 17 00:00:00 2001 From: Richard Soderberg Date: Thu, 3 Sep 2015 06:11:40 -0700 Subject: [PATCH] fixes for "SC2145: Argument mixes string and array. Use * or separate argument." In cipherscan line 941: verbose "Server supported curves: ${tmp_curves[@]}" ^-- SC2145: Argument mixes string and array. Use * or separate argument. In cipherscan line 968: verbose "ephem_data: ${ephem_data[@]}" ^-- SC2145: Argument mixes string and array. Use * or separate argument. --- cipherscan | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cipherscan b/cipherscan index cf115b7..1ecb3ee 100755 --- a/cipherscan +++ b/cipherscan @@ -938,7 +938,7 @@ test_curves() { # local tmp_curves=(${current_curves//,/ }) - verbose "Server supported curves: ${tmp_curves[@]}" + verbose "Server supported curves: ${tmp_curves[*]}" # server supports just one or none, so it effectively uses server side # ordering (as it dictates what curves client must support) @@ -965,7 +965,7 @@ test_curves() { else local ephem_data=(${current_pfs//,/ }) verbose "Server selected $current_cipher with $current_pfs" - verbose "ephem_data: ${ephem_data[@]}" + verbose "ephem_data: ${ephem_data[*]}" if [[ ${ephem_data[0]} =~ ECDH ]]; then verbose "Server did select ${ephem_data[1]} curve"