summaryrefslogtreecommitdiff
authoralwin <alwin>2004-10-23 19:42:19 (UTC)
committer alwin <alwin>2004-10-23 19:42:19 (UTC)
commitb3d4cb2af9cd4b3a56b581ba89a4c9fa0d9312af (patch) (unidiff)
tree993e629f5f8c1dcf66e1c294d840cc4aa0100ae1
parentb5d111eb79896d02aa13d41a04ce6aa4dbc4aa49 (diff)
downloadopie-b3d4cb2af9cd4b3a56b581ba89a4c9fa0d9312af.zip
opie-b3d4cb2af9cd4b3a56b581ba89a4c9fa0d9312af.tar.gz
opie-b3d4cb2af9cd4b3a56b581ba89a4c9fa0d9312af.tar.bz2
holla - fixed a wonderfull bug letting opie-mail crash when
accessing some pop3-server
Diffstat (more/less context) (ignore whitespace changes)
-rw-r--r--noncore/net/mail/libmailwrapper/genericwrapper.cpp15
1 files changed, 7 insertions, 8 deletions
diff --git a/noncore/net/mail/libmailwrapper/genericwrapper.cpp b/noncore/net/mail/libmailwrapper/genericwrapper.cpp
index 3e4293b..1caa375 100644
--- a/noncore/net/mail/libmailwrapper/genericwrapper.cpp
+++ b/noncore/net/mail/libmailwrapper/genericwrapper.cpp
@@ -138,5 +138,5 @@ void Genericwrapper::traverseBody(RecBodyP&target,mailmessage*message,mailmime*m
138{ 138{
139 if (current_rec >= 10) { 139 if (current_rec >= 10) {
140 odebug << "too deep recursion!" << oendl; 140 odebug << "too deep recursion!" << oendl;
141 } 141 }
142 if (!message || !mime) { 142 if (!message || !mime) {
@@ -275,5 +275,5 @@ QString Genericwrapper::parseAddressList( mailimf_address_list *list )
275 break; 275 break;
276 default: 276 default:
277 odebug << "Generic: unkown mailimf address type" << oendl; 277 odebug << "Generic: unkown mailimf address type" << oendl;
278 break; 278 break;
279 } 279 }
@@ -368,5 +368,5 @@ void Genericwrapper::cleanMimeCache()
368 } 368 }
369 bodyCache.clear(); 369 bodyCache.clear();
370 odebug << "Genericwrapper: cache cleaned" << oendl; 370 odebug << "Genericwrapper: cache cleaned" << oendl;
371} 371}
372 372
@@ -397,10 +397,10 @@ void Genericwrapper::parseList(QValueList<Opie::Core::OSmartPointer<RecMail> > &
397 r = mailsession_get_messages_list(session,&env_list); 397 r = mailsession_get_messages_list(session,&env_list);
398 if (r != MAIL_NO_ERROR) { 398 if (r != MAIL_NO_ERROR) {
399 odebug << "Error message list" << oendl; 399 odebug << "Error message list" << oendl;
400 return; 400 return;
401 } 401 }
402 r = mailsession_get_envelopes_list(session, env_list); 402 r = mailsession_get_envelopes_list(session, env_list);
403 if (r != MAIL_NO_ERROR) { 403 if (r != MAIL_NO_ERROR) {
404 odebug << "Error filling message list" << oendl; 404 odebug << "Error filling message list" << oendl;
405 if (env_list) { 405 if (env_list) {
406 mailmessage_list_free(env_list); 406 mailmessage_list_free(env_list);
@@ -416,5 +416,5 @@ void Genericwrapper::parseList(QValueList<Opie::Core::OSmartPointer<RecMail> > &
416 msg = (mailmessage*)carray_get(env_list->msg_tab, i); 416 msg = (mailmessage*)carray_get(env_list->msg_tab, i);
417 if (msg->msg_fields == NULL) { 417 if (msg->msg_fields == NULL) {
418 //odebug << "could not fetch envelope of message " << i << "" << oendl; 418 //odebug << "could not fetch envelope of message " << i << "" << oendl;
419 continue; 419 continue;
420 } 420 }
@@ -449,7 +449,6 @@ void Genericwrapper::parseList(QValueList<Opie::Core::OSmartPointer<RecMail> > &
449 mail->setDate( parseDateTime( single_fields.fld_orig_date->dt_date_time ) ); 449 mail->setDate( parseDateTime( single_fields.fld_orig_date->dt_date_time ) );
450 // crashes when accessing pop3 account? 450 // crashes when accessing pop3 account?
451 if (single_fields.fld_message_id->mid_value) { 451 if (single_fields.fld_message_id && single_fields.fld_message_id->mid_value) {
452 mail->setMsgid(QString(single_fields.fld_message_id->mid_value)); 452 mail->setMsgid(QString(single_fields.fld_message_id->mid_value));
453 odebug << "Msgid == " << mail->Msgid().latin1() << "" << oendl;
454 } 453 }
455 454