Commit c4e0c113 authored by Tatsuo Ishii's avatar Tatsuo Ishii

Fix text_substr bug intrduced in 7.3 development

using Joe Conway's patches (submitted at pgsql-patches on 2002/04/08)
 + small fix.
parent 3767970c
......@@ -8,7 +8,7 @@
*
*
* IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/utils/adt/varlena.c,v 1.82 2002/04/03 05:39:32 petere Exp $
* $Header: /cvsroot/pgsql/src/backend/utils/adt/varlena.c,v 1.83 2002/04/15 07:54:37 ishii Exp $
*
*-------------------------------------------------------------------------
*/
......@@ -373,7 +373,10 @@ text_substr(PG_FUNCTION_ARGS)
if (eml > 1)
{
sm = 0;
if (n > -1)
sn = (m + n) * eml + 3; /* +3 to avoid mb characters overhanging slice end */
else
sn = n; /* n < 0 is special-cased by heap_tuple_untoast_attr_slice */
}
#endif
......@@ -387,7 +390,10 @@ text_substr(PG_FUNCTION_ARGS)
PG_RETURN_NULL(); /* notreached: suppress compiler warning */
#endif
#ifdef MULTIBYTE
if (n > -1)
len = pg_mbstrlen_with_len (VARDATA (string), sn - 3);
else /* n < 0 is special-cased; need full string length */
len = pg_mbstrlen_with_len (VARDATA (string), VARSIZE(string)-VARHDRSZ);
if (m > len)
{
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment