[PATCH 1/4] list-objects-filter: don't memset after releasing filter struct
- From
Jeff King <peff@peff.net>
- Date
- Sep 11, 2022, 04:58 UTC
- Message-ID
- <Yx1q4crkHGNzwLhh@coredump.intra.peff.net>
- In-Reply-To
- <Yx1qwlpNbxGEiUgc@coredump.intra.peff.net>
If we see an error while parsing a "combine" filter, we call list_objects_filter_release() to free any allocated memory, and then use memset() to return the struct to a known state. But the release function already does that reinitializing. Doing it again is pointless.
Signed-off-by: Jeff King <peff@peff.net> --- list-objects-filter-options.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/list-objects-filter-options.c b/list-objects-filter-options.c index 6cc4eb8e1c..ea989db260 100644 --- a/list-objects-filter-options.c +++ b/list-objects-filter-options.c @@ -187,10 +187,8 @@ static int parse_combine_filter( cleanup: strbuf_list_free(subspecs); - if (result) { + if (result) list_objects_filter_release(filter_options); - memset(filter_options, 0, sizeof(*filter_options)); - } return result; }
-- 2.37.3.1242.g835d375f85