RFR: 8225715: jhsdb jmap fails to write binary heap dump of a jshell process

[Fairoz Matte]

Hi Chris,

Thanks for the review.

Thanks,
Fairoz

> -----Original Message-----
> From: Chris Plummer
> Sent: Monday, July 01, 2019 11:21 PM
> To: Fairoz Matte <fairoz.matte at oracle.com>; serviceability-
> dev at openjdk.java.net
> Subject: Re: RFR: 8225715: jhsdb jmap fails to write binary heap dump of a
> jshell process
> 
> Looks good.
> 
> Chris
> 
> On 6/30/19 9:31 PM, Fairoz Matte wrote:
> > Hi,
> >
> > Please review a fix that potentially handle NPE and allow heap to dump
> > from a process were we get source file name as null.
> >
> > Background:
> > For taking heap dump of JShell process, we get getSourceFileName() as
> null.
> > Current implementation doesn't have null check.
> > This fix will handle the null check and avoid breaking in call to
> writeObjectID(sym).
> > Regression test provided with patch will demonstrate the problem.
> >
> > JBS bug - https://bugs.openjdk.java.net/browse/JDK-8225715
> > Webrev - http://cr.openjdk.java.net/~fmatte/8225715/webrev.00/
> >
> > Testing: Mach5 tier1,tier2 tier3, hs-tier1, hs-tier2 and hs-tier3
> > tests successfully passed
> >
> > Thanks,
> > Fairoz
> >
>