From f002b3573afcd25c0bc19b4b2c8d22fda87af517 Mon Sep 17 00:00:00 2001
From: Richard Soderberg <rsoderberg@gmail.com>
Date: Thu, 3 Sep 2015 06:36:32 -0700
Subject: [PATCH] fixes for "SC2004: $/${} is unnecessary on arithmetic
 variables."

In cipherscan line 451:
        for ((i=0; i<$certificate_count; i=i+1 )); do
                     ^-- SC2004: $/${} is unnecessary on arithmetic variables.

In cipherscan line 603:
    cipherbenchms="$((t/1000/$BENCHMARKITER))"
                             ^-- SC2004: $/${} is unnecessary on arithmetic variables.
---
 cipherscan | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/cipherscan b/cipherscan
index 8093918..db5c143 100755
--- a/cipherscan
+++ b/cipherscan
@@ -448,7 +448,7 @@ test_cipher_on_target() {
         local certificate_count=$certs_found
         debug "server presented $certificate_count certificates"
         local i
-        for ((i=0; i<$certificate_count; i=i+1 )); do
+        for ((i=0; i<certificate_count; i=i+1 )); do
 
             # extract i'th certificate
             local cert="${current_raw_certificates[$i]}"
@@ -600,7 +600,7 @@ bench_cipher() {
     local t="$(($(date +%s%N) - t))"
     verbose "Benchmarking done in $t nanoseconds"
     # Microseconds
-    cipherbenchms="$((t/1000/$BENCHMARKITER))"
+    cipherbenchms="$((t/1000/BENCHMARKITER))"
 }
 
 # Connect to the target and retrieve the chosen cipher