From 5f768045a1a25847a3eea59d309e28c8141aed44 Mon Sep 17 00:00:00 2001 From: Alvaro Herrera Date: Wed, 3 Apr 2019 09:56:20 -0300 Subject: [PATCH] Correctly initialize newly added struct member Valgrind was rightly complaining that IndexVacuumInfo->report_progress (added by commit ab0dfc961b6a) was not being initialized in some code paths. Repair. Per buildfarm member lousyjack. --- src/backend/access/heap/vacuumlazy.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/backend/access/heap/vacuumlazy.c b/src/backend/access/heap/vacuumlazy.c index b5b464e4a9d..392b35ebb77 100644 --- a/src/backend/access/heap/vacuumlazy.c +++ b/src/backend/access/heap/vacuumlazy.c @@ -1717,6 +1717,7 @@ lazy_vacuum_index(Relation indrel, ivinfo.index = indrel; ivinfo.analyze_only = false; + ivinfo.report_progress = false; ivinfo.estimated_count = true; ivinfo.message_level = elevel; /* We can only provide an approximate value of num_heap_tuples here */ @@ -1749,6 +1750,7 @@ lazy_cleanup_index(Relation indrel, ivinfo.index = indrel; ivinfo.analyze_only = false; + ivinfo.report_progress = false; ivinfo.estimated_count = (vacrelstats->tupcount_pages < vacrelstats->rel_pages); ivinfo.message_level = elevel;