Re: beta testing - pg_upgrade bug fix - double free - Mailing list pgsql-hackers

From Robert Haas
Subject Re: beta testing - pg_upgrade bug fix - double free
Date
Msg-id AANLkTilXOxfc-M_IGMYae3_rVR03L-u5FUFBQWHGkNHo@mail.gmail.com
Whole thread Raw
In response to beta testing - pg_upgrade bug fix - double free  (Pavel Stehule <pavel.stehule@gmail.com>)
List pgsql-hackers
On Fri, May 21, 2010 at 10:11 PM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
> it fixes bug
>
> pg_upgrade(13359) malloc: *** error for object 0x801600:
> non-page-aligned, non-allocated pointer being freed
> *** set a breakpoint in malloc_error_break to debug
>
>
> arget 03:31 /usr/local/src/postgresql/contrib/pg_upgrade git diff .
> diff --git a/contrib/pg_upgrade/check.c b/contrib/pg_upgrade/check.c
> index 31f12fb..f989229 100644
> --- a/contrib/pg_upgrade/check.c
> +++ b/contrib/pg_upgrade/check.c
> @@ -154,7 +154,6 @@ issue_warnings(migratorContext *ctx, char
> *sequence_script_file_name)
>                                         ctx->new.bindir,
> ctx->new.port, sequence_script_file_name,
>                                         ctx->logfile);
>                       unlink(sequence_script_file_name);
> -                       pg_free(sequence_script_file_name);
>                       check_ok(ctx);
>               }
>
> by Jan Matousek

This patch looks correct to me, although I haven't had a chance to test it yet.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise Postgres Company


pgsql-hackers by date:

Previous
From: Mohammad Heykal Abdillah
Date:
Subject: Re: "unexpected" query behaviour after i change parser code
Next
From: Robert Haas
Date:
Subject: Re: "unexpected" query behaviour after i change parser code