I'm working on writing a script that automatically tests all the
built-in functions. As a first step, I just passed NULL for all
of the arguments for each built-in function. I've attached a
list of the 90 functions that crash the backend when this is done.

Since the culprit functions all seem related (I believe they're
all generated by src/backend/utils/mb/conversion_procs), I'd
imagine this should be pretty easy to fix. I haven't played
with this part of the backend before, so if someone else would
like to make the necessary changes, let me know. Otherwise
I'll do it.

As for the testing script itself, it's not finished (testing
NULL inputs is just a trivial first step); when it's done, I'll
submit it for inclusion in CVS.

-Neil
Fri, 8 Aug 2003 02:53:49 -0400
Neil Conway <neilc@samurai.com>
poorly written builtin functions
Neil Conway <neilc@samurai.com> writes:
> Since the culprit functions all seem related (I believe they're
> all generated by src/backend/utils/mb/conversion_procs), I'd
> imagine this should be pretty easy to fix.

They should all be marked STRICT.  Not sure how this got missed before;
thanks for catching it.

			regards, tom lane

Fri, 08 Aug 2003 10:29:00 -0400
Tom Lane <tgl@sss.pgh.pa.us>
Re: poorly written builtin functions 

I assume this has been addressed?

---------------------------------------------------------------------------

Neil Conway wrote:
> I'm working on writing a script that automatically tests all the
> built-in functions. As a first step, I just passed NULL for all
> of the arguments for each built-in function. I've attached a
> list of the 90 functions that crash the backend when this is done.
> 
> Since the culprit functions all seem related (I believe they're
> all generated by src/backend/utils/mb/conversion_procs), I'd
> imagine this should be pretty easy to fix. I haven't played
> with this part of the backend before, so if someone else would
> like to make the necessary changes, let me know. Otherwise
> I'll do it.
> 
> As for the testing script itself, it's not finished (testing
> NULL inputs is just a trivial first step); when it's done, I'll
> submit it for inclusion in CVS.
> 
> -Neil
> 

[ Attachment, skipping... ]

> 
> ---------------------------(end of broadcast)---------------------------
> TIP 6: Have you searched our list archives?
> 
>                http://archives.postgresql.org

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

Sun, 17 Aug 2003 00:39:53 -0400 (EDT)
Bruce Momjian <pgman@candle.pha.pa.us>
Re: poorly written builtin functions
On Sun, Aug 17, 2003 at 12:39:53AM -0400, Bruce Momjian wrote:
> I assume this has been addressed?

Yes, Tom checked a fix into CVS for the functions in question.

-Neil


Tue, 19 Aug 2003 12:01:12 -0400
Neil Conway <neilc@samurai.com>
Re: poorly written builtin functions