Fix comment indentation.
[gpgme.git] / gpgme / conversion.c
index 36b11fd..5b2cb67 100644 (file)
@@ -1,23 +1,22 @@
 /* conversion.c - String conversion helper functions.
 /* conversion.c - String conversion helper functions.
- *     Copyright (C) 2000 Werner Koch (dd9jn)
- *      Copyright (C) 2001, 2002 g10 Code GmbH
- *
- * This file is part of GPGME.
- *
- * GPGME is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * GPGME is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA
- */
+   Copyright (C) 2000 Werner Koch (dd9jn)
+   Copyright (C) 2001, 2002, 2003 g10 Code GmbH
+   This file is part of GPGME.
+
+   GPGME is free software; you can redistribute it and/or modify it
+   under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
+   GPGME is distributed in the hope that it will be useful, but
+   WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   General Public License for more details.
+   You should have received a copy of the GNU General Public License
+   along with GPGME; if not, write to the Free Software Foundation,
+   Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 
 #if HAVE_CONFIG_H
 #include <config.h>
 
 #if HAVE_CONFIG_H
 #include <config.h>
 #include <errno.h>
 #include <ctype.h>
 #include <sys/types.h>
 #include <errno.h>
 #include <ctype.h>
 #include <sys/types.h>
+#include <stdlib.h>
 
 #include "gpgme.h"
 #include "util.h"
 
 
 #include "gpgme.h"
 #include "util.h"
 
-
+\f
+/* Convert two hexadecimal digits from STR to the value they
+   represent.  Returns -1 if one of the characters is not a
+   hexadecimal digit.  */
 int
 int
-_gpgme_hextobyte (const byte *str)
+_gpgme_hextobyte (const unsigned char *str)
 {
   int val = 0;
   int i;
 {
   int val = 0;
   int i;
@@ -57,84 +60,100 @@ _gpgme_hextobyte (const byte *str)
 }
 
 
 }
 
 
+/* Decode the C formatted string SRC and store the result in the
+   buffer *DESTP which is LEN bytes long.  If LEN is zero, then a
+   large enough buffer is allocated with malloc and *DESTP is set to
+   the result.  Currently, LEN is only used to specify if allocation
+   is desired or not, the caller is expected to make sure that *DESTP
+   is large enough if LEN is not zero.  */
 GpgmeError
 GpgmeError
-_gpgme_decode_c_string (const char *src, char **destp)
+_gpgme_decode_c_string (const char *src, char **destp, int len)
 {
   char *dest;
 
 {
   char *dest;
 
-  /* We can malloc a buffer of the same length, because the converted
-     string will never be larger.  */
-  dest = malloc (strlen (src) + 1);
-  if (!dest)
-    return mk_error (Out_Of_Core);
+  /* Set up the destination buffer.  */
+  if (len)
+    {
+      if (len < strlen (src) + 1)
+       return GPGME_General_Error;
+
+      dest = *destp;
+    }
+  else
+    {
+      /* The converted string will never be larger than the original
+        string.  */
+      dest = malloc (strlen (src) + 1);
+      if (!dest)
+       return GPGME_Out_Of_Core;
 
 
-  *destp = dest;
+      *destp = dest;
+    }
 
 
+  /* Convert the string.  */
   while (*src)
     {
       if (*src != '\\')
   while (*src)
     {
       if (*src != '\\')
-       *(dest++) = *(src++);
-      else if (src[1] == '\\')
        {
        {
-         src++;
-         *(dest++) = *(src++); 
-        }
-      else if (src[1] == 'n')
-       {
-         src += 2;
-         *(dest++) = '\n'; 
-        }
-      else if (src[1] == 'r')
-       {
-         src += 2;
-         *(dest++) = '\r'; 
-        }
-      else if (src[1] == 'v')
-       {
-         src += 2;
-         *(dest++) = '\v'; 
-        }
-      else if (src[1] == 'b')
-       {
-         src += 2;
-         *(dest++) = '\b'; 
-        }
-      else if (src[1] == '0')
-       {
-         /* Hmmm: no way to express this */
-         src += 2;
-         *(dest++) = '\\';
-         *(dest++) = '\0'; 
-        }
-      else if (src[1] == 'x' && isxdigit (src[2]) && isxdigit (src[3]))
-       {
-         int val = _gpgme_hextobyte (&src[2]);
-         if (val == -1)
-           {
-             /* Should not happen.  */
-             *(dest++) = *(src++);
-             *(dest++) = *(src++);
-             *(dest++) = *(src++);
-             *(dest++) = *(src++);
-           }
-         else
-           {
-             if (!val)
-               {
-                 *(dest++) = '\\';
-                 *(dest++) = '\0'; 
-               }
-             else 
-               *(byte*)dest++ = val;
-             src += 4;
-           }
-        }
-      else
-       {
-         /* should not happen */
-         src++;
-         *(dest++) = '\\'; 
          *(dest++) = *(src++);
          *(dest++) = *(src++);
+         continue;
+       }
+
+      switch (src[1])
+       {
+#define DECODE_ONE(match,result)       \
+       case match:                     \
+         src += 2;                     \
+         *(dest++) = result;           \
+         break;
+
+         DECODE_ONE ('\'', '\'');
+         DECODE_ONE ('\"', '\"');
+         DECODE_ONE ('\?', '\?');
+         DECODE_ONE ('\\', '\\');
+         DECODE_ONE ('a', '\a');
+         DECODE_ONE ('b', '\b');
+         DECODE_ONE ('f', '\f');
+         DECODE_ONE ('n', '\n');
+         DECODE_ONE ('r', '\r');
+         DECODE_ONE ('t', '\t');
+         DECODE_ONE ('v', '\v');
+
+       case 'x':
+         {
+           int val = _gpgme_hextobyte (&src[2]);
+
+           if (val == -1)
+             {
+               /* Should not happen.  */
+               *(dest++) = *(src++);
+               *(dest++) = *(src++);
+               if (*src)
+                 *(dest++) = *(src++);
+               if (*src)
+                 *(dest++) = *(src++);
+             }
+           else
+             {
+               if (!val)
+                 {
+                   /* A binary zero is not representable in a C
+                      string.  */
+                   *(dest++) = '\\';
+                   *(dest++) = '0'; 
+                 }
+               else 
+                 *((unsigned char *) dest++) = val;
+               src += 4;
+             }
+         }
+
+       default:
+         {
+           /* Should not happen.  */
+           *(dest++) = *(src++);
+           *(dest++) = *(src++);
+         }
         } 
     }
   *(dest++) = 0;
         } 
     }
   *(dest++) = 0;
@@ -147,13 +166,13 @@ GpgmeError
 _gpgme_data_append (GpgmeData dh, const char *buffer, size_t length)
 {
   if (!dh || !buffer)
 _gpgme_data_append (GpgmeData dh, const char *buffer, size_t length)
 {
   if (!dh || !buffer)
-    return mk_error (Invalid_Value);
+    return GPGME_Invalid_Value;
 
   do
     {
       ssize_t amt = gpgme_data_write (dh, buffer, length);
       if (amt == 0 || (amt < 0 && errno != EINTR))
 
   do
     {
       ssize_t amt = gpgme_data_write (dh, buffer, length);
       if (amt == 0 || (amt < 0 && errno != EINTR))
-       return mk_error (File_Error);
+       return GPGME_File_Error;
       buffer += amt;
       length -= amt;
     }
       buffer += amt;
       length -= amt;
     }
@@ -181,7 +200,7 @@ _gpgme_data_append_for_xml (GpgmeData dh, const char *buffer, size_t len)
   int err = 0;
 
   if (!dh || !buffer)
   int err = 0;
 
   if (!dh || !buffer)
-    return mk_error (Invalid_Value);
+    return GPGME_Invalid_Value;
 
   do
     {
 
   do
     {
@@ -233,8 +252,8 @@ _gpgme_data_append_string_for_xml (GpgmeData dh, const char *str)
 GpgmeError
 _gpgme_data_append_percentstring_for_xml (GpgmeData dh, const char *str)
 {
 GpgmeError
 _gpgme_data_append_percentstring_for_xml (GpgmeData dh, const char *str)
 {
-  const byte *src;
-  byte *buf, *dst;
+  const unsigned char *src;
+  unsigned char *buf, *dst;
   int val;
   GpgmeError err;
 
   int val;
   GpgmeError err;