Fix off-by-one memory allocation, as reported by Rod Taylor. Also

avoid repalloc'ing twice when once is sufficient.

diff --git a/src/backend/libpq/hba.c b/src/backend/libpq/hba.c
index 1f0b29896c050e64d4cb1c50792b63329d716d14..c3c3d9c9bec0626a035c1215a51776b68b31bf9d 100644 (file)
--- a/src/backend/libpq/hba.c
+++ b/src/backend/libpq/hba.c
@@ -10,7 +10,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/libpq/hba.c,v 1.87.2.4 2003/04/13 04:07:31 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/libpq/hba.c,v 1.87.2.5 2004/11/17 19:54:53 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -281,7 +281,7 @@ tokenize_inc_file(const char *inc_filename)
                {
                        if (strlen(comma_str))
                        {
-                               comma_str = repalloc(comma_str, strlen(comma_str) + 1);
+                               comma_str = repalloc(comma_str, strlen(comma_str) + 1 + 1);
                                strcat(comma_str, MULTI_VALUE_SEP);
                        }
                        comma_str = repalloc(comma_str,