projects / postgresql.git / commitdiff
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: af99745)
A backport patch.
authorTatsuo Ishii <ishii@postgresql.org>
Mon, 8 Apr 2002 06:21:31 +0000 (06:21 +0000)
committerTatsuo Ishii <ishii@postgresql.org>
Mon, 8 Apr 2002 06:21:31 +0000 (06:21 +0000)
Fix PQescapeBytea/PQunescapeBytea so that they handle bytes > 0x7f.
This is necessary for mulibyte character sequences.
See "[HACKERS] PQescapeBytea is not multibyte aware" thread posted around
2002/04/05 for more details.

src/interfaces/libpq/fe-exec.c patch | blob | blame | history

diff --git a/src/interfaces/libpq/fe-exec.c b/src/interfaces/libpq/fe-exec.c
index 123d7abd1296dd25dc51d478add8b7118581b9e3..581288d149976c9469819dd2b07002d59794d069 100644 (file)
--- a/src/interfaces/libpq/fe-exec.c
+++ b/src/interfaces/libpq/fe-exec.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/interfaces/libpq/fe-exec.c,v 1.113 2001/10/25 05:50:13 momjian Exp $
+ *   $Header: /cvsroot/pgsql/src/interfaces/libpq/fe-exec.c,v 1.113.2.1 2002/04/08 06:21:31 ishii Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -115,6 +115,7 @@ PQescapeString(char *to, const char *from, size_t length)
  *     '\0' == ASCII  0 == \\000
  *     '\'' == ASCII 39 == \'
  *     '\\' == ASCII 92 == \\\\
+ *     anything >= 0x80 ---> \\ooo (where ooo is an octal expression)
  */
 unsigned char *
 PQescapeBytea(unsigned char *bintext, size_t binlen, size_t *bytealen)
@@ -131,40 +132,39 @@ PQescapeBytea(unsigned char *bintext, size_t binlen, size_t *bytealen)
    len = 1;
 
    vp = bintext;
-   for (i = binlen; i != 0; i--, vp++)
+   for (i = binlen; i > 0; i--, vp++)
    {
-       if (*vp == 0)
-           len += 5;
-       else if (*vp == 39)
+       if (*vp == 0 || *vp >= 0x80)
+           len += 5;   /* '5' is for '\\ooo' */
+       else if (*vp == '\'')
            len += 2;
-       else if (*vp == 92)
+       else if (*vp == '\\')
            len += 4;
        else
            len++;
    }
 
    rp = result = (unsigned char *) malloc(len);
+   if (rp == NULL)
+       return NULL;
+
    vp = bintext;
    *bytealen = len;
 
-   for (i = binlen; i != 0; i--, vp++)
+   for (i = binlen; i > 0; i--, vp++)
    {
-       if (*vp == 0)
+       if (*vp == 0 || *vp >= 0x80)
        {
-           rp[0] = '\\';
-           rp[1] = '\\';
-           rp[2] = '0';
-           rp[3] = '0';
-           rp[4] = '0';
+           (void)sprintf(rp,"\\\\%03o",*vp);
            rp += 5;
        }
-       else if (*vp == 39)
+       else if (*vp == '\'')
        {
            rp[0] = '\\';
            rp[1] = '\'';
            rp += 2;
        }
-       else if (*vp == 92)
+       else if (*vp == '\\')
        {
            rp[0] = '\\';
            rp[1] = '\\';
This is the main PostgreSQL git repository.