On 2/6/07, Marko Kreen <[EMAIL PROTECTED]> wrote:
Indeed, the code can crash on NULL values as the NULL checks
are missing or wrong in the functions.  Actually all the various
functions except newid() should be declared STRICT IMMUTABLE
thus immidiately avoiding problems with NULLs.

Could you reproduce the crash with this change?  I'll try
to play with this myself too.

STRICT IMMUTABLE fixed the crash for me so seems it was bug
in the module.  Although it did not happen in 8.2.1 so seems
some change in 8.2.2 made it trigger.

Attached is a patch for uniqueindent-0.2 that removes the
buggy checks and makes functions STRICT IMMUTABLE.

--
marko

Attachment: null.fix.diff
Description: Binary data

---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster

Reply via email to