Re: pg_upgrade segfault (was: pg_migrator segfault) - Mailing list pgsql-general

From hernan gonzalez
Subject Re: pg_upgrade segfault (was: pg_migrator segfault)
Date
Msg-id AANLkTinkg=BoBgY-S6Mbi+WukciNQPKBYqWi4NDgWMGj@mail.gmail.com
Whole thread Raw
In response to Re: pg_upgrade segfault (was: pg_migrator segfault)  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: pg_upgrade segfault (was: pg_migrator segfault)
Re: pg_upgrade segfault (was: pg_migrator segfault)
List pgsql-general
In pg_upgrade/controldata.c  , putenv2 function :

        char       *envstr = (char *) pg_malloc(ctx, strlen(var) +  strlen(val) + 1);
        sprintf(envstr, "%s=%s", var, val);

Shouldn't it be  "+ 2 " instead of "+ 1" ? (one for the '=', plus one for the null terminating char) ?

I think that fixes it.


Hernán J. González
http://hjg.com.ar/

pgsql-general by date:

Previous
From: Tom Lane
Date:
Subject: Re: pg_upgrade segfault (was: pg_migrator segfault)
Next
From: hernan gonzalez
Date:
Subject: Re: pg_upgrade segfault (was: pg_migrator segfault)