From 907791ddbb0dc70f67d41ac6bd4a661609caee11 Mon Sep 17 00:00:00 2001
From: Bob Mottram <bob@robotics.uk.to>
Date: Sat, 1 Oct 2016 10:48:07 +0100
Subject: [PATCH] quotes

---
 src/freedombone-app-blog       |  4 ++--
 src/freedombone-app-emacs      |  4 ++--
 src/freedombone-app-irc        |  2 +-
 src/freedombone-app-syncthing  |  4 ++--
 src/freedombone-app-tahoelafs  |  2 +-
 src/freedombone-app-xmpp       |  2 +-
 src/freedombone-archive-mail   |  2 +-
 src/freedombone-backup-local   |  2 +-
 src/freedombone-backup-remote  |  4 ++--
 src/freedombone-controlpanel   |  2 +-
 src/freedombone-restore-local  | 22 +++++++++++-----------
 src/freedombone-restore-remote | 22 +++++++++++-----------
 src/freedombone-sec            |  4 ++--
 src/freedombone-syncthing      | 10 +++++-----
 src/freedombone-utils-backup   |  2 +-
 15 files changed, 44 insertions(+), 44 deletions(-)

diff --git a/src/freedombone-app-blog b/src/freedombone-app-blog
index cfb3f9351..822994308 100755
--- a/src/freedombone-app-blog
+++ b/src/freedombone-app-blog
@@ -244,7 +244,7 @@ function restore_local_blog {
         fi
         for d in /home/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ -d /var/www/${FULLBLOG_DOMAIN_NAME}/htdocs/content/$USERNAME/blog/uncategorized/post ]; then
                     mv /var/www/${FULLBLOG_DOMAIN_NAME}/htdocs/content/$USERNAME/blog/*.md /var/www/${FULLBLOG_DOMAIN_NAME}/htdocs/content/$USERNAME/blog/uncategorized/post
                 fi
@@ -296,7 +296,7 @@ function restore_remote_blog {
         fi
         for d in /home/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ -d /var/www/${FULLBLOG_DOMAIN_NAME}/htdocs/content/$USERNAME/blog/uncategorized/post ]; then
                     mv /var/www/${FULLBLOG_DOMAIN_NAME}/htdocs/content/$USERNAME/blog/*.md /var/www/${FULLBLOG_DOMAIN_NAME}/htdocs/content/$USERNAME/blog/uncategorized/post
                 fi
diff --git a/src/freedombone-app-emacs b/src/freedombone-app-emacs
index 36cef8ec2..a9d5a0a1a 100755
--- a/src/freedombone-app-emacs
+++ b/src/freedombone-app-emacs
@@ -49,7 +49,7 @@ function upgrade_emacs {
 function backup_local_emacs {
     for d in /home/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -d /home/$USERNAME/.emacs.d ]; then
                 echo $"Backing up Emacs config for $USERNAME"
                 if [ -f /home/$USERNAME/.emacs ]; then
@@ -67,7 +67,7 @@ function restore_local_emacs {
     if [ -d $USB_MOUNT/backup/emacs ]; then
         for d in $USB_MOUNT/backup/emacs/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
                 fi
diff --git a/src/freedombone-app-irc b/src/freedombone-app-irc
index 378494cf3..1885ac883 100755
--- a/src/freedombone-app-irc
+++ b/src/freedombone-app-irc
@@ -61,7 +61,7 @@ function irc_set_global_password {
             # replace the password for all users
             for d in /home/*/ ; do
                 IRC_USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-                if [[ $(is_valid_user $IRC_USERNAME) == "1" ]]; then
+                if [[ $(is_valid_user "$IRC_USERNAME") == "1" ]]; then
                     if [ -f /home/$IRC_USERNAME/.irssi/config ]; then
                         sed -i "s|$EXISTING_IRC_PASSWORD|$NEW_IRC_PASSWORD|g" /home/$IRC_USERNAME/.irssi/config
                         chown -R $IRC_USERNAME:$IRC_USERNAME /home/$IRC_USERNAME/.irssi
diff --git a/src/freedombone-app-syncthing b/src/freedombone-app-syncthing
index f2e4193d9..1829e1ee7 100755
--- a/src/freedombone-app-syncthing
+++ b/src/freedombone-app-syncthing
@@ -278,7 +278,7 @@ function restore_local_syncthing {
     if [ -d $USB_MOUNT/backup/syncthing ]; then
         for d in $USB_MOUNT/backup/syncthing/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
                 fi
@@ -366,7 +366,7 @@ function restore_remote_syncthing {
     if [ -d $SERVER_DIRECTORY/backup/syncthing ]; then
         for d in $SERVER_DIRECTORY/backup/syncthing/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
                 fi
diff --git a/src/freedombone-app-tahoelafs b/src/freedombone-app-tahoelafs
index 53e1e0edd..29e5f4238 100755
--- a/src/freedombone-app-tahoelafs
+++ b/src/freedombone-app-tahoelafs
@@ -50,7 +50,7 @@ function change_password_tahoelafs {
 function reconfigure_tahoelafs {
     for d in /home/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -d /home/$USERNAME/.tahoe ]; then
                 su -c "$TAHOE_COMMAND stop" - $USERNAME
                 rm -rf /home/$USERNAME/.tahoe
diff --git a/src/freedombone-app-xmpp b/src/freedombone-app-xmpp
index 8e3d3c58d..fde47fa81 100755
--- a/src/freedombone-app-xmpp
+++ b/src/freedombone-app-xmpp
@@ -246,7 +246,7 @@ function remove_xmpp {
 function xmpp_email_headers {
     for d in /home/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -f /home/$USERNAME/.muttrc ]; then
                 if ! grep -q "Jabber-ID" /home/$USERNAME/.muttrc; then
                     echo "my_hdr Jabber-ID: ${USERNAME}@${HOSTNAME}" >> /home/$USERNAME/.muttrc
diff --git a/src/freedombone-archive-mail b/src/freedombone-archive-mail
index de21c549a..95ed11967 100755
--- a/src/freedombone-archive-mail
+++ b/src/freedombone-archive-mail
@@ -37,7 +37,7 @@ source /usr/share/${PROJECT_NAME}/utils/${PROJECT_NAME}-utils-validation
 
 for d in /home/*/ ; do
     USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-    if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+    if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
         # for every user who has a mail directory
         if [ -d /home/$USERNAME/Maildir ]; then
             MUTTRC=/home/$USERNAME/.muttrc
diff --git a/src/freedombone-backup-local b/src/freedombone-backup-local
index 07d911bc0..ded0f8fc9 100755
--- a/src/freedombone-backup-local
+++ b/src/freedombone-backup-local
@@ -98,7 +98,7 @@ function backup_users {
     # Backup user files
     for d in /home/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
 
             # Backup any gpg keys
             if [ -d /home/$USERNAME/.gnupg ]; then
diff --git a/src/freedombone-backup-remote b/src/freedombone-backup-remote
index d2aae8896..88a8ee2cd 100755
--- a/src/freedombone-backup-remote
+++ b/src/freedombone-backup-remote
@@ -122,7 +122,7 @@ function backup_configuration {
 function backup_users {
     for d in /home/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
 
             # personal settings
             if [ -d /home/$USERNAME/personal ]; then
@@ -451,7 +451,7 @@ do
         # Social key management
         for d in /home/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 disperse_key_shares $USERNAME $REMOTE_DOMAIN $REMOTE_SSH_PORT "$REMOTE_PASSWORD" $REMOTE_SERVER
             fi
         done
diff --git a/src/freedombone-controlpanel b/src/freedombone-controlpanel
index 6d82f4149..8c0c3bc9d 100755
--- a/src/freedombone-controlpanel
+++ b/src/freedombone-controlpanel
@@ -459,7 +459,7 @@ function show_users {
     echo '--------------------------------------------------------------------------'
     for d in /home/*/ ; do
         USRNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USRNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USRNAME") == "1" ]]; then
             echo -n -e "$(pad_string ${USRNAME})"
             # get the SIP extension
             SIPEXT=
diff --git a/src/freedombone-restore-local b/src/freedombone-restore-local
index c2ae9c2e0..c96637c7f 100755
--- a/src/freedombone-restore-local
+++ b/src/freedombone-restore-local
@@ -238,7 +238,7 @@ function restore_mutt_settings {
     if [ -d $USB_MOUNT/backup/mutt ]; then
         for d in $USB_MOUNT/backup/mutt/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
                 fi
@@ -272,7 +272,7 @@ function restore_gpg {
     if [ -d $USB_MOUNT/backup/gnupg ]; then
         for d in $USB_MOUNT/backup/gnupg/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
                 fi
@@ -309,7 +309,7 @@ function restore_procmail {
     if [ -d $USB_MOUNT/backup/procmail ]; then
         for d in $USB_MOUNT/backup/procmail/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
                 fi
@@ -338,7 +338,7 @@ function restore_spamassassin {
     if [ -d $USB_MOUNT/backup/spamassassin ]; then
         for d in $USB_MOUNT/backup/spamassassin/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ -d $USB_MOUNT/backup/spamassassin/$USERNAME ]; then
                     if [ ! -d /home/$USERNAME ]; then
                         ${PROJECT_NAME}-adduser $USERNAME
@@ -399,7 +399,7 @@ function restore_user_ssh_keys {
     if [ -d $USB_MOUNT/backup/ssh ]; then
         for d in $USB_MOUNT/backup/ssh/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
                 fi
@@ -428,7 +428,7 @@ function restore_user_config {
     if [ -d $USB_MOUNT/backup/config ]; then
         for d in $USB_MOUNT/backup/config/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
                 fi
@@ -469,7 +469,7 @@ function restore_user_monkeysphere {
     if [ -d $USB_MOUNT/backup/monkeysphere ]; then
         for d in $USB_MOUNT/backup/monkeysphere/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
                 fi
@@ -508,7 +508,7 @@ function restore_user_fin {
     if [ -d $USB_MOUNT/backup/fin ]; then
         for d in $USB_MOUNT/backup/fin/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
                 fi
@@ -537,7 +537,7 @@ function restore_user_local {
     if [ -d $USB_MOUNT/backup/local ]; then
         for d in $USB_MOUNT/backup/local/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
                 fi
@@ -610,7 +610,7 @@ function restore_personal_settings {
     if [ -d $USB_MOUNT/backup/personal ]; then
         for d in $USB_MOUNT/backup/personal/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ -d $USB_MOUNT/backup/personal/$USERNAME ]; then
                     if [ ! -d /home/$USERNAME ]; then
                         ${PROJECT_NAME}-adduser $USERNAME
@@ -663,7 +663,7 @@ function restore_email {
     if [ -d $USB_MOUNT/backup/mail ]; then
         for d in $USB_MOUNT/backup/mail/*/ ; do
             USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-            if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+            if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
                 fi
diff --git a/src/freedombone-restore-remote b/src/freedombone-restore-remote
index ce540e622..3fc1b90dd 100755
--- a/src/freedombone-restore-remote
+++ b/src/freedombone-restore-remote
@@ -227,7 +227,7 @@ function restore_mutt_settings {
     fi
     for d in $SERVER_DIRECTORY/backup/mutt/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -d $SERVER_DIRECTORY/backup/mutt/$USERNAME ]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
@@ -259,7 +259,7 @@ function restore_gpg {
     fi
     for d in $SERVER_DIRECTORY/backup/gnupg/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -d $SERVER_DIRECTORY/backup/gnupg/$USERNAME ]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
@@ -292,7 +292,7 @@ function restore_procmail {
     fi
     for d in $SERVER_DIRECTORY/backup/procmail/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -d $SERVER_DIRECTORY/backup/procmail/$USERNAME ]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
@@ -319,7 +319,7 @@ function restore_spamassassin {
     fi
     for d in $SERVER_DIRECTORY/backup/spamassassin/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -d $SERVER_DIRECTORY/backup/spamassassin/$USERNAME ]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
@@ -365,7 +365,7 @@ function restore_ssh_keys {
     fi
     for d in $SERVER_DIRECTORY/backup/ssh/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -d $SERVER_DIRECTORY/backup/ssh/$USERNAME ]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
@@ -392,7 +392,7 @@ function restore_user_config {
     fi
     for d in $SERVER_DIRECTORY/backup/config/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -d $SERVER_DIRECTORY/backup/config/$USERNAME ]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
@@ -431,7 +431,7 @@ function restore_user_monkeysphere {
     fi
     for d in $SERVER_DIRECTORY/backup/monkeysphere/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -d $SERVER_DIRECTORY/backup/monkeysphere/$USERNAME ]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
@@ -468,7 +468,7 @@ function restore_user_fin {
     fi
     for d in $SERVER_DIRECTORY/backup/fin/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -d $SERVER_DIRECTORY/backup/fin/$USERNAME ]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
@@ -495,7 +495,7 @@ function restore_user_local {
     fi
     for d in $SERVER_DIRECTORY/backup/local/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -d $SERVER_DIRECTORY/backup/local/$USERNAME ]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
@@ -563,7 +563,7 @@ function restore_personal_settings {
     fi
     for d in $SERVER_DIRECTORY/backup/personal/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -d $SERVER_DIRECTORY/backup/personal/$USERNAME ]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
@@ -610,7 +610,7 @@ function restore_email {
     fi
     for d in $SERVER_DIRECTORY/backup/mail/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $6}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -d $SERVER_DIRECTORY/backup/mail/$USERNAME ]; then
                 if [ ! -d /home/$USERNAME ]; then
                     ${PROJECT_NAME}-adduser $USERNAME
diff --git a/src/freedombone-sec b/src/freedombone-sec
index 5b126d5da..4debbe9ac 100755
--- a/src/freedombone-sec
+++ b/src/freedombone-sec
@@ -338,7 +338,7 @@ function send_monkeysphere_server_keys_to_users {
     monkeysphere_server_keys=$(monkeysphere-host show-key | grep $"OpenPGP fingerprint" | awk -F ' ' '{print $3}')
     for d in /home/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ ! -d /home/$USERNAME/.monkeysphere ]; then
                 mkdir /home/$USERNAME/.monkeysphere
             fi
@@ -956,7 +956,7 @@ function export_settings {
 function refresh_gpg_keys {
     for d in /home/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             su -c 'gpg --refresh-keys' - $USERNAME
         fi
     done
diff --git a/src/freedombone-syncthing b/src/freedombone-syncthing
index 777f14044..5267db3d9 100755
--- a/src/freedombone-syncthing
+++ b/src/freedombone-syncthing
@@ -99,7 +99,7 @@ function create_syncthing_config {
 
     for d in /home/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             echo "    <folder id=\"$USERNAME\" path=\"/home/$USERNAME/Sync/\" ro=\"false\" rescanIntervalS=\"60\" ignorePerms=\"false\" autoNormalize=\"true\">" >> $SYNCTHING_CONFIG_FILE
             # include any specified device IDs for this user
             if [ -f /home/$USERNAME/$SYNCTHING_USER_IDS_FILE ]; then
@@ -139,7 +139,7 @@ function create_syncthing_config {
     echo "" > $TEMP_IDS_FILE
     for d in /home/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -f /home/$USERNAME/$SYNCTHING_USER_IDS_FILE ]; then
                 while read -r line || [[ -n "$line" ]]; do
                     line2="$(echo -e "${line}" | sed -e 's/^[[:space:]]*//' -e 's/[[:space:]]*$//')"
@@ -179,7 +179,7 @@ function create_syncthing_config {
     echo "" > $TEMP_IDS_FILE
     for d in /home/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -f /home/$USERNAME/$SYNCTHING_USER_IDS_FILE ]; then
                 while read -r line || [[ -n "$line" ]]; do
                     line2="$(echo -e "${line}" | sed -e 's/^[[:space:]]*//' -e 's/[[:space:]]*$//')"
@@ -238,7 +238,7 @@ function create_syncthing_config {
     # This allows it to appear within the user control panel
     for d in /home/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             echo "$SYNCTHING_ID" > /home/$USERNAME/.syncthing-server-id
             chown $USERNAME:$USERNAME /home/$USERNAME/.syncthing-server-id
         fi
@@ -259,7 +259,7 @@ function user_devices_changed {
 
     for d in /home/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             if [ -f /home/$USERNAME/$SYNCTHING_UPDATE_FILE ]; then
                 CHANGED=1
             fi
diff --git a/src/freedombone-utils-backup b/src/freedombone-utils-backup
index a076c32d1..3b934e9e7 100755
--- a/src/freedombone-utils-backup
+++ b/src/freedombone-utils-backup
@@ -275,7 +275,7 @@ function set_user_permissions {
     echo $"Setting permissions"
     for d in /home/*/ ; do
         USERNAME=$(echo "$d" | awk -F '/' '{print $3}')
-        if [[ $(is_valid_user $USERNAME) == "1" ]]; then
+        if [[ $(is_valid_user "$USERNAME") == "1" ]]; then
             chown -R $USERNAME:$USERNAME /home/$USERNAME
         fi
     done
-- 
GitLab