Plug memory leak in index_get_partition

The list of indexes was being leaked when asked for an index that
doesn't have an index partition in the table partition.  Not a common
case admittedly --and in most cases where it occurs, caller throws an
error anyway-- but worth fixing for cleanliness and in case any
third-party code is calling this function.

While at it, remove use of lfirst_oid() to obtain a value we already
have.

Author: Justin Pryzby 
Reviewed-by: Michael Paquier 
Discussion: https://postgr.es/m/20201105203606[email protected]

diff --git a/src/backend/catalog/partition.c b/src/backend/catalog/partition.c
index e96620e4018f801348ef19efb70997dde96d8033..c41fba7be3b0d7d04dd1d37c2d14a9f7b56d9c54 100644 (file)
--- a/src/backend/catalog/partition.c
+++ b/src/backend/catalog/partition.c
@@ -171,13 +171,14 @@ index_get_partition(Relation partition, Oid indexId)
        ReleaseSysCache(tup);
        if (!ispartition)
            continue;
-       if (get_partition_parent(lfirst_oid(l)) == indexId)
+       if (get_partition_parent(partIdx) == indexId)
        {
            list_free(idxlist);
            return partIdx;
        }
    }
 
+   list_free(idxlist);
    return InvalidOid;
 }