talons

commit 42cbb5fdf01b3dd62ad047473cc11e0a2e92f4bc
parent f3895e2fff2902b5cbca3239d1c77a83236152f0
Author: Andrej Kacian <andrej@kacian.sk>
Date:   Sun, 12 May 2013 15:09:52 +0200

Do not try to destroy a NULL session after an unsuccesful NNTP connect attempt.

Diffstat:
M
src/news.c
 | 
6
++++--

1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/news.c b/src/news.c
@@ -352,7 +352,8 @@ static Session *news_session_new(Folder *folder, const gchar *server, gushort po
 	
 	if (r != NEWSNNTP_NO_ERROR) {
 		log_error(LOG_PROTOCOL, _("Error logging in to %s:%d...\n"), server, port);
-		session_destroy(SESSION(session));
+		if (session != NULL)
+			session_destroy(SESSION(session));
 		return NULL;
 	}
 
@@ -443,7 +444,8 @@ static Session *news_session_new_for_folder(Folder *folder)
 	    else {
 	        /* An error state bail out */
 	        log_error(LOG_PROTOCOL, _("Error creating session with %s:%d\n"), ac->nntp_server, port);
-		session_destroy(SESSION(session));
+		if (session != NULL)
+			session_destroy(SESSION(session));
 		g_free(passwd);
 		if (ac->session_passwd) {
 			g_free(ac->session_passwd);