Bash array with spaces and no spaces in elements

Question

I know this question has been asked in different manners, and I've referred to some answers on here to get to where I am now.

I'm trying to create a script to essentially watch a folder and virus scan files once they're not being written to.

The files/strings I need it to handle will sometimes contain spaces and sometimes not, as well sometimes special characters. At the moment it will actually work only on files with spaces in the name (as in actually scan and move them), but not for files without spaces. Also after each file (spaces or not) the while loop breaks thus stopping the script with the following output;

./howscan.sh: line 29: snmp.conf: syntax error: invalid arithmetic operator (error token is ".conf")
./howscan.sh: line 34: snmp.conf: syntax error: invalid arithmetic operator (error token is ".conf")  

I had it working to handle file names without any spaces, but since I introduced the "${files[$i]}" method to use the array elements it only works on files with spaces and outputs the above error.

Feel free to omit the sepscan part of this, as I'm sure if I can get it working with the other tasks it'll work for that too (just wanted to show the full script for a complete understanding).

Current Script:

#!/bin/bash

set -x

workingpath='/mnt/Incoming/ToScan/'
outputpath='/mnt/Incoming/Scanned/'
logfile='/var/log/howscan.log'
faildir='/mnt/Incoming/ScanFailed/'
sepscan='/opt/Symantec/symantec_antivirus/sav manualscan -c'

# Change to working directory
cd $workingpath

# Exclude files with a given extension, in this case .aspx, and declare the remaining files as the array "files"
shopt -s extglob nullglob

# Loop the below - ie it's a watch folder
while true
    do
    # Exclude files with .aspx in the file name
    files=( !(*.aspx) )
    # If the array files is not empty then...
    if [ ${#files[@]} -ne 0 ]; then
        for i in ${files[*]}
        # For every item in the array files process them as follows
        # Declare any variables you wish to happen on files here, not globally
        # Check each file to see if it's in use using fuser, do nothing and log it if its still in use, or process it and log the results
        do
            fileopen=`fuser "${files[$i]}" | wc -c`
            # Here for 'fileopen' we are checking if the file is being writen to.
            if [ $fileopen -ne 0 ]; then
                echo `date` "${files[$i]}" is being used so has been ignored >> $logfile
            else
                echo `date` File "${files[$i]}" not being used or accessed >> $logfile

                    sepscan=`$sepscan "${files[$i]}" | wc -c`
                    if [ $sepscan = 0 ]; then
                        mv "${files[$i]}" $outputpath
                        echo `date` "${files[$i]}" has been virus scanned and moved to $outputpath >> $logfile
                    else 
                        echo `date` "${files[$i]}" has been moved to $faildir as a virus or error was detected >> $logfile
                    fi
            fi
        done
    fi
    echo `date` 'No more files to process. Waiting 60 seconds...' >> $logfile
    sleep 60
done  

Let me know if I can provide anything else to help clarify my issue.

Update:
There is a file in the /mnt/Incoming/ToScan/ directory called snmp.conf by the way.

Answer 1

for i in ${files[*]}

should be

for i in ${!files[*]}
# or
for i in ${!files[@]}

${files[*]} expands to the contents of the array and undergoes word splitting. The above syntax expands to a list of indices of the array.

You might also need to double quote the variables, e.g.

if [ "$fileopen" -ne 0 ]; then