Do we have any platforms that allow null pointer dereference?

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-hackers(at)postgreSQL(dot)org, pgsql-ports(at)postgreSQL(dot)org
Subject: Do we have any platforms that allow null pointer dereference?
Date: 2001-04-03 04:43:18
Message-ID: 330.986272998@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-ports

Do we have any supported platforms where dereferencing a null pointer
doesn't trigger coredump?

I'm wondering about this after noticing the likely side effects of
fd.c's failure to check for null result from malloc(): it'll try to
strcpy() filenames to location zero. If it succeeds, you could end up
with multiple VFDs sharing the same filename string. Which could lead
to, eg, writing on or even deleting one file under the delusion that
we were writing/deleting another.

With sufficient suspension of disbelief about how long a backend
could run at zero free memory before elog'ing, this might explain
the two recent reports of Postgres apparently deleting a file it
shouldn't have. (I'm not sure I really believe that, but given
the way palloc works it's not out of the question. I've added
appropriate checks to fd.c, just in case.)

AFAIK, null pointer deref -> SIGSEGV is standard behavior on most
platforms these days, and we take steps to select that behavior on
some nonconformists like HPUX. But I'm wondering if there are any
platforms we could select it on and have forgotten to. I think it
would be a real good idea to turn on null pointer crash anywhere
we can.

regards, tom lane

Browse pgsql-hackers by date

  From Date Subject
Next Message Adriaan Joubert 2001-04-03 05:29:25 Re: Bug in user-defined types?
Previous Message Tom Lane 2001-04-03 03:57:30 Re: Re: Changing the default value of an inherited column

Browse pgsql-ports by date

  From Date Subject
Next Message Jason Tishler 2001-04-03 13:15:50 Re: Cygwin 7.1RC2 timestamp Regression Test Failures
Previous Message Tom Lane 2001-04-02 22:33:40 Re: Cygwin 7.1RC2 timestamp Regression Test Failures