crash in pg_identify_object_as_address - Mailing list pgsql-hackers

From Alvaro Herrera
Subject crash in pg_identify_object_as_address
Date
Msg-id 20190218202743.GA12392@alvherre.pgsql
Whole thread Raw
Responses Re: crash in pg_identify_object_as_address
List pgsql-hackers
Hello

I just came across a crash while debugging some corrupted system
catalogs; pg_identify_object_as_address fails to cope with some NULL
input, causing a crash.  Attached patch fixes it.  Naturally, the output
array will contain NULL members in the output, but that's better than
crashing ...

(The first hunk is purely speculative; I haven't seen anything that
requires that.  The actual fix is in the other hunks.  But seems better
to be defensive.)

The crash can be reproduced thusly

create function f() returns int language plpgsql as $$ begin return 1; end; $$;
update pg_proc set pronamespace = 9999 where proname = 'f' returning oid \gset
select * from pg_identify_object_as_address('pg_proc'::regclass, :oid, 0);

After the patch, the last line returns:

   type   | object_names | object_args 
----------+--------------+-------------
 function | {NULL,f}     | {}

where the NULL obviously corresponds to the bogus pg_namespace OID being
referenced.

The patch is on 9.6.  I checked 10 and it applies fine there.  My
intention is to apply to all branches since 9.5.

-- 
Álvaro Herrera                PostgreSQL Expert, https://www.2ndQuadrant.com/

Attachment

pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: unconstify equivalent for volatile
Next
From: Tom Lane
Date:
Subject: Re: Proving IS NOT NULL inference for ScalarArrayOpExpr's