search

javaee / jaxb-v2

Other
211 stars 99 forks source link

FastInfosetConnector fails to process fragmented data #719

Closed glassfishrobot closed 14 years ago

glassfishrobot commented 14 years ago

Simple Webservice [1] fails, when "image" data comes in fragmented chunks.

[1] @WebService(serviceName = "storeImageService", portName = "storeImagePort", targetNamespace = "http://mobileid.nec.co.nz/wsdl/mobileId/storeImage") public class ImageStoreService {

public java.lang.String storeImageOperation(byte[] image)

{ return "ok"; }

}

Environment

Operating System: All Platform: Macintosh

Affected Versions

[2.2]

glassfishrobot commented 14 years ago

Reported by oleksiys

glassfishrobot commented 14 years ago

oleksiys said: removed "base64" from summary. It's not essential.

glassfishrobot commented 14 years ago

oleksiys said: Proposed patch:

Index: com/sun/xml/bind/v2/runtime/unmarshaller/FastInfosetConnector.java

==== RCS file: /cvs/jaxb2-sources/jaxb-ri/runtime- fi/src/com/sun/xml/bind/v2/runtime/unmarshaller/FastInfosetConnector.java,v retrieving revision 1.9.2.3 diff -r1.9.2.3 FastInfosetConnector.java 167c167 < —

169,171c169,171 < buffer.append(fastInfosetStreamReader.accessTextCharacters(), < fastInfosetStreamReader.accessTextStart(), < fastInfosetStreamReader.accessTextLength()); — buffer.append(fastInfosetStreamReader.getTextCharacters(), fastInfosetStreamReader.getTextStart(), fastInfosetStreamReader.getTextLength()); 187,189c187,189 < buffer.append(fastInfosetStreamReader.accessTextCharacters(), < fastInfosetStreamReader.accessTextStart(), < fastInfosetStreamReader.accessTextLength()); — buffer.append(fastInfosetStreamReader.getTextCharacters(), fastInfosetStreamReader.getTextStart(), fastInfosetStreamReader.getTextLength()); 225,227c225,227 < ch = fastInfosetStreamReader.accessTextCharacters(); < start = fastInfosetStreamReader.accessTextStart(); < length = fastInfosetStreamReader.accessTextLength(); — ch = fastInfosetStreamReader.getTextCharacters(); start = fastInfosetStreamReader.getTextStart(); length = fastInfosetStreamReader.getTextLength();

glassfishrobot commented 14 years ago

@pavelbucek said: reassigning

glassfishrobot commented 14 years ago

@pavelbucek said: committed into the trunk

glassfishrobot commented 14 years ago

@pavelbucek said: tests are ok, merged into branches.

glassfishrobot commented 7 years ago

This issue was imported from java.net JIRA JAXB-719

glassfishrobot commented 14 years ago

Marked as fixed on Thursday, November 19th 2009, 6:25:28 pm