]> granicus.if.org Git - postgresql/commit
Avoid dangling pointer to relation name in RLS code path in DoCopy().
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 6 Mar 2017 21:50:47 +0000 (16:50 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 6 Mar 2017 21:50:47 +0000 (16:50 -0500)
commit420d9ec0aebb0cd8d3198d25d6acaf07b0a576b9
treec6f98bea14cfe8d17ba93b7a68aa3fad4cfd17c8
parent807df31d19e7014df1d3621292589d3653f614f2
Avoid dangling pointer to relation name in RLS code path in DoCopy().

With RLS active, "COPY tab TO ..." failed under -DRELCACHE_FORCE_RELEASE,
and would sometimes fail without that, because it used the relation name
directly from the relcache as part of the parsetree it's building.  That
becomes a potentially-dangling pointer as soon as the relcache entry is
closed, a bit further down.  Typical symptom if the relcache entry chanced
to get cleared would be "relation does not exist" error with a garbage
relation name, or possibly a core dump; but if you were really truly
unlucky, the COPY might copy from the wrong table.

Per report from Andrew Dunstan that regression tests fail with
-DRELCACHE_FORCE_RELEASE.  The core tests now pass for me (but have
not tried "make check-world" yet).

Discussion: https://postgr.es/m/7b52f900-0579-cda9-ae2e-de5da17090e6@2ndQuadrant.com
src/backend/commands/copy.c