From 0f72bf29fe38b24c8a8ecb7c4c38762514fb2497 Mon Sep 17 00:00:00 2001
From: Stefano Ceccherini <stefano.ceccherini@gmail.com>
Date: Sun, 15 Aug 2010 09:42:53 +0000
Subject: [PATCH] Check for NULL pointers. CID 1589 and 1590.

git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@38107 a95241bf-73f2-0310-859d-f6bbb57e9c96
---
 src/preferences/mail/AutoConfigWindow.cpp | 56 +++++++++++------------
 1 file changed, 28 insertions(+), 28 deletions(-)

diff --git a/src/preferences/mail/AutoConfigWindow.cpp b/src/preferences/mail/AutoConfigWindow.cpp
index 602fa0e635..452a29eefd 100644
--- a/src/preferences/mail/AutoConfigWindow.cpp
+++ b/src/preferences/mail/AutoConfigWindow.cpp
@@ -154,55 +154,55 @@ AutoConfigWindow::QuitRequested(void)
 Account*
 AutoConfigWindow::GenerateBasicAccount()
 {
-	if(!fAccount){
+	if(!fAccount) {
 		fAccount = Accounts::NewAccount();
 		fAccount->SetType(fAccountInfo.type);
 		fAccount->SetName(fAccountInfo.accountName.String());
 		fAccount->SetRealName(fAccountInfo.name.String());
 		fAccount->SetReturnAddress(fAccountInfo.email.String());
 				
-		BMailChain *inbound = fAccount->Inbound();
-
 		BString inServerName;
 		int32 authType = 0;
 		int32 ssl = 0;
 		if (fAccountInfo.inboundType == IMAP) {
 			inServerName = fAccountInfo.providerInfo.imap_server;
 			ssl = fAccountInfo.providerInfo.ssl_imap;
-		}
-		else {
+		} else {
 			inServerName = fAccountInfo.providerInfo.pop_server;
 			authType = fAccountInfo.providerInfo.authentification_pop;
 			ssl = fAccountInfo.providerInfo.ssl_pop;
 		}
 		if (fAccountInfo.type == INBOUND_TYPE
-				|| fAccountInfo.type == IN_AND_OUTBOUND_TYPE)
-		{
-			BMessage inboundArchive;
-			inboundArchive.AddString("server", inServerName);
-			inboundArchive.AddInt32("auth_method", authType);
-			inboundArchive.AddInt32("flavor", ssl);
-			inboundArchive.AddString("username", fAccountInfo.loginName);
-			inboundArchive.AddString("password", fAccountInfo.password);
-			inboundArchive.AddBool("leave_mail_on_server", true);
-			inboundArchive.AddBool("delete_remote_when_local", true);
-			inbound->SetFilter(0, inboundArchive, fAccountInfo.inboundProtocol);
+				|| fAccountInfo.type == IN_AND_OUTBOUND_TYPE) {
+			BMailChain *inbound = fAccount->Inbound();
+			if (inbound != NULL) {
+				BMessage inboundArchive;
+				inboundArchive.AddString("server", inServerName);
+				inboundArchive.AddInt32("auth_method", authType);
+				inboundArchive.AddInt32("flavor", ssl);
+				inboundArchive.AddString("username", fAccountInfo.loginName);
+				inboundArchive.AddString("password", fAccountInfo.password);
+				inboundArchive.AddBool("leave_mail_on_server", true);
+				inboundArchive.AddBool("delete_remote_when_local", true);
+				inbound->SetFilter(0, inboundArchive, fAccountInfo.inboundProtocol);
+			}
 		}
 
 		if (fAccountInfo.type == OUTBOUND_TYPE
-				|| fAccountInfo.type == IN_AND_OUTBOUND_TYPE)
-		{
+				|| fAccountInfo.type == IN_AND_OUTBOUND_TYPE) {
 			BMailChain *outbound = fAccount->Outbound();
-			BMessage outboundArchive;
-			outboundArchive.AddString("server",
-										fAccountInfo.providerInfo.smtp_server);
-			outboundArchive.AddString("username", fAccountInfo.loginName);
-			outboundArchive.AddString("password", fAccountInfo.password);
-			outboundArchive.AddInt32("auth_method",
-										fAccountInfo.providerInfo.authentification_smtp);
-			outboundArchive.AddInt32("flavor", fAccountInfo.providerInfo.ssl_smtp);
-			outbound->SetFilter(1, outboundArchive,
-									fAccountInfo.outboundProtocol);
+			if (outbound != NULL) {
+				BMessage outboundArchive;
+				outboundArchive.AddString("server",
+											fAccountInfo.providerInfo.smtp_server);
+				outboundArchive.AddString("username", fAccountInfo.loginName);
+				outboundArchive.AddString("password", fAccountInfo.password);
+				outboundArchive.AddInt32("auth_method",
+											fAccountInfo.providerInfo.authentification_smtp);
+				outboundArchive.AddInt32("flavor", fAccountInfo.providerInfo.ssl_smtp);
+				outbound->SetFilter(1, outboundArchive,
+										fAccountInfo.outboundProtocol);
+			}
 		}
 	}
 	return fAccount;