URGENT: RR(XS):8015604 JDP packets containing ideographic characters are broken

Staffan Larsen staffan.larsen at oracle.com
Tue Jun 4 02:58:00 PDT 2013


Thanks for changing this.

Looks good!

/Staffan

On 4 jun 2013, at 11:50, Dmitry Samersoff <Dmitry.Samersoff at oracle.com> wrote:

> Staffan,
> 
> Two bytes len prefix is a part of JDP protocol specification, so I would
> prefer to write it explicitly.
> 
> But changed to *writeUTF()* as recommended.
> 
> see: http://cr.openjdk.java.net/~dsamersoff/8015604/webrev.02/
> 
> -Dmitry
> 
> 
> On 2013-06-04 11:48, Staffan Larsen wrote:
>> Looks good. 
>> 
>> Even better would have been to use DataOutputStream.writeUTF().
>> 
>> /Staffan
>> 
>> On 3 jun 2013, at 23:33, Dmitry Samersoff <Dmitry.Samersoff at oracle.com> wrote:
>> 
>>> Hi Everybody,
>>> 
>>> The problem:
>>> 
>>> Code uses string length rather than byte array length to write length
>>> field of JDP packet entry. So if an entry contains non-ASCII characters
>>> code produces invalid packet.
>>> 
>>> 
>>> http://cr.openjdk.java.net/~dsamersoff/8015604/webrev.01/
>>> 
>>> -Dmitry
>>> 
>>> 
>>> -- 
>>> Dmitry Samersoff
>>> Oracle Java development team, Saint Petersburg, Russia
>>> * I would love to change the world, but they won't give me the sources.
>> 
> 
> 
> -- 
> Dmitry Samersoff
> Oracle Java development team, Saint Petersburg, Russia
> * I would love to change the world, but they won't give me the sources.



More information about the serviceability-dev mailing list