Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit 7b65862

Browse files
committed
Correct type of front_pathkey to PathKey
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
1 parent 6d554e3 commit 7b65862

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/backend/optimizer/path/joinpath.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1258,7 +1258,7 @@ sort_inner_and_outer(PlannerInfo *root,
12581258

12591259
foreach(l, all_pathkeys)
12601260
{
1261-
List *front_pathkey = (List *) lfirst(l);
1261+
PathKey *front_pathkey = (PathKey *) lfirst(l);
12621262
List *cur_mergeclauses;
12631263
List *outerkeys;
12641264
List *innerkeys;

0 commit comments

Comments
 (0)