On Wed, Jul 5, 2017 at 11:53 AM, Peter Korsgaard <pe...@korsgaard.com> wrote:
> dpkg moved to away from dynamically allocating the hashtables in commit
> c87339d584 (dpkg: trivial code shrinkage, and redo G trick correctly) almost
> ten years ago, but the cleanup code was never adjusted to match.
>
> Glibc loudly complains about this:
>
> *** Error in `dpkg': free(): invalid pointer: 0x0000007fac3478c0 ***
>
> Signed-off-by: Peter Korsgaard <pe...@korsgaard.com>
> ---
>  archival/dpkg.c | 4 ----
>  1 file changed, 4 deletions(-)
>
> diff --git a/archival/dpkg.c b/archival/dpkg.c
> index 1cd45eda4..da3b0864e 100644
> --- a/archival/dpkg.c
> +++ b/archival/dpkg.c
> @@ -1938,10 +1938,6 @@ int dpkg_main(int argc UNUSED_PARAM, char **argv)
>                 for (i = 0; i < STATUS_HASH_PRIME; i++) {
>                         free(status_hashtable[i]);
>                 }
> -
> -               free(status_hashtable);
> -               free(package_hashtable);
> -               free(name_hashtable);
>         }
>
>         return EXIT_SUCCESS;

Applied, thanks!
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to