Magnus Hagander <magnus@hagander.net> writes:
> I'd also like a comment from at least one other "patch reviewer" that
> the methods used are good.
It looks reasonable as far as it goes. One thought is that pg_dump
really should have noticed that it was writing a broken archive.
On machines where off_t is 32 bits, can't we detect the overflow
situation?
regards, tom lane