On 02/20, Stefan Beller wrote:
> Free the memory and reset alt_odb_{list, tail} to NULL.

Good to see memory leaks being avoided (well they will be on other
repository objects)

> 
> Signed-off-by: Stefan Beller <sbel...@google.com>
> ---
>  object.c | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/object.c b/object.c
> index 11d904c033..17b1da6d6b 100644
> --- a/object.c
> +++ b/object.c
> @@ -446,7 +446,24 @@ void clear_commit_marks_all(unsigned int flags)
>       }
>  }
>  
> +static void free_alt_odb(struct alternate_object_database *alt)
> +{
> +     strbuf_release(&alt->scratch);
> +     oid_array_clear(&alt->loose_objects_cache);
> +}
> +
> +static void free_alt_odbs(struct raw_object_store *o)
> +{
> +     while (o->alt_odb_list) {
> +             free_alt_odb(o->alt_odb_list);
> +             o->alt_odb_list = o->alt_odb_list->next;
> +     }
> +}
> +
>  void raw_object_store_clear(struct raw_object_store *o)
>  {
>       free(o->objectdir);
> +
> +     free_alt_odbs(o);
> +     o->alt_odb_tail = NULL;
>  }
> -- 
> 2.16.1.291.g4437f3f132-goog
> 

-- 
Brandon Williams

Reply via email to