diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2021-04-11 11:46:32 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2021-04-11 11:46:46 -0400 |
| commit | 9cb92334092fa75afc62a71243bbc1f4612ecfa4 (patch) | |
| tree | 07a4e36c0f8be1f857888a21817694d7aef7a9f5 /src | |
| parent | 81a23dd87999ec9fb62554328c69c5b678612d56 (diff) | |
| download | postgresql-9cb92334092fa75afc62a71243bbc1f4612ecfa4.tar.gz postgresql-9cb92334092fa75afc62a71243bbc1f4612ecfa4.zip | |
Fix uninitialized variable from commit a4d75c86b.
The path for *exprs != NIL would misbehave, and likely crash,
since pull_varattnos expects its last argument to be valid
at call.
Found by Coverity --- we have no coverage of this path in
the regression tests.
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/statistics/extended_stats.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/statistics/extended_stats.c b/src/backend/statistics/extended_stats.c index 4674168ff80..4bbd85f401e 100644 --- a/src/backend/statistics/extended_stats.c +++ b/src/backend/statistics/extended_stats.c @@ -1609,7 +1609,7 @@ statext_is_compatible_clause(PlannerInfo *root, Node *clause, Index relid, if (pg_class_aclcheck(rte->relid, userid, ACL_SELECT) != ACLCHECK_OK) { - Bitmapset *clause_attnums; + Bitmapset *clause_attnums = NULL; /* Don't have table privilege, must check individual columns */ if (*exprs != NIL) |