Web lists-archives.com

Re: [PATCH] repository: fix free problem with repo_clear(the_repository)

Stefan Beller <sbeller@xxxxxxxxxx> writes:

> So this would go with the latest sb/object-store-alloc ?
> My impression was that we never call repo_clear() on
> the_repository, which would allow us to special case
> the_repository further just as I did in v2 of that series[1] by
> having static allocations for certain objects in case of \
> the_repository.
> [1] https://public-inbox.org/git/20180501213403.14643-14-sbeller@xxxxxxxxxx/
> We could just deal with all the exceptions, but that makes repo_clear
> ugly IMHO.

So perhaps

         void repo_clear(...)
        +       if (repo == the_repository)
        +               BUG("repo_clear() called on the repository");

or something?