NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit62148c3

committed

Add check for syscache lookup failure in update_relispartition().

Omitted in commit05b38c7 (though it looks like the original blamebelongs to9e9befa). A failure is admittedly unlikely, but if itdid happen, SIGSEGV is not the approved method of reporting it.Per Coverity. Back-patch to v11 where the broken code originated.

1 parent84e4570 commit62148c3Copy full SHA for 62148c3

File tree

1 file changed

-1

lines changed

src/backend/commands
- indexcmds.c

1 file changed

-1

lines changed

`‎src/backend/commands/indexcmds.c`

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -3451,10 +3451,11 @@ update_relispartition(Oid relationId, bool newval)`
`3451`	`3451`
`3452`	`3452`	`classRel=table_open(RelationRelationId,RowExclusiveLock);`
`3453`	`3453`	`tup=SearchSysCacheCopy1(RELOID,ObjectIdGetDatum(relationId));`
	`3454`	`+if (!HeapTupleIsValid(tup))`
	`3455`	`+elog(ERROR,"cache lookup failed for relation %u",relationId);`
`3454`	`3456`	`Assert(((Form_pg_class)GETSTRUCT(tup))->relispartition!=newval);`
`3455`	`3457`	`((Form_pg_class)GETSTRUCT(tup))->relispartition=newval;`
`3456`	`3458`	`CatalogTupleUpdate(classRel,&tup->t_self,tup);`
`3457`	`3459`	`heap_freetuple(tup);`
`3458`		`-`
`3459`	`3460`	`table_close(classRel,RowExclusiveLock);`
`3460`	`3461`	`}`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit62148c3

File tree

1 file changed

1 file changed

`‎src/backend/commands/indexcmds.c`

0 commit comments