On Tue, Aug 26, 2014 at 4:10 AM, Michael Paquier <michael.paqu...@gmail.com> wrote: > > On Tue, Aug 26, 2014 at 3:48 AM, Heikki Linnakangas > <hlinnakan...@vmware.com> wrote: > > AFAICS, the namespace can never be NULL in any of these. There is a > > "selectSourceSchema(fout, tbinfo->dobj.namespace->dobj.name)" call before or > > after printing the message, so if tbinfo->dobj.namespace is NULL, you'll > > crash anyway. Please double-check, and remove the dead code if you agree. > Ah right, this field is used in many places. Even for > pg_backup_archiver.c, the portion of code processing data always has > the namespace set. I am sure that Fabrizio would have done that > quickly, but as I was on this thread I simplified the patch as > attached. >
Thanks Michael... last night I was working on a refactoring in "tablecmds.c". Regards, -- Fabrízio de Royes Mello Consultoria/Coaching PostgreSQL >> Timbira: http://www.timbira.com.br >> Blog: http://fabriziomello.github.io >> Linkedin: http://br.linkedin.com/in/fabriziomello >> Twitter: http://twitter.com/fabriziomello >> Github: http://github.com/fabriziomello
