diff --git a/mailnews/compose/src/nsSmtpServer.cpp b/mailnews/compose/src/nsSmtpServer.cpp
index a5e4bd0d2c18..bfc210937443 100644
--- a/mailnews/compose/src/nsSmtpServer.cpp
+++ b/mailnews/compose/src/nsSmtpServer.cpp
@@ -151,7 +151,8 @@ nsSmtpServer::SetDescription(const nsACString &aDescription)
     if (!aDescription.IsEmpty())
         return mPrefBranch->SetCharPref("description", PromiseFlatCString(aDescription).get());
     else
-        return mPrefBranch->ClearUserPref("description");
+        mPrefBranch->ClearUserPref("description");
+    return NS_OK;
 }
 
 // if GetPort returns 0, it means default port