diff --git a/src/backend/utils/cache/lsyscache.c b/src/backend/utils/cache/lsyscache.c index c13c08a97b..27602fa49c 100644 --- a/src/backend/utils/cache/lsyscache.c +++ b/src/backend/utils/cache/lsyscache.c @@ -3157,7 +3157,8 @@ get_range_subtype(Oid rangeOid) * * Given the index OID and column number, * return opclass of the index column - * or InvalidOid if the index was not found. + * or InvalidOid if the index was not found + * or column is non-key one. */ Oid get_index_column_opclass(Oid index_oid, int attno) @@ -3180,11 +3181,20 @@ get_index_column_opclass(Oid index_oid, int attno) /* caller is supposed to guarantee this */ Assert(attno > 0 && attno <= rd_index->indnatts); + /* Non-key attributes don't have an opclass */ + if (attno > rd_index->indnkeyatts) + { + ReleaseSysCache(tuple); + return InvalidOid; + } + datum = SysCacheGetAttr(INDEXRELID, tuple, Anum_pg_index_indclass, &isnull); Assert(!isnull); indclass = ((oidvector *) DatumGetPointer(datum)); + + Assert(attno <= indclass->dim1); opclass = indclass->values[attno - 1]; ReleaseSysCache(tuple);