Re: beta6 pg_restore core dumps - Mailing list pgsql-hackers

From Tom Lane
Subject Re: beta6 pg_restore core dumps
Date
Msg-id 20643.984880669@sss.pgh.pa.us
Whole thread Raw
In response to Re: beta6 pg_restore core dumps  (Philip Warner <pjw@rhyme.com.au>)
Responses Re: beta6 pg_restore core dumps  (Philip Warner <pjw@rhyme.com.au>)
List pgsql-hackers
Philip Warner <pjw@rhyme.com.au> writes:
> At 12:31 17/03/01 -0500, Tom Lane wrote:
>> This would be a lot simpler and cleaner if _PrintData() simply didn't
>> append a zero byte to the buffer contents.  Philip, is it actually
>> necessary for it to do that?

> Strictly, I think the answer is that it is not necessary. The output of the
> uncompress may be a string, which could be passed to one of the str*
> functions by a downstream call. AFAICT, this is not the case, and the code
> should work without it, but it's probably safer in the long run to leave it
> there.

Considering that the data we are working with is binary, and may contain
nulls, any code that insisted on null-termination would probably be ipso
facto broken.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Philip Warner
Date:
Subject: Re: beta6 pg_restore core dumps
Next
From: Tom Lane
Date:
Subject: Re: beta6 pg_restore core dumps