]> diplodocus.org Git - nmh/blobdiff - sbr/fmt_rfc2047.c
Reworked attach to add charset to Content-Type string for
[nmh] / sbr / fmt_rfc2047.c
index a87fc0e8680a24740661bdaffc818dd53276379b..5e5a5a8e51c7958afb422d1ed73c99afe8b004a4 100644 (file)
@@ -2,17 +2,15 @@
 /*
  * fmt_rfc2047.c -- decode RFC-2047 header format 
  *
- * $Id$
- *
  * This code is Copyright (c) 2002, by the authors of nmh.  See the
  * COPYRIGHT file in the root directory of the nmh distribution for
  * complete copyright information.
  */
 
 #include <h/mh.h>
+#include <h/utils.h>
 #ifdef HAVE_ICONV
 #  include <iconv.h>
-#  include <errno.h>
 #endif
 
 static signed char hexindex[] = {
@@ -55,11 +53,14 @@ unqp (unsigned char byte1, unsigned char byte2)
  * Decode the string as a RFC-2047 header field
  */
 
+/* Add character to the destination buffer, and bomb out if it fills up */
+#define ADDCHR(C) do { *q++ = (C); dstlen--; if (!dstlen) goto buffull; } while (0)
+
 int
-decode_rfc2047 (char *str, char *dst
+decode_rfc2047 (char *str, char *dst, size_t dstlen)
 {
     char *p, *q, *pp;
-    char *startofmime, *endofmime;
+    char *startofmime, *endofmime, *endofcharset;
     int c, quoted_printable;
     int encoding_found = 0;    /* did we decode anything?                */
     int between_encodings = 0; /* are we between two encodings?          */
@@ -67,9 +68,10 @@ decode_rfc2047 (char *str, char *dst)
     int whitespace = 0;                /* how much whitespace between encodings? */
 #ifdef HAVE_ICONV
     int use_iconv = 0;          /* are we converting encoding with iconv? */
-    iconv_t cd;
-    int fromutf8;
-    char *saveq, *convbuf;
+    iconv_t cd = NULL;
+    int fromutf8 = 0;
+    char *saveq, *convbuf = NULL;
+    size_t savedstlen;
 #endif
 
     if (!str)
@@ -96,7 +98,7 @@ decode_rfc2047 (char *str, char *dst)
         * last iteration, then add it first.
         */
        if (equals_pending) {
-           *q++ = '=';
+           ADDCHR('=');
            equals_pending = 0;
            between_encodings = 0;      /* we have added non-whitespace text */
        }
@@ -107,7 +109,7 @@ decode_rfc2047 (char *str, char *dst)
                whitespace++;
            else
                between_encodings = 0;  /* we have added non-whitespace text */
-           *q++ = *p;
+           ADDCHR(*p);
            continue;
        }
 
@@ -124,11 +126,25 @@ decode_rfc2047 (char *str, char *dst)
            if (!*pp)
                continue;
 
+           /*
+            * RFC 2231 specifies that language information can appear
+            * in a charset specification like so:
+            *
+            * =?us-ascii*en?Q?Foo?=
+            *
+            * Right now we don't use language information, so ignore it.
+            */
+
+           for (endofcharset = startofmime;
+                       *endofcharset != '*' && endofcharset < pp;
+                                                       endofcharset++)
+               ;
+
            /* Check if character set can be handled natively */
-           if (!check_charset(startofmime, pp - startofmime)) {
+           if (!check_charset(startofmime, endofcharset - startofmime)) {
 #ifdef HAVE_ICONV
                /* .. it can't. We'll use iconv then. */
-               *pp = '\0';
+               *endofcharset = '\0';
                cd = iconv_open(get_charset(), startofmime);
                fromutf8 = !strcasecmp(startofmime, "UTF-8");
                *pp = '?';
@@ -185,15 +201,33 @@ decode_rfc2047 (char *str, char *dst)
             * We will roll back the buffer the number of whitespace
             * characters we've seen since last encoded word.
             */
-           if (between_encodings)
+           if (between_encodings) {
                q -= whitespace;
+               dstlen += whitespace;
+           }
 
 #ifdef HAVE_ICONV
+           /*
+            * empty encoded text. This ensures that we don't
+            * malloc 0 bytes but skip on to the end
+            */
+           if (endofmime == startofmime && use_iconv) {
+               use_iconv = 0;
+               iconv_close(cd);
+            }
+
            if (use_iconv) {
-               saveq = q;
-               if (!(q = convbuf = (char *)malloc(endofmime - startofmime)))
-                   continue;
+               saveq = q;
+               savedstlen = dstlen;
+                q = convbuf = (char *) mh_xmalloc(endofmime - startofmime);
             }
+/* ADDCHR2 is for adding characters when q is or might be convbuf:
+ * in this case on buffer-full we want to run iconv before returning.
+ * I apologise for the dreadful name.
+ */
+#define ADDCHR2(C) do { *q++ = (C); dstlen--; if (!dstlen) goto iconvbuffull; } while (0)
+#else
+#define ADDCHR2(C) ADDCHR(C)
 #endif
 
            /* Now decode the text */
@@ -207,14 +241,15 @@ decode_rfc2047 (char *str, char *dst)
                            *q++ = c;
                        pp += 2;
                    } else if (*pp == '_') {
-                       *q++ = ' ';
+                       ADDCHR2(' ');
                    } else {
-                       *q++ = *pp;
+                       ADDCHR2(*pp);
                    }
                }
            } else {
                /* base64 */
                int c1, c2, c3, c4;
+               c1 = c2 = c3 = c4 = -1;
 
                pp = startofmime;
                while (pp < endofmime) {
@@ -231,7 +266,7 @@ decode_rfc2047 (char *str, char *dst)
                        pp++;
                    }
                    if (pp < endofmime && c1 != -1 && c2 != -1) {
-                       *q++ = (c1 << 2) | (c2 >> 4);
+                       ADDCHR2((c1 << 2) | (c2 >> 4));
                        pp++;
                    }
                    /* 4 + 4 bits */
@@ -240,7 +275,7 @@ decode_rfc2047 (char *str, char *dst)
                        pp++;
                    }
                    if (pp < endofmime && c2 != -1 && c3 != -1) {
-                       *q++ = ((c2 & 0xF) << 4) | (c3 >> 2);
+                       ADDCHR2(((c2 & 0xF) << 4) | (c3 >> 2));
                        pp++;
                    }
                    /* 2 + 6 bits */
@@ -249,37 +284,56 @@ decode_rfc2047 (char *str, char *dst)
                        pp++;
                    }
                    if (pp < endofmime && c3 != -1 && c4 != -1) {
-                       *q++ = ((c3 & 0x3) << 6) | (c4);
+                       ADDCHR2(((c3 & 0x3) << 6) | (c4));
                        pp++;
                    }
                }
            }
 
 #ifdef HAVE_ICONV
+       iconvbuffull:
+           /* NB that the string at convbuf is not necessarily NUL terminated here:
+            * q points to the first byte after the valid part.
+            */
             /* Convert to native character set */
            if (use_iconv) {
                size_t inbytes = q - convbuf;
-               size_t outbytes = BUFSIZ;
                ICONV_CONST char *start = convbuf;
                
                while (inbytes) {
-                   if (iconv(cd, &start, &inbytes, &saveq, &outbytes) ==
+                   if (iconv(cd, &start, &inbytes, &saveq, &savedstlen) ==
                            (size_t)-1) {
                        if (errno != EILSEQ) break;
                        /* character couldn't be converted. we output a `?'
                         * and try to carry on which won't work if
                         * either encoding was stateful */
-                       iconv (cd, 0, 0, &saveq, &outbytes);
+                       iconv (cd, 0, 0, &saveq, &savedstlen);
+                       if (!savedstlen)
+                           break;
                        *saveq++ = '?';
-                        /* skip to next input character */
+                       savedstlen--;
+                       if (!savedstlen)
+                           break;
+                       /* skip to next input character */
                        if (fromutf8) {
-                           for (start++;(*start & 192) == 128;start++)
-                               inbytes--;
+                           for (++start, --inbytes;
+                                start < q  &&  (*start & 192) == 128;
+                                ++start, --inbytes)
+                               continue;
                        } else
                            start++, inbytes--;
+                       if (start >= q)
+                           break;
                    }
                }
                q = saveq;
+               /* Stop now if (1) we hit the end of the buffer trying to do
+                * MIME decoding and have just iconv-converted a partial string
+                * or (2) our iconv-conversion hit the end of the buffer.
+                */
+               if (!dstlen || !savedstlen)
+                   goto buffull;
+               dstlen = savedstlen;
                free(convbuf);
            }
 #endif
@@ -301,8 +355,14 @@ decode_rfc2047 (char *str, char *dst)
 
     /* If an equals was pending at end of string, add it now. */
     if (equals_pending)
-       *q++ = '=';
+       ADDCHR('=');
     *q = '\0';
 
     return encoding_found;
+
+  buffull:
+    /* q is currently just off the end of the buffer, so rewind to NUL terminate */
+    q--;
+    *q = '\0';
+    return encoding_found;
 }