From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-hackers(at)postgresql(dot)org, pgsql-patches(at)postgresql(dot)org |
Subject: | Sigh: 7.0 pg_dump fails if user table has index on OID |
Date: | 2000-05-19 23:16:40 |
Message-ID: | 10300.958778200@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
If any user-created index is on a system column (eg, OID),
7.0 pg_dump fails with "parseNumericArray: bogus number".
Mea culpa, mea maxima culpa --- some well-intentioned error checking
code was a bit too tight. (But how'd this get through beta with no
one noticing? It's been broken since January...)
The attached patch is committed for 7.0.1, but you will need to apply
it by hand if you have such indexes and you want to make a dump before
7.0.1 comes out. (Alternatively, drop the indexes and remake them
by hand later.)
Thanks to Kyle Bateman for the bug report.
regards, tom lane
*** src/bin/pg_dump/common.c.orig Wed Apr 12 13:16:14 2000
--- src/bin/pg_dump/common.c Fri May 19 19:00:00 2000
***************
*** 190,196 ****
}
else
{
! if (!isdigit(s) || j >= sizeof(temp) - 1)
{
fprintf(stderr, "parseNumericArray: bogus number\n");
exit(2);
--- 190,196 ----
}
else
{
! if (!(isdigit(s) || s == '-') || j >= sizeof(temp) - 1)
{
fprintf(stderr, "parseNumericArray: bogus number\n");
exit(2);
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2000-05-19 23:19:38 | Re: OO Patch |
Previous Message | Tom Lane | 2000-05-19 22:47:23 | Re: OO Patch |