Re: pg_group_name_index corrupt? - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: pg_group_name_index corrupt?
Date
Msg-id 200005050441.AAA11008@candle.pha.pa.us
Whole thread Raw
In response to Re: pg_group_name_index corrupt?  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: pg_group_name_index corrupt?  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
> I wrote:
> > It occurred to me that this would be the likely symptom if a CREATE
> > GROUP or ALTER GROUP command had neglected to update the indexes on
> > pg_group.  However, I can't spot any such problem in the code nor
> > cause it to happen by hand.  Anyone else have an idea?
> 
> After further looking, I notice that users.c is one of the few places
> that will drop AccessExclusiveLock at heap_close time rather than
> holding it till xact commit.  I wonder whether this is a bug...
> it could allow another backend to get in and start vacuuming the file
> before our updates have committed.  I am not sure that vacuum would do
> the wrong thing in that case, but maybe so.  Comments anyone (Vadim?)
> 
> I also notice that there definitely is a glaring bug there:
> write_password_file() leaks one kernel file descriptor each time it runs
> (note the creat() call).  Alter enough pg_shadow entries in one session
> and your backend stops working.  I think this is a "must fix" problem
> --- any objections?

Please fix both, I think.

--  Bruce Momjian                        |  http://www.op.net/~candle pgman@candle.pha.pa.us               |  (610)
853-3000+  If your life is a hard drive,     |  830 Blythe Avenue +  Christ can be your backup.        |  Drexel Hill,
Pennsylvania19026
 


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: pg_group_name_index corrupt?
Next
From: The Hermit Hacker
Date:
Subject: Re: pg_group_name_index corrupt?