Fix RelationIdGetRelation calls that weren't bothering with error checks. Some of these are quite old, but that doesn't make them not bugs. We'd rather report a failure via elog than SIGSEGV.
While at it, uniformly spell the error check as !RelationIsValid(rel) rather than a bare rel == NULL test. The machine code is the same but it seems better to be consistent. Coverity complained about this today, not sure why, because the mistake is in fact old. Branch ------ REL_11_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/69f883fef14a3fc5849126799278abcc43f40f56 Modified Files -------------- src/backend/access/heap/heapam.c | 3 +++ src/backend/replication/logical/reorderbuffer.c | 8 ++++++-- 2 files changed, 9 insertions(+), 2 deletions(-)
