database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-11-24 00:23:06 +03:00
Code Activity

Work around buggy strxfrm() present in some Solaris releases.

Browse Source
This commit is contained in:
Tom Lane
2003-07-17 20:52:42 +00:00
parent 19de9d2fee
commit df32af15b6
1 changed files with 16 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
src/backend/utils/adt/selfuncs.c
View File

@@ -15,7 +15,7 @@
* *
* *
* IDENTIFICATION * IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/utils/adt/selfuncs.c,v 1.119.2.4 2003/04/16 04:38:05 tgl Exp $ * $Header: /cvsroot/pgsql/src/backend/utils/adt/selfuncs.c,v 1.119.2.5 2003/07/17 20:52:42 tgl Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
@@ -2313,9 +2313,6 @@ static unsigned char *
convert_string_datum(Datum value, Oid typid) convert_string_datum(Datum value, Oid typid)
{ {
char *val; char *val;
char *xfrmstr;
size_t xfrmsize;
size_t xfrmlen;
switch (typid) switch (typid)
{ {
@@ -2355,17 +2352,21 @@ convert_string_datum(Datum value, Oid typid)
if (!lc_collate_is_c()) if (!lc_collate_is_c())
{ {
/* Guess that transformed string is not much bigger than original */ char *xfrmstr;
xfrmsize = strlen(val) + 32; /* arbitrary pad value here... */ size_t xfrmlen;
xfrmstr = (char *) palloc(xfrmsize); size_t xfrmlen2;
xfrmlen = strxfrm(xfrmstr, val, xfrmsize);
if (xfrmlen >= xfrmsize) /*
{ * Note: originally we guessed at a suitable output buffer size,
/* Oops, didn't make it */ * and only needed to call strxfrm twice if our guess was too small.
pfree(xfrmstr); * However, it seems that some versions of Solaris have buggy
* strxfrm that can write past the specified buffer length in that
* scenario. So, do it the dumb way for portability.
*/
xfrmlen = strxfrm(NULL, val, 0);
xfrmstr = (char *) palloc(xfrmlen + 1); xfrmstr = (char *) palloc(xfrmlen + 1);
xfrmlen = strxfrm(xfrmstr, val, xfrmlen + 1); xfrmlen2 = strxfrm(xfrmstr, val, xfrmlen + 1);
} Assert(xfrmlen2 == xfrmlen);
pfree(val); pfree(val);
val = xfrmstr; val = xfrmstr;
} }