Correct type of front_pathkey to PathKey

tvondra · tvondra · commit 72ac4d71b54f · 2022-01-23T04:05:08.000+01:00
In sort_inner_and_outer we iterate a list of PathKey elements, but the variable is declared as (List *). This mistake is benign, because we only pass the pointer to lcons() and never dereference it. This exists since ~2004, but it's confusing. So fix and backpatch to all supported branches. Backpatch-through: 10 Discussion: https://postgr.es/m/bf3a6ea1-a7d8-7211-0669-189d5c169374%40enterprisedb.com
diff --git a/src/backend/optimizer/path/joinpath.c b/src/backend/optimizer/path/joinpath.c
@@ -1243,7 +1243,7 @@ sort_inner_and_outer(PlannerInfo *root,
 
 	foreach(l, all_pathkeys)
 	{
-		List	   *front_pathkey = (List *) lfirst(l);
+		PathKey	   *front_pathkey = (PathKey *) lfirst(l);
 		List	   *cur_mergeclauses;
 		List	   *outerkeys;
 		List	   *innerkeys;

Original file line number	Diff line number	Diff line change
`@@ -1243,7 +1243,7 @@ sort_inner_and_outer(PlannerInfo *root,`
`1243`	`1243`
`1244`	`1244`	`foreach(l, all_pathkeys)`
`1245`	`1245`	`{`
`1246`		`- List front_pathkey = (List ) lfirst(l);`
	`1246`	`+ PathKey front_pathkey = (PathKey ) lfirst(l);`
`1247`	`1247`	`List *cur_mergeclauses;`
`1248`	`1248`	`List *outerkeys;`
`1249`	`1249`	`List *innerkeys;`