Fix backpatching error in recent patch for ALTER USER f RESET ALL behavior.
authorTom Lane <[email protected]>
Wed, 5 May 2010 02:55:11 +0000 (02:55 +0000)
committerTom Lane <[email protected]>
Wed, 5 May 2010 02:55:11 +0000 (02:55 +0000)
The argument list for array_set() changed in 8.2 (in connection with allowing
nulls in arrays) but the newer argument list was used in the patches applied
to 8.1 and 8.0 branches.  The patch for 7.4 was OK though.  Per compiler
warnings.

src/backend/utils/misc/guc.c

index 034f469b05b61e47391ae6aa7c222321a4419f75..92005b30a6787b99fc4defd671caad0a86c5eb93 100644 (file)
@@ -10,7 +10,7 @@
  * Written by Peter Eisentraut <[email protected]>.
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/utils/misc/guc.c,v 1.252.4.10 2010/03/25 14:45:51 alvherre Exp $
+ *       $PostgreSQL: pgsql/src/backend/utils/misc/guc.c,v 1.252.4.11 2010/05/05 02:55:11 tgl Exp $
  *
  *--------------------------------------------------------------------
  */
@@ -5327,13 +5327,14 @@ GUCArrayDelete(ArrayType *array, const char *name)
                /* else add it to the output array */
                if (newarray)
                {
+                       isnull = false;
                        newarray = array_set(newarray, 1, &index,
                                                                 d,
-                                                                false,
                                                                 -1 /* varlenarray */ ,
                                                                 -1 /* TEXT's typlen */ ,
                                                                 false /* TEXT's typbyval */ ,
-                                                                'i' /* TEXT's typalign */ );
+                                                                'i' /* TEXT's typalign */ ,
+                                                                &isnull);
                }
                else
                        newarray = construct_array(&d, 1,