Project

General

Profile

Bug #10227

XML Character Escaping issue on the last chunk

Added by Teodor Gorghe about 1 year ago. Updated about 1 year ago.

Status:
Test
Priority:
Normal
Assignee:
Target version:
-
Start date:
Due date:
% Done:

100%

billable:
No
vendor_id:
GCD
case_num:
version_reported:
version_resolved:
production:
No
env_name:
topics:

History

#1 Updated by Teodor Gorghe about 1 year ago

  • Status changed from New to WIP

When reading XML into a TEMP-TABLE, there is a issue when escapes are used.

SaxReaderImpl com.ctc.wstx.exc.WstxEOFException: Unexpected EOF; was expecting a close tag for element <TagName>

The cause of this issue is EscapeControlCharsReader.read(char[], int, int).
- when there are some character escapes, there are some characters left in the buffer, which are provided into the buffer on the next EscapeControlCharsReader.read(char[], int, int) call.
- when there are some characters left into the buffer, but the FilterReader returns -1, these characters are no longer sent.

A test that illustrates this issue is the following:

#3 Updated by Teodor Gorghe about 1 year ago

  • Status changed from WIP to Review
  • % Done changed from 0 to 100

Created task branch 10227a.
Committed revision 1783 on testcases (added tests)
Committed revision 16013 on task branch 10227a.

#4 Updated by Constantin Asofiei about 1 year ago

What's the edge case here? Is this the case when:
  • there are no more characters to be read (firstBufLen < len) and super.read returns -1
  • something was read from the buffer, placed into cbuf, and we need to return the number of chars read from buffer and placed in cbuf?

#5 Updated by Teodor Gorghe about 1 year ago

Constantin Asofiei wrote:

What's the edge case here? Is this the case when:
  • there are no more characters to be read (firstBufLen < len) and super.read returns -1
  • something was read from the buffer, placed into cbuf, and we need to return the number of chars read from buffer and placed in cbuf?

Exactly, yes.

#7 Updated by Constantin Asofiei about 1 year ago

  • Status changed from Review to Merge Pending

Please check the test from #5494-3 - the file is in the archive at that task. If that works, then we can merge.

#8 Updated by Teodor Gorghe about 1 year ago

Constantin Asofiei wrote:

Please check the test from #5494-3 - the file is in the archive at that task. If that works, then we can merge.

It works:
Procedure complete. Press space bar to continue.

I am merging right now.

#9 Updated by Teodor Gorghe about 1 year ago

  • Status changed from Merge Pending to Test

Branch 10227a was merged to trunk rev 16014 and archived.

#10 Updated by Constantin Asofiei about 1 year ago

  • Project changed from Runtime Infrastructure to Base Language

Also available in: Atom PDF