2007-11-07 [colin] 3.0.2cvs121
authorColin Leroy <colin@colino.net>
Wed, 7 Nov 2007 07:34:54 +0000 (07:34 +0000)
committerColin Leroy <colin@colino.net>
Wed, 7 Nov 2007 07:34:54 +0000 (07:34 +0000)
* src/plugins/pgpinline/pgpinline.c
Fix bug 1377, 'unspecific errors on
signature checking'. Fix trimming of
UTF8 data.

ChangeLog
PATCHSETS
configure.ac
src/plugins/pgpinline/pgpinline.c

index f8622a1..45fc746 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2007-11-07 [colin]     3.0.2cvs121
+
+       * src/plugins/pgpinline/pgpinline.c
+               Fix bug 1377, 'unspecific errors on 
+               signature checking'. Fix trimming of
+               UTF8 data.
+
 2007-11-06 [iwkse]     3.0.2cvs120
 
        * src/prefs_summary_open.c
index edf2ea2..ecaf335 100644 (file)
--- a/PATCHSETS
+++ b/PATCHSETS
 ( cvs diff -u -r 1.179.2.196 -r 1.179.2.197 src/imap.c;  ) > 3.0.2cvs118.patchset
 ( cvs diff -u -r 1.25.2.23 -r 1.25.2.24 tools/Makefile.am;  cvs diff -u -r 1.30.2.25 -r 1.30.2.26 tools/README;  diff -u /dev/null tools/csv2addressbook.pl;  ) > 3.0.2cvs119.patchset
 ( cvs diff -u -r 1.1.2.4 -r 1.1.2.5 src/prefs_summary_open.c;  ) > 3.0.2cvs120.patchset
+( cvs diff -u -r 1.1.2.36 -r 1.1.2.37 src/plugins/pgpinline/pgpinline.c;  ) > 3.0.2cvs121.patchset
index 997a2b5..e9c3bc1 100644 (file)
@@ -11,7 +11,7 @@ MINOR_VERSION=0
 MICRO_VERSION=2
 INTERFACE_AGE=0
 BINARY_AGE=0
-EXTRA_VERSION=120
+EXTRA_VERSION=121
 EXTRA_RELEASE=
 EXTRA_GTK2_VERSION=
 
index e5ba000..fe34dbe 100644 (file)
@@ -157,7 +157,11 @@ static gchar *get_part_as_string(MimeInfo *mimeinfo)
                        textdata = tmp;
                }
        }
-       if (textdata && mimeinfo->offset && 
+
+       if (textdata && mimeinfo->offset &&
+           mimeinfo->offset+mimeinfo->length == g_utf8_strlen(textdata, -1)) {
+               /* textdata is OK */    
+       } else if (textdata && mimeinfo->offset && 
            mimeinfo->offset+ mimeinfo->length <= strlen(textdata)) {
                real_data = g_strdup(textdata + mimeinfo->offset);
                real_data[mimeinfo->length] = '\0';
@@ -234,16 +238,20 @@ static gint pgpinline_check_signature(MimeInfo *mimeinfo)
 
        g_return_val_if_fail(mimeinfo != NULL, 0);
 
-       if (procmime_mimeinfo_parent(mimeinfo) == NULL)
+       if (procmime_mimeinfo_parent(mimeinfo) == NULL) {
+               privacy_set_error(_("Incorrect part"));
                return 0; /* not parent */
-       if (mimeinfo->type != MIMETYPE_TEXT)
+       }
+       if (mimeinfo->type != MIMETYPE_TEXT) {
+               privacy_set_error(_("Not a text part"));
+               debug_print("type %d\n", mimeinfo->type);
                return 0;
-
+       }
        g_return_val_if_fail(mimeinfo->privacy != NULL, 0);
        data = (PrivacyDataPGP *) mimeinfo->privacy;
 
        textdata = get_part_as_string(mimeinfo);
-       
+
        if (!textdata) {
                g_free(textdata);
                privacy_set_error(_("Couldn't get text data."));