From 9f36302efac5ddffd7798da0893ec6e8af59cb0b Mon Sep 17 00:00:00 2001 From: John Forte Date: Fri, 16 Aug 2019 17:15:25 -0400 Subject: [PATCH] Was failing the check if clicking test notifications twice in a row. So it was saving the asterisks and overwriting the current password. (cherry picked from commit 6176862f98749e15524e02ccaa705b65c498ceed) --- src/usr/local/www/system_advanced_notifications.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/usr/local/www/system_advanced_notifications.php b/src/usr/local/www/system_advanced_notifications.php index 584b09ce4d..ee025ab5d0 100644 --- a/src/usr/local/www/system_advanced_notifications.php +++ b/src/usr/local/www/system_advanced_notifications.php @@ -139,7 +139,7 @@ if ($_POST) { $config['notifications']['smtp']['notifyemailaddress'] = $_POST['smtpnotifyemailaddress']; $config['notifications']['smtp']['username'] = $_POST['smtpusername']; - if ($_POST['smtppassword'] != DMYPWD) { + if (strcmp($_POST['smtppassword'], DMYPWD)!= 0) { if ($_POST['smtppassword'] == $_POST['smtppassword_confirm']) { $config['notifications']['smtp']['password'] = $_POST['smtppassword']; } else {