Be a bit less cavalier with both the code and the comment for UNKNOWN fix.

diff --git a/src/pl/plpgsql/src/pl_exec.c b/src/pl/plpgsql/src/pl_exec.c
index d75b7a90b9fa01c754884f755e29c6f162478c25..4a14b0c01afdb31352c39894b50a3d060b6b22c6 100644 (file)
--- a/src/pl/plpgsql/src/pl_exec.c
+++ b/src/pl/plpgsql/src/pl_exec.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/pl/plpgsql/src/pl_exec.c,v 1.261.2.2 2010/08/19 16:54:48 heikki Exp $
+ *   $PostgreSQL: pgsql/src/pl/plpgsql/src/pl_exec.c,v 1.261.2.3 2010/08/19 17:31:50 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -5516,14 +5516,16 @@ exec_eval_using_params(PLpgSQL_execstate *estate, List *params)
        if (ppd->types[i] == UNKNOWNOID)
        {
            /*
-            * Treat 'unknown' parameters as text, that's what most people
-            * would expect. The backend can coerce unknown constants in a
-            * more intelligent way, but not unknown Params. 
+            * Treat 'unknown' parameters as text, since that's what most
+            * people would expect. SPI_execute_with_args can coerce unknown
+            * constants in a more intelligent way, but not unknown Params.
+            * This code also takes care of copying into the right context.
+            * Note we assume 'unknown' has the representation of C-string.
             */
            ppd->types[i] = TEXTOID;
            if (!isnull)
            {
-               ppd->values[i] = CStringGetTextDatum((char *) ppd->values[i]);
+               ppd->values[i] = CStringGetTextDatum(DatumGetCString(ppd->values[i]));
                ppd->freevals[i] = true;
            }
        }