In parse_bool_with_len, avoid crash when no result pointer is passed. Probably

an unlikely call mode, but better be safe.

diff --git a/src/backend/utils/adt/bool.c b/src/backend/utils/adt/bool.c
index 7d3f9e55a1c6213e07d7937625d4ebd8e81195dd..a9f9c5b20789801dbabdff86f1f8f57cc810228a 100644 (file)
--- a/src/backend/utils/adt/bool.c
+++ b/src/backend/utils/adt/bool.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/utils/adt/bool.c,v 1.46 2009/03/09 14:34:34 petere Exp $
+ *       $PostgreSQL: pgsql/src/backend/utils/adt/bool.c,v 1.47 2009/03/09 16:49:12 petere Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -109,7 +109,8 @@ parse_bool_with_len(const char *value, size_t len, bool *result)
                        break;
        }
 
-       *result = false;        /* suppress compiler warning */
+       if (result)
+               *result = false;        /* suppress compiler warning */
        return false;
 }