Make TupleDescInitBuiltinEntry throw error for unsupported types.

tglsfdc · tglsfdc · commit b7a29695f744 · 2018-12-10T10:38:48.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
@@ -744,6 +744,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
`@@ -744,6 +744,9 @@ TupleDescInitBuiltinEntry(TupleDesc desc,`
`744`	`744`	`att->attstorage = 'p';`
`745`	`745`	`att->attcollation = InvalidOid;`
`746`	`746`	`break;`
	`747`	`+`
	`748`	`+ default:`
	`749`	`+ elog(ERROR, "unsupported type %u", oidtypeid);`
`747`	`750`	`}`
`748`	`751`	`}`
`749`	`752`