You are viewing a plain text version of this content. The canonical link for it is here.
Posted to c-dev@xerces.apache.org by "Scott Case (JIRA)" <xe...@xml.apache.org> on 2004/11/19 14:20:24 UTC

[jira] Created: (XERCESC-1300) XMLString::transocde() => ICULCPTranscoder::transcode not terminating string

XMLString::transocde() => ICULCPTranscoder::transcode not terminating string
----------------------------------------------------------------------------

         Key: XERCESC-1300
         URL: http://nagoya.apache.org/jira/browse/XERCESC-1300
     Project: Xerces-C++
        Type: Bug
  Components: Utilities  
    Versions: 2.5.0, 2.6.0    
 Environment: Win32, default codepage was "windows-936-2000", Xerces 2.5 with ICU 2.8
    Reporter: Scott Case
    Priority: Minor


Sometimes data doesn't get null terminated when transcoding Chinese characters from UTF-16 to windows-936-2000 via "char *XMLString::trancode(XMLCh*)".

What appears to happen is that once the call gets into the function:
"char* ICULCPTranscoder::transcode(const XMLCh* const toTranscode)"
the amount of memory allocated is too small and the first attempt to ucnv_fromUChars() fails after which the buffer is enlarged and the transcoding is attempted again.  This second call to ucnv_fromUChars() returns U_STRING_NOT_TERMINATED_WARNING which is not an error so the overall function succeeds but leaves the data unterminated.

The fix is to modify the second call to ucnv_fromUChars as shown below:
ucnv_fromUChars(fConverter,retBuf
            , targetCap   // Change to: targetCap +1
            , actualSrc, -1, &err);

A workaround is to use "bool XMLString::transcode(const XMLCh *const toTranscode, char *const toFill, const unsigned int maxChars)" which doesn't exhibit this problem.


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://nagoya.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: xerces-c-dev-unsubscribe@xml.apache.org
For additional commands, e-mail: xerces-c-dev-help@xml.apache.org


[jira] Closed: (XERCESC-1300) XMLString::transocde() => ICULCPTranscoder::transcode not terminating string

Posted by "Alberto Massari (JIRA)" <xe...@xml.apache.org>.
     [ http://issues.apache.org/jira/browse/XERCESC-1300?page=history ]
     
Alberto Massari closed XERCESC-1300:
------------------------------------

    Assign To: Alberto Massari

> XMLString::transocde() => ICULCPTranscoder::transcode not terminating string
> ----------------------------------------------------------------------------
>
>          Key: XERCESC-1300
>          URL: http://issues.apache.org/jira/browse/XERCESC-1300
>      Project: Xerces-C++
>         Type: Bug
>   Components: Utilities
>     Versions: 2.5.0, 2.6.0
>  Environment: Win32, default codepage was "windows-936-2000", Xerces 2.5 with ICU 2.8
>     Reporter: Scott Case
>     Assignee: Alberto Massari
>     Priority: Minor

>
> Sometimes data doesn't get null terminated when transcoding Chinese characters from UTF-16 to windows-936-2000 via "char *XMLString::trancode(XMLCh*)".
> What appears to happen is that once the call gets into the function:
> "char* ICULCPTranscoder::transcode(const XMLCh* const toTranscode)"
> the amount of memory allocated is too small and the first attempt to ucnv_fromUChars() fails after which the buffer is enlarged and the transcoding is attempted again.  This second call to ucnv_fromUChars() returns U_STRING_NOT_TERMINATED_WARNING which is not an error so the overall function succeeds but leaves the data unterminated.
> The fix is to modify the second call to ucnv_fromUChars as shown below:
> ucnv_fromUChars(fConverter,retBuf
>             , targetCap   // Change to: targetCap +1
>             , actualSrc, -1, &err);
> A workaround is to use "bool XMLString::transcode(const XMLCh *const toTranscode, char *const toFill, const unsigned int maxChars)" which doesn't exhibit this problem.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: xerces-c-dev-unsubscribe@xml.apache.org
For additional commands, e-mail: xerces-c-dev-help@xml.apache.org


[jira] Commented: (XERCESC-1300) XMLString::transocde() => ICULCPTranscoder::transcode not terminating string

Posted by "Scott Case (JIRA)" <xe...@xml.apache.org>.
     [ http://issues.apache.org/jira/browse/XERCESC-1300?page=comments#action_58684 ]
     
Scott Case commented on XERCESC-1300:
-------------------------------------

I rebuilt the library with version 1.18 of apache CVS file: xml-xerces/c/src/xercesc/util/Transcoders/ICU/ICUTransService.cpp 
and tested our application.  This fix resolved our issues.

Thanks!
Scott

> XMLString::transocde() => ICULCPTranscoder::transcode not terminating string
> ----------------------------------------------------------------------------
>
>          Key: XERCESC-1300
>          URL: http://issues.apache.org/jira/browse/XERCESC-1300
>      Project: Xerces-C++
>         Type: Bug
>   Components: Utilities
>     Versions: 2.5.0, 2.6.0
>  Environment: Win32, default codepage was "windows-936-2000", Xerces 2.5 with ICU 2.8
>     Reporter: Scott Case
>     Priority: Minor

>
> Sometimes data doesn't get null terminated when transcoding Chinese characters from UTF-16 to windows-936-2000 via "char *XMLString::trancode(XMLCh*)".
> What appears to happen is that once the call gets into the function:
> "char* ICULCPTranscoder::transcode(const XMLCh* const toTranscode)"
> the amount of memory allocated is too small and the first attempt to ucnv_fromUChars() fails after which the buffer is enlarged and the transcoding is attempted again.  This second call to ucnv_fromUChars() returns U_STRING_NOT_TERMINATED_WARNING which is not an error so the overall function succeeds but leaves the data unterminated.
> The fix is to modify the second call to ucnv_fromUChars as shown below:
> ucnv_fromUChars(fConverter,retBuf
>             , targetCap   // Change to: targetCap +1
>             , actualSrc, -1, &err);
> A workaround is to use "bool XMLString::transcode(const XMLCh *const toTranscode, char *const toFill, const unsigned int maxChars)" which doesn't exhibit this problem.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: xerces-c-dev-unsubscribe@xml.apache.org
For additional commands, e-mail: xerces-c-dev-help@xml.apache.org


[jira] Resolved: (XERCESC-1300) XMLString::transocde() => ICULCPTranscoder::transcode not terminating string

Posted by "Alberto Massari (JIRA)" <xe...@xml.apache.org>.
     [ http://nagoya.apache.org/jira/browse/XERCESC-1300?page=history ]
     
Alberto Massari resolved XERCESC-1300:
--------------------------------------

    Resolution: Fixed

You are right, if ICU finds the buffer too small it returns the length of the converted string (excluding the NULL terminator); but ucnv_fromUChars wants the size of the target buffer, including the space for the NULL.

The fix is in CVS. Please verify.

Alberto

> XMLString::transocde() => ICULCPTranscoder::transcode not terminating string
> ----------------------------------------------------------------------------
>
>          Key: XERCESC-1300
>          URL: http://nagoya.apache.org/jira/browse/XERCESC-1300
>      Project: Xerces-C++
>         Type: Bug
>   Components: Utilities
>     Versions: 2.5.0, 2.6.0
>  Environment: Win32, default codepage was "windows-936-2000", Xerces 2.5 with ICU 2.8
>     Reporter: Scott Case
>     Priority: Minor

>
> Sometimes data doesn't get null terminated when transcoding Chinese characters from UTF-16 to windows-936-2000 via "char *XMLString::trancode(XMLCh*)".
> What appears to happen is that once the call gets into the function:
> "char* ICULCPTranscoder::transcode(const XMLCh* const toTranscode)"
> the amount of memory allocated is too small and the first attempt to ucnv_fromUChars() fails after which the buffer is enlarged and the transcoding is attempted again.  This second call to ucnv_fromUChars() returns U_STRING_NOT_TERMINATED_WARNING which is not an error so the overall function succeeds but leaves the data unterminated.
> The fix is to modify the second call to ucnv_fromUChars as shown below:
> ucnv_fromUChars(fConverter,retBuf
>             , targetCap   // Change to: targetCap +1
>             , actualSrc, -1, &err);
> A workaround is to use "bool XMLString::transcode(const XMLCh *const toTranscode, char *const toFill, const unsigned int maxChars)" which doesn't exhibit this problem.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://nagoya.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: xerces-c-dev-unsubscribe@xml.apache.org
For additional commands, e-mail: xerces-c-dev-help@xml.apache.org