database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-11-03 09:13:20 +03:00
Code Activity

Slightly more paranoia and slightly better comments for use of

Browse Source 
Windows-specific MultiByteToWideChar/WideCharToMultiByte calls.
This commit is contained in:
Tom Lane
2007-11-24 21:16:55 +00:00
parent ffd8721228
commit 0f20e7a83e
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/backend/utils/adt/oracle_compat.c
View File

@@ -9,7 +9,7 @@
* *
* *
* IDENTIFICATION * IDENTIFICATION
* $PostgreSQL: pgsql/src/backend/utils/adt/oracle_compat.c,v 1.74 2007/11/15 21:14:39 momjian Exp $ * $PostgreSQL: pgsql/src/backend/utils/adt/oracle_compat.c,v 1.75 2007/11/24 21:16:55 tgl Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
@@ -187,7 +187,7 @@ win32_utf8_texttowcs(const text *txt)
r = MultiByteToWideChar(CP_UTF8, 0, VARDATA_ANY(txt), nbytes, r = MultiByteToWideChar(CP_UTF8, 0, VARDATA_ANY(txt), nbytes,
result, nbytes); result, nbytes);
if (!r) /* assume it's NO_UNICODE_TRANSLATION */ if (r <= 0) /* assume it's NO_UNICODE_TRANSLATION */
{ {
/* see notes above about error reporting */ /* see notes above about error reporting */
pg_verifymbstr(VARDATA_ANY(txt), nbytes, false); pg_verifymbstr(VARDATA_ANY(txt), nbytes, false);
@@ -198,6 +198,7 @@ win32_utf8_texttowcs(const text *txt)
} }
} }
/* Append trailing null wchar (MultiByteToWideChar won't have) */
Assert(r <= nbytes); Assert(r <= nbytes);
result[r] = 0; result[r] = 0;
@@ -212,8 +213,9 @@ win32_utf8_wcstotext(const wchar_t *str)
int nbytes; int nbytes;
int r; int r;
/* Compute size of output string (this *will* include trailing null) */
nbytes = WideCharToMultiByte(CP_UTF8, 0, str, -1, NULL, 0, NULL, NULL); nbytes = WideCharToMultiByte(CP_UTF8, 0, str, -1, NULL, 0, NULL, NULL);
if (nbytes == 0) /* shouldn't happen */ if (nbytes <= 0) /* shouldn't happen */
ereport(ERROR, ereport(ERROR,
(errcode(ERRCODE_CHARACTER_NOT_IN_REPERTOIRE), (errcode(ERRCODE_CHARACTER_NOT_IN_REPERTOIRE),
errmsg("UTF-16 to UTF-8 translation failed: %lu", errmsg("UTF-16 to UTF-8 translation failed: %lu",
@@ -223,7 +225,7 @@ win32_utf8_wcstotext(const wchar_t *str)
r = WideCharToMultiByte(CP_UTF8, 0, str, -1, VARDATA(result), nbytes, r = WideCharToMultiByte(CP_UTF8, 0, str, -1, VARDATA(result), nbytes,
NULL, NULL); NULL, NULL);
if (r == 0) /* shouldn't happen */ if (r != nbytes) /* shouldn't happen */
ereport(ERROR, ereport(ERROR,
(errcode(ERRCODE_CHARACTER_NOT_IN_REPERTOIRE), (errcode(ERRCODE_CHARACTER_NOT_IN_REPERTOIRE),
errmsg("UTF-16 to UTF-8 translation failed: %lu", errmsg("UTF-16 to UTF-8 translation failed: %lu",