fetchBody in POP3 & weird segfault

6 files changed, 158 insertions, 98 deletions

diff --git a/noncore/net/mail/accountview.cpp b/noncore/net/mail/accountview.cpp
index 6963027..c7b1eeb 100644
--- a/noncore/net/mail/accountview.cpp
+++ b/noncore/net/mail/accountview.cpp
@@ -30,10 +30,10 @@ void POP3viewItem::refresh( QList<RecMail> &target )
 }
 
 
-RecBody POP3viewItem::fetchBody( const RecMail & )
+RecBody POP3viewItem::fetchBody( const RecMail &mail )
 {
-    qDebug( "POP3 fetchBody: IMPLEMENT ME!!" );
-    return RecBody();
+    qDebug( "POP3 fetchBody" );
+    return wrapper->fetchBody( mail );
 }
 
 /**
diff --git a/noncore/net/mail/accountview.h b/noncore/net/mail/accountview.h
index fe832ec..4cac673 100644
--- a/noncore/net/mail/accountview.h
+++ b/noncore/net/mail/accountview.h
@@ -29,7 +29,7 @@ public:
     POP3viewItem( POP3account *a, QListView *parent );
     ~POP3viewItem();
     virtual void refresh( QList<RecMail> &target );
-    virtual RecBody fetchBody(const RecMail & );
+    virtual RecBody fetchBody( const RecMail &mail );
 
 private:
     POP3account *account;
diff --git a/noncore/net/mail/libmailwrapper/pop3wrapper.cpp b/noncore/net/mail/libmailwrapper/pop3wrapper.cpp
index 633faf6..abb5a42 100644
--- a/noncore/net/mail/libmailwrapper/pop3wrapper.cpp
+++ b/noncore/net/mail/libmailwrapper/pop3wrapper.cpp
@@ -19,6 +19,46 @@ void POP3wrapper::pop3_progress( size_t current, size_t maximum )
     qDebug( "POP3: %i of %i", current, maximum );
 }
 
+RecBody POP3wrapper::fetchBody( const RecMail &mail )
+{
+    int err = MAILPOP3_NO_ERROR;
+    char *message;
+    size_t length;
+
+    login();
+    if ( !m_pop3 ) return RecBody();
+    
+    err = mailpop3_retr( m_pop3, mail.getNumber(), &message, &length );
+    if ( err != MAILPOP3_NO_ERROR ) {
+        qDebug( "POP3: error retrieving body with index %i", mail.getNumber() );
+        logout();
+        return RecBody();
+    }
+    
+    logout();
+
+    return parseBody( message );
+}
+
+RecBody POP3wrapper::parseBody( const char *message )
+{
+    int err = MAILIMF_NO_ERROR;
+    size_t curTok;
+    mailimf_message *result;
+    RecBody body;
+    
+    err = mailimf_message_parse( (char *) message, strlen( message ), &curTok, &result );
+    if ( err != MAILIMF_NO_ERROR ) return body;
+    
+    if ( result && result->msg_body && result->msg_body->bd_text ) {
+        qDebug( "POP3: bodytext found" );
+        // TODO: why does this line segfault???? gdb says segfault in strlen(), maybe a bug in libetpan.
+        body.setBodytext( QString( result->msg_body->bd_text ) );
+    }
+
+    return body;
+}
+
 void POP3wrapper::listMessages( QList<RecMail> &target )
 {
     int err = MAILPOP3_NO_ERROR;
@@ -59,22 +99,22 @@ RecMail *POP3wrapper::parseHeader( const char *header )
         mailimf_field *field = (mailimf_field *) current->data;
         switch ( field->fld_type ) {
             case MAILIMF_FIELD_FROM:
-                mail->setFrom( *parseMailboxList( field->fld_data.fld_from->frm_mb_list ) );
+                mail->setFrom( parseMailboxList( field->fld_data.fld_from->frm_mb_list ) );
                 break;
             case MAILIMF_FIELD_TO:
-                mail->setTo( *parseAddressList( field->fld_data.fld_to->to_addr_list ) );
+                mail->setTo( parseAddressList( field->fld_data.fld_to->to_addr_list ) );
                 break;
             case MAILIMF_FIELD_CC:
-                mail->setCC( *parseAddressList( field->fld_data.fld_cc->cc_addr_list ) );
+                mail->setCC( parseAddressList( field->fld_data.fld_cc->cc_addr_list ) );
                 break;
             case MAILIMF_FIELD_BCC:
-                mail->setBcc( *parseAddressList( field->fld_data.fld_bcc->bcc_addr_list ) );
+                mail->setBcc( parseAddressList( field->fld_data.fld_bcc->bcc_addr_list ) );
                 break;
             case MAILIMF_FIELD_SUBJECT:
                 mail->setSubject( QString( field->fld_data.fld_subject->sbj_value ) );
                 break;
             case MAILIMF_FIELD_ORIG_DATE:
-                mail->setDate( *parseDateTime( field->fld_data.fld_orig_date->dt_date_time ) );
+                mail->setDate( parseDateTime( field->fld_data.fld_orig_date->dt_date_time ) );
                 break;
             default:
                 break;
@@ -84,44 +124,36 @@ RecMail *POP3wrapper::parseHeader( const char *header )
     return mail;
 }
 
-QString *POP3wrapper::parseDateTime( mailimf_date_time *date )
+QString POP3wrapper::parseDateTime( mailimf_date_time *date )
 {
     char tmp[23];
     
     snprintf( tmp, 23,  "%02i.%02i.%04i %02i:%02i:%02i %+05i", 
         date->dt_day, date->dt_month, date->dt_year, date->dt_hour, date->dt_min, date->dt_sec, date->dt_zone );
     
-    QString *result = new QString( tmp );
-
-    return result;
+    return QString( tmp );
 }
 
-QString *POP3wrapper::parseAddressList( mailimf_address_list *list )
+QString POP3wrapper::parseAddressList( mailimf_address_list *list )
 {
-    QString *result = new QString( "" );
+    QString result( "" );
 
     bool first = true;
     for ( clistiter *current = clist_begin( list->ad_list ); current != NULL; current = current->next ) {
         mailimf_address *addr = (mailimf_address *) current->data;
         
         if ( !first ) {
-            result->append( "," );
+            result.append( "," );
         } else {
             first = false;
         }
 
-        QString *tmp;
-        
         switch ( addr->ad_type ) {
             case MAILIMF_ADDRESS_MAILBOX:
-                tmp = parseMailbox( addr->ad_data.ad_mailbox );
-                result->append( *tmp );
-                delete tmp;
+                result.append( parseMailbox( addr->ad_data.ad_mailbox ) );
                 break;
             case MAILIMF_ADDRESS_GROUP:
-                tmp = parseGroup( addr->ad_data.ad_group );
-                result->append( *tmp );
-                delete tmp;
+                result.append( parseGroup( addr->ad_data.ad_group ) );
                 break;
             default:
                 qDebug( "POP3: unkown mailimf address type" );
@@ -132,57 +164,53 @@ QString *POP3wrapper::parseAddressList( mailimf_address_list *list )
     return result;
 }
 
-QString *POP3wrapper::parseGroup( mailimf_group *group )
+QString POP3wrapper::parseGroup( mailimf_group *group )
 {
-    QString *result =  new QString( "" );
+    QString result( "" );
 
-    result->append( group->grp_display_name );
-    result->append( ": " );
+    result.append( group->grp_display_name );
+    result.append( ": " );
 
     if ( group->grp_mb_list != NULL ) {
-        QString *tmp = parseMailboxList( group->grp_mb_list );
-        result->append( *tmp );
-        delete tmp;
+        result.append( parseMailboxList( group->grp_mb_list ) );
     }
 
-    result->append( ";" );
+    result.append( ";" );
     
     return result;
 }
 
-QString *POP3wrapper::parseMailbox( mailimf_mailbox *box )
+QString POP3wrapper::parseMailbox( mailimf_mailbox *box )
 {
-    QString *result =  new QString( "" );
+    QString result( "" );
 
     if ( box->mb_display_name == NULL ) {
-        result->append( box->mb_addr_spec );
+        result.append( box->mb_addr_spec );
     } else {
-        result->append( box->mb_display_name );
-        result->append( " <" );
-        result->append( box->mb_addr_spec );
-        result->append( ">" );
+        result.append( box->mb_display_name );
+        result.append( " <" );
+        result.append( box->mb_addr_spec );
+        result.append( ">" );
     }
         
     return result;
 }
 
-QString *POP3wrapper::parseMailboxList( mailimf_mailbox_list *list )
+QString POP3wrapper::parseMailboxList( mailimf_mailbox_list *list )
 {
-    QString *result =  new QString( "" );
+    QString result( "" );
 
     bool first = true;
     for ( clistiter *current = clist_begin( list->mb_list ); current != NULL; current = current->next ) {
         mailimf_mailbox *box = (mailimf_mailbox *) current->data;
 
         if ( !first ) {
-            result->append( "," );
+            result.append( "," );
         } else {
             first = false;
         }
        
-        QString *tmp = parseMailbox( box );
-        result->append( *tmp );
-        delete tmp;
+        result.append( parseMailbox( box ) );
     }
     
     return result;
@@ -240,4 +268,3 @@ void POP3wrapper::logout()
     m_pop3 = NULL;
 }
 
-
diff --git a/noncore/net/mail/libmailwrapper/pop3wrapper.h b/noncore/net/mail/libmailwrapper/pop3wrapper.h
index 4e4abad..995bed0 100644
--- a/noncore/net/mail/libmailwrapper/pop3wrapper.h
+++ b/noncore/net/mail/libmailwrapper/pop3wrapper.h
@@ -4,6 +4,7 @@
 #include "mailwrapper.h"
 
 class RecMail;
+class RecBody;
 struct mailpop3;
 
 class POP3wrapper : public QObject
@@ -14,6 +15,7 @@ public:
     POP3wrapper( POP3account *a );
     virtual ~POP3wrapper();
     void listMessages( QList<RecMail> &target );
+    RecBody fetchBody( const RecMail &mail );
     static void pop3_progress( size_t current, size_t maximum );
 
 protected:
@@ -22,11 +24,12 @@ protected:
 
 private:
     RecMail *parseHeader( const char *header );
-    QString *parseMailboxList( mailimf_mailbox_list *list );
-    QString *parseMailbox( mailimf_mailbox *box );
-    QString *parseGroup( mailimf_group *group );
-    QString *parseAddressList( mailimf_address_list *list );
-    QString *parseDateTime( mailimf_date_time *date );
+    RecBody parseBody( const char *message );
+    QString parseMailboxList( mailimf_mailbox_list *list );
+    QString parseMailbox( mailimf_mailbox *box );
+    QString parseGroup( mailimf_group *group );
+    QString parseAddressList( mailimf_address_list *list );
+    QString parseDateTime( mailimf_date_time *date );
     POP3account *account;
     mailpop3 *m_pop3;
 
diff --git a/noncore/net/mail/pop3wrapper.cpp b/noncore/net/mail/pop3wrapper.cpp
index 633faf6..abb5a42 100644
--- a/noncore/net/mail/pop3wrapper.cpp
+++ b/noncore/net/mail/pop3wrapper.cpp
@@ -19,6 +19,46 @@ void POP3wrapper::pop3_progress( size_t current, size_t maximum )
     qDebug( "POP3: %i of %i", current, maximum );
 }
 
+RecBody POP3wrapper::fetchBody( const RecMail &mail )
+{
+    int err = MAILPOP3_NO_ERROR;
+    char *message;
+    size_t length;
+
+    login();
+    if ( !m_pop3 ) return RecBody();
+    
+    err = mailpop3_retr( m_pop3, mail.getNumber(), &message, &length );
+    if ( err != MAILPOP3_NO_ERROR ) {
+        qDebug( "POP3: error retrieving body with index %i", mail.getNumber() );
+        logout();
+        return RecBody();
+    }
+    
+    logout();
+
+    return parseBody( message );
+}
+
+RecBody POP3wrapper::parseBody( const char *message )
+{
+    int err = MAILIMF_NO_ERROR;
+    size_t curTok;
+    mailimf_message *result;
+    RecBody body;
+    
+    err = mailimf_message_parse( (char *) message, strlen( message ), &curTok, &result );
+    if ( err != MAILIMF_NO_ERROR ) return body;
+    
+    if ( result && result->msg_body && result->msg_body->bd_text ) {
+        qDebug( "POP3: bodytext found" );
+        // TODO: why does this line segfault???? gdb says segfault in strlen(), maybe a bug in libetpan.
+        body.setBodytext( QString( result->msg_body->bd_text ) );
+    }
+
+    return body;
+}
+
 void POP3wrapper::listMessages( QList<RecMail> &target )
 {
     int err = MAILPOP3_NO_ERROR;
@@ -59,22 +99,22 @@ RecMail *POP3wrapper::parseHeader( const char *header )
         mailimf_field *field = (mailimf_field *) current->data;
         switch ( field->fld_type ) {
             case MAILIMF_FIELD_FROM:
-                mail->setFrom( *parseMailboxList( field->fld_data.fld_from->frm_mb_list ) );
+                mail->setFrom( parseMailboxList( field->fld_data.fld_from->frm_mb_list ) );
                 break;
             case MAILIMF_FIELD_TO:
-                mail->setTo( *parseAddressList( field->fld_data.fld_to->to_addr_list ) );
+                mail->setTo( parseAddressList( field->fld_data.fld_to->to_addr_list ) );
                 break;
             case MAILIMF_FIELD_CC:
-                mail->setCC( *parseAddressList( field->fld_data.fld_cc->cc_addr_list ) );
+                mail->setCC( parseAddressList( field->fld_data.fld_cc->cc_addr_list ) );
                 break;
             case MAILIMF_FIELD_BCC:
-                mail->setBcc( *parseAddressList( field->fld_data.fld_bcc->bcc_addr_list ) );
+                mail->setBcc( parseAddressList( field->fld_data.fld_bcc->bcc_addr_list ) );
                 break;
             case MAILIMF_FIELD_SUBJECT:
                 mail->setSubject( QString( field->fld_data.fld_subject->sbj_value ) );
                 break;
             case MAILIMF_FIELD_ORIG_DATE:
-                mail->setDate( *parseDateTime( field->fld_data.fld_orig_date->dt_date_time ) );
+                mail->setDate( parseDateTime( field->fld_data.fld_orig_date->dt_date_time ) );
                 break;
             default:
                 break;
@@ -84,44 +124,36 @@ RecMail *POP3wrapper::parseHeader( const char *header )
     return mail;
 }
 
-QString *POP3wrapper::parseDateTime( mailimf_date_time *date )
+QString POP3wrapper::parseDateTime( mailimf_date_time *date )
 {
     char tmp[23];
     
     snprintf( tmp, 23,  "%02i.%02i.%04i %02i:%02i:%02i %+05i", 
         date->dt_day, date->dt_month, date->dt_year, date->dt_hour, date->dt_min, date->dt_sec, date->dt_zone );
     
-    QString *result = new QString( tmp );
-
-    return result;
+    return QString( tmp );
 }
 
-QString *POP3wrapper::parseAddressList( mailimf_address_list *list )
+QString POP3wrapper::parseAddressList( mailimf_address_list *list )
 {
-    QString *result = new QString( "" );
+    QString result( "" );
 
     bool first = true;
     for ( clistiter *current = clist_begin( list->ad_list ); current != NULL; current = current->next ) {
         mailimf_address *addr = (mailimf_address *) current->data;
         
         if ( !first ) {
-            result->append( "," );
+            result.append( "," );
         } else {
             first = false;
         }
 
-        QString *tmp;
-        
         switch ( addr->ad_type ) {
             case MAILIMF_ADDRESS_MAILBOX:
-                tmp = parseMailbox( addr->ad_data.ad_mailbox );
-                result->append( *tmp );
-                delete tmp;
+                result.append( parseMailbox( addr->ad_data.ad_mailbox ) );
                 break;
             case MAILIMF_ADDRESS_GROUP:
-                tmp = parseGroup( addr->ad_data.ad_group );
-                result->append( *tmp );
-                delete tmp;
+                result.append( parseGroup( addr->ad_data.ad_group ) );
                 break;
             default:
                 qDebug( "POP3: unkown mailimf address type" );
@@ -132,57 +164,53 @@ QString *POP3wrapper::parseAddressList( mailimf_address_list *list )
     return result;
 }
 
-QString *POP3wrapper::parseGroup( mailimf_group *group )
+QString POP3wrapper::parseGroup( mailimf_group *group )
 {
-    QString *result =  new QString( "" );
+    QString result( "" );
 
-    result->append( group->grp_display_name );
-    result->append( ": " );
+    result.append( group->grp_display_name );
+    result.append( ": " );
 
     if ( group->grp_mb_list != NULL ) {
-        QString *tmp = parseMailboxList( group->grp_mb_list );
-        result->append( *tmp );
-        delete tmp;
+        result.append( parseMailboxList( group->grp_mb_list ) );
     }
 
-    result->append( ";" );
+    result.append( ";" );
     
     return result;
 }
 
-QString *POP3wrapper::parseMailbox( mailimf_mailbox *box )
+QString POP3wrapper::parseMailbox( mailimf_mailbox *box )
 {
-    QString *result =  new QString( "" );
+    QString result( "" );
 
     if ( box->mb_display_name == NULL ) {
-        result->append( box->mb_addr_spec );
+        result.append( box->mb_addr_spec );
     } else {
-        result->append( box->mb_display_name );
-        result->append( " <" );
-        result->append( box->mb_addr_spec );
-        result->append( ">" );
+        result.append( box->mb_display_name );
+        result.append( " <" );
+        result.append( box->mb_addr_spec );
+        result.append( ">" );
     }
         
     return result;
 }
 
-QString *POP3wrapper::parseMailboxList( mailimf_mailbox_list *list )
+QString POP3wrapper::parseMailboxList( mailimf_mailbox_list *list )
 {
-    QString *result =  new QString( "" );
+    QString result( "" );
 
     bool first = true;
     for ( clistiter *current = clist_begin( list->mb_list ); current != NULL; current = current->next ) {
         mailimf_mailbox *box = (mailimf_mailbox *) current->data;
 
         if ( !first ) {
-            result->append( "," );
+            result.append( "," );
         } else {
             first = false;
         }
        
-        QString *tmp = parseMailbox( box );
-        result->append( *tmp );
-        delete tmp;
+        result.append( parseMailbox( box ) );
     }
     
     return result;
@@ -240,4 +268,3 @@ void POP3wrapper::logout()
     m_pop3 = NULL;
 }
 
-
diff --git a/noncore/net/mail/pop3wrapper.h b/noncore/net/mail/pop3wrapper.h
index 4e4abad..995bed0 100644
--- a/noncore/net/mail/pop3wrapper.h
+++ b/noncore/net/mail/pop3wrapper.h
@@ -4,6 +4,7 @@
 #include "mailwrapper.h"
 
 class RecMail;
+class RecBody;
 struct mailpop3;
 
 class POP3wrapper : public QObject
@@ -14,6 +15,7 @@ public:
     POP3wrapper( POP3account *a );
     virtual ~POP3wrapper();
     void listMessages( QList<RecMail> &target );
+    RecBody fetchBody( const RecMail &mail );
     static void pop3_progress( size_t current, size_t maximum );
 
 protected:
@@ -22,11 +24,12 @@ protected:
 
 private:
     RecMail *parseHeader( const char *header );
-    QString *parseMailboxList( mailimf_mailbox_list *list );
-    QString *parseMailbox( mailimf_mailbox *box );
-    QString *parseGroup( mailimf_group *group );
-    QString *parseAddressList( mailimf_address_list *list );
-    QString *parseDateTime( mailimf_date_time *date );
+    RecBody parseBody( const char *message );
+    QString parseMailboxList( mailimf_mailbox_list *list );
+    QString parseMailbox( mailimf_mailbox *box );
+    QString parseGroup( mailimf_group *group );
+    QString parseAddressList( mailimf_address_list *list );
+    QString parseDateTime( mailimf_date_time *date );
     POP3account *account;
     mailpop3 *m_pop3;