Re: malloc in xlog.c

Bruce Momjian <pgman@candle.pha.pa.us> writes:
> I see a few malloc's in backend/access/transam/xlog.c that don't check
> to see if malloc returns NULL/failure.  I think there should be at least
> an Assert() in there.

It'll dump core just fine without the help of an Assert ;-).  I don't
see that an Assert adds much of anything.

In practice those mallocs are done during startup, so it seems very
unlikely that they could fail, anyway.

> Also, seems we use malloc() a few other places where palloc should be
> used, like variable.c.  Is that correct?

Those are correct because of guc.c's conventions.  Note the comments.
        regards, tom lane