Re: [PATCHES] pg_ctl crash fix

2006-01-14 Thread Magnus Hagander
> Magnus Hagander wrote:
> > I don't have the back branches around right now, but I beleive at 
> > least 8.1 is affected. Probably 8.2 as well. Backpatch? I think so
> > ;-) (and probalby should've written that in the original mail, but
> > forgot)
> 
> Done.

Thanks!

//Magnus

---(end of broadcast)---
TIP 6: explain analyze is your friend


Re: [PATCHES] pg_ctl crash fix

2006-01-14 Thread Peter Eisentraut
Magnus Hagander wrote:
> I don't have the back branches around right now, but I beleive at
> least 8.1 is affected. Probably 8.2 as well. Backpatch? I think so
> ;-) (and probalby should've written that in the original mail, but
> forgot)

Done.

-- 
Peter Eisentraut
http://developer.postgresql.org/~petere/

---(end of broadcast)---
TIP 1: if posting/reading through Usenet, please send an appropriate
   subscribe-nomail command to [EMAIL PROTECTED] so that your
   message can get through to the mailing list cleanly


Re: [PATCHES] pg_ctl crash fix

2006-01-14 Thread Magnus Hagander
> > pg_dump crashes on "unregister" when a data directory is not 
> > specified. Even though a data directory is not required. Attached 
> > patch fixes that.
> 
> Fixed.

I don't have the back branches around right now, but I beleive at least
8.1 is affected. Probably 8.2 as well. Backpatch? I think so ;-) (and
probalby should've written that in the original mail, but forgot)

//Magnus

---(end of broadcast)---
TIP 6: explain analyze is your friend


Re: [PATCHES] pg_ctl crash fix

2006-01-14 Thread Peter Eisentraut
Magnus Hagander wrote:
> pg_dump crashes on "unregister" when a data directory is not
> specified. Even though a data directory is not required. Attached
> patch fixes that.

Fixed.

-- 
Peter Eisentraut
http://developer.postgresql.org/~petere/

---(end of broadcast)---
TIP 5: don't forget to increase your free space map settings


[PATCHES] pg_ctl crash fix

2006-01-14 Thread Magnus Hagander
pg_dump crashes on "unregister" when a data directory is not specified.
Even though a data directory is not required. Attached patch fixes that.

(There is a check a couple of rows up that specifically allow for
pg_data to be null with unregister, but then we try to use it without
checking if it is...)

//Magnus



pg_ctl_crash.patch
Description: pg_ctl_crash.patch

---(end of broadcast)---
TIP 6: explain analyze is your friend