repository: fix free problem with repo_clear(the_repository)
authorNguyễn Thái Ngọc Duy <pclouds@gmail.com>
Thu, 10 May 2018 06:13:10 +0000 (08:13 +0200)
committerJunio C Hamano <gitster@pobox.com>
Thu, 10 May 2018 09:33:09 +0000 (18:33 +0900)
commit74373b5f10e2e8720ad2ae8211811757601550fd
treee1052290a093b3790bed21da2ed78fccc37deee1
parent3013dff8662eae06457fe6e5348dfe2270810ab2
repository: fix free problem with repo_clear(the_repository)

the_repository is special. One of the special things about it is that
it does not allocate a new index_state object like submodules but
points to the global the_index variable instead. As a global variable,
the_index cannot be free()'d.

Add an exception for this in repo_clear(). In the future perhaps we
would be able to allocate the_repository's index on heap too. Then we
can revert this.

the_repository->index remains pointed to a clean the_index even after
repo_clear() so that it could still be used next time (e.g. in a crazy
use case where a dev switches repo in the same process).

Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
repository.c