pgsql: Fix unportable usages of tolower().

From: tgl(at)postgresql(dot)org (Tom Lane)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix unportable usages of tolower().
Date: 2008-03-01 03:26:44
Message-ID: 20080301032644.5FCE1754108@cvs.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Log Message:
-----------
Fix unportable usages of tolower(). On signed-char machines, it is necessary
to explicitly cast the output back to char before comparing it to a char
value, else we get the wrong result for high-bit-set characters. Found by
Rolf Jentsch. Also, fix several places where <ctype.h> functions were being
called without casting the argument to unsigned char; this is likewise
unportable, but we keep making that mistake :-(. These found by buildfarm
member salamander, which I will desperately miss if it ever goes belly-up.

Tags:
----
REL8_3_STABLE

Modified Files:
--------------
pgsql/src/backend/utils/adt:
like_match.c (r1.20 -> r1.20.2.1)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/like_match.c?r1=1.20&r2=1.20.2.1)
pgsql/src/interfaces/ecpg/ecpglib:
execute.c (r1.76 -> r1.76.2.1)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/execute.c?r1=1.76&r2=1.76.2.1)
pgsql/src/interfaces/ecpg/preproc:
preproc.y (r1.359.2.1 -> r1.359.2.2)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/preproc/preproc.y?r1=1.359.2.1&r2=1.359.2.2)

Browse pgsql-committers by date

  From Date Subject
Next Message Magnus Hagander 2008-03-01 09:57:32 Re: pgsql: Fix handling of restricted processes for Windows Vista (mainly),
Previous Message Tom Lane 2008-03-01 03:26:35 pgsql: Fix unportable usages of tolower().