diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2018-12-10 10:38:48 -0500 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2018-12-10 10:38:48 -0500 |
commit | b7a29695f744c3140350a4f1bb8511e950acc086 (patch) | |
tree | 1cef3b7199398bc34f41ac4a72718cccd28deadc | |
parent | eeeb1dfc87fafe1cf4332c09692779662a95511e (diff) | |
download | postgresql-b7a29695f744c3140350a4f1bb8511e950acc086.tar.gz postgresql-b7a29695f744c3140350a4f1bb8511e950acc086.zip |
Make TupleDescInitBuiltinEntry throw error for unsupported types.
Previously, it would just pass back a partially-uninitialized tupdesc,
which doesn't seem like a safe or useful behavior.
Backpatch to v10 where this code came in.
Discussion: https://postgr.es/m/30830.1544384975@sss.pgh.pa.us
-rw-r--r-- | src/backend/access/common/tupdesc.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/backend/access/common/tupdesc.c b/src/backend/access/common/tupdesc.c index 5354a04639b..e3e4ab636bb 100644 --- a/src/backend/access/common/tupdesc.c +++ b/src/backend/access/common/tupdesc.c @@ -744,6 +744,9 @@ TupleDescInitBuiltinEntry(TupleDesc desc, att->attstorage = 'p'; att->attcollation = InvalidOid; break; + + default: + elog(ERROR, "unsupported type %u", oidtypeid); } } |