diff options
| author | David Rowley <drowley@postgresql.org> | 2023-09-14 11:27:43 +1200 |
|---|---|---|
| committer | David Rowley <drowley@postgresql.org> | 2023-09-14 11:27:43 +1200 |
| commit | 6341cb0b02ae16d7bbb8b4e7ebabf80c551a685e (patch) | |
| tree | f14ac2e3c51781171bf8b985ba0afb580070cf8b /src/backend/utils/adt/xml.c | |
| parent | e2452c2a630a6952289816d9b241d227a38cb93d (diff) | |
| download | postgresql-6341cb0b02ae16d7bbb8b4e7ebabf80c551a685e.tar.gz postgresql-6341cb0b02ae16d7bbb8b4e7ebabf80c551a685e.zip | |
Fix incorrect logic in plan dependency recording
Both 50e17ad28 and 29f45e299 mistakenly tried to record a plan dependency
on a function but mistakenly inverted the OidIsValid test. This meant
that we'd record a dependency only when the function's Oid was
InvalidOid. Clearly this was meant to *not* record the dependency in
that case.
50e17ad28 made this mistake first, then in v15 29f45e299 copied the same
mistake.
Reported-by: Tom Lane
Backpatch-through: 14, where 50e17ad28 first made this mistake
Discussion: https://postgr.es/m/2277537.1694301772@sss.pgh.pa.us
Diffstat (limited to 'src/backend/utils/adt/xml.c')
0 files changed, 0 insertions, 0 deletions