Make TupleDescInitBuiltinEntry throw error for unsupported types.

tglsfdc · tglsfdc · commit 9eaba23c414d · 2018-12-10T10:38:49.000-05:00
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
diff --git a/src/backend/access/common/tupdesc.c b/src/backend/access/common/tupdesc.c
@@ -643,6 +643,9 @@ TupleDescInitBuiltinEntry(TupleDesc desc,
 			att->attstorage = 'p';
 			att->attcollation = InvalidOid;
 			break;
+
+		default:
+			elog(ERROR, "unsupported type %u", oidtypeid);
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -643,6 +643,9 @@ TupleDescInitBuiltinEntry(TupleDesc desc,`
`643`	`643`	`att->attstorage = 'p';`
`644`	`644`	`att->attcollation = InvalidOid;`
`645`	`645`	`break;`
	`646`	`+`
	`647`	`+ default:`
	`648`	`+ elog(ERROR, "unsupported type %u", oidtypeid);`
`646`	`649`	`}`
`647`	`650`	`}`
`648`	`651`