BufferOverflowException in SocketChannelOutputBuffer about grizzly HOT 13 CLOSED

• Issue Imported From: javaee/grizzly#23
• Original Issue Raised By:@glassfishrobot
• Original Issue Assigned To: @glassfishrobot
• Original Issue Closed By:@glassfishrobot

from grizzly.

@glassfishrobot Commented
Reported by [email protected]

from grizzly.

@glassfishrobot Commented
[email protected] said:
Created an attachment (id=3)
patch for this issue

from grizzly.

@glassfishrobot Commented
[email protected] said:
Oops, sorry. The original code is
public void realWriteBytes(byte cbuf[], int off, int len)
throws IOException {
if (len > 0) {
if (len > outputByteBuffer.remaining())

{ ByteBuffer tmp = ByteBuffer.allocate( outputByteBuffer.capacity() * 2); outputByteBuffer.flip(); tmp.put(outputByteBuffer); outputByteBuffer = tmp; }

outputByteBuffer.put(cbuf, off, len);
}
}

from grizzly.

@glassfishrobot Commented
[email protected] said:
Thanks! It was fixed in 1.0, but I forgot to port the fix.

from grizzly.

@glassfishrobot Commented
[email protected] said:
What about the fix from 1.0:

public void realWriteBytes(byte cbuf[], int off, int len)
throws IOException {
if (len > 0) {
if (len > outputByteBuffer.remaining())

{ int size = Math.max(outputByteBuffer.capacity() * 2, len + outputByteBuffer.position()); ByteBuffer tmp = ByteBuffer.allocate(size); outputByteBuffer.flip(); tmp.put(outputByteBuffer); outputByteBuffer = tmp; }

outputByteBuffer.put(cbuf, off, len);
}
}

from grizzly.

@glassfishrobot Commented
[email protected] said:
+1

from grizzly.

@glassfishrobot Commented
[email protected] said:
Fixed. Thanks!!

[ja120114@localhost trunk]$ svn commit
Sending
trunk/modules/http/src/main/java/com/sun/grizzly/http/SocketChannelOutputBuffer.java
Transmitting file data .
Committed revision 510.

from grizzly.

@glassfishrobot Commented
[email protected] said:
Fix milestones

from grizzly.

@glassfishrobot Commented
File: ias-2007-09-17.patch
Attached By: [email protected]

from grizzly.

@glassfishrobot Commented
Was assigned to grizzly-issues

from grizzly.

@glassfishrobot Commented
This issue was imported from java.net JIRA GRIZZLY-23

from grizzly.

@glassfishrobot Commented
Marked as fixed on Wednesday, December 16th 2009, 6:14:16 pm

from grizzly.