fix(preferences(js)): conditionally sanitize forward addresses

Fixes #5085
Alinto · Jul 16, 2020 · b78e66a · b78e66a
1 parent e6e994b
commit b78e66a
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/UI/WebServerResources/js/Preferences/PreferencesController.js b/UI/WebServerResources/js/Preferences/PreferencesController.js
@@ -387,14 +387,16 @@
       // We do some sanity checks
 
       // We check if we're allowed or not to forward based on the domain defaults
-      addresses = this.preferences.defaults.Forward.forwardAddress;
-      try {
-        for (i = 0; i < addresses.length; i++) {
-          validateForwardAddress(addresses[i]);
+      if (this.preferences.defaults.Forward && this.preferences.defaults.Forward.forwardAddress) {
+        addresses = this.preferences.defaults.Forward.forwardAddress;
+        try {
+          for (i = 0; i < addresses.length; i++) {
+            validateForwardAddress(addresses[i]);
+          }
+        } catch (err) {
+          Dialog.alert(l('Error'), err);
+          sendForm = false;
         }
-      } catch (err) {
-        Dialog.alert(l('Error'), err);
-        sendForm = false;
       }
 
       // IMAP labels must be unique