RFR(S): 8067030 JDWP crash in transport_startTransport on OOM

[Dmitry Samersoff]

Serguei,

Fixed two more missed error checks (in-place, press shift-reload)

http://cr.openjdk.java.net/~dsamersoff/JDK-8067030/webrev.01/

-Dmitry

On 2014-12-10 01:30, serguei.spitsyn at oracle.com wrote:
> Hi Dmitry,
> 
> The fix looks good.
> 
> However, there are a couple of more places in that file
> where the result of the jvmtiAllocate() is not checked:
> 
>   68         utf8msg = (jbyte*)jvmtiAllocate(maxlen+1);
> 
>  393     buf = jvmtiAllocate(len*3+3);
> 
> 
> Could you fix this as well?
> 
> 
> Thanks,
> Serguei
> 
> 
> On 12/9/14 11:50 AM, Dmitry Samersoff wrote:
>> Hi Everybody,
>>
>> Please review small fix.
>>
>> http://cr.openjdk.java.net/~dsamersoff/JDK-8067030/webrev.01/
>>
>> JDWP crash if allocation fails because it calls strcpy before
>> check of allocation results.
>>
>> -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.