Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 276921 - JVM crash on attach in heapprof enabled mode
Summary: JVM crash on attach in heapprof enabled mode
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: TPTP (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P2 major (vote)
Target Milestone: ---   Edit
Assignee: Jonathan West CLA
QA Contact: Kathy Chan CLA
URL:
Whiteboard: Milestone2
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-19 11:10 EDT by Kathy Chan CLA
Modified: 2016-05-05 10:59 EDT (History)
10 users (show)

See Also:
oec: pmc_approved+
kathy: pmc_approved? (chris.l.elford)
sluiman: pmc_approved+
ernest: pmc_approved+
kathy: pmc_approved? (kathy)
paulslau: pmc_approved+
ewchan: pmc_approved+


Attachments
Patch - 4521 - JVM crash on attach in heapprof enabled mode (972 bytes, patch)
2009-05-19 11:42 EDT, Jonathan West CLA
no flags Details | Diff
Patch for updating copyright year to 2009 (4.54 KB, patch)
2009-05-20 09:44 EDT, Kathy Chan CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kathy Chan CLA 2009-05-19 11:10:39 EDT
Removing all filters and attaching to HeapJniCarTest in HeapProf enabled mode
causes JVM crash.

This problem was fixed in HEAD under bug 275321 for TPTP 4.6.  This defect address back porting bug 275321 to TPTP 4.5.2.1 Milestone 2.

Please see bug 275321 for more details.
Comment 1 Jonathan West CLA 2009-05-19 11:42:46 EDT
Created attachment 136321 [details]
Patch - 4521 - JVM crash on attach in heapprof enabled mode
Comment 2 Jonathan West CLA 2009-05-19 11:47:15 EDT
Requesting approval to include the following defect in 4.5.2.1 branch, on behalf of Chengrui Deng.

Explain why you believe this is a stop-ship defect. How does the defect
manifest itself, and how will users of TPTP / consuming products be affected if
the defect is not fixed? 
Removing all filters and attaching to an agent in HeapProf enabled mode causes JVM crash (using Sun JVM).

Is there a work-around? If so, why do you believe the work-around is insufficient? 
No.

Is this a regression or API breakage? Explain. 
Yes, this functionality worked previously.

Does this require new API? 
No.

Who performed the code review? 
Myself, as Chengrui produced the patch.

Is there a test case attached to the bugzilla record? 
No.

What is the nature of the fix? What is the scope of the fix? What is the risk associated with this fix? 
Fix is a simple one-line 'is null' check to one of the heap prof classes. See original bug for details. Risk is low.

Is this fix related to any standards that TPTP adheres to? If so, who has validated that the fix continues to adhere to the standard? 
No.
Comment 3 Kathy Chan CLA 2009-05-19 11:58:17 EDT
PMC,

Yunan has requested to have bug 275321 backported to TPTP 4.5.2.1.  We are now seeking PMC's approval to check this into TPTP 4.5.2.1 Milestone 2.
Comment 4 Paul Slauenwhite CLA 2009-05-19 12:04:09 EDT
PMC approved assuming the scenario is covered by an existing test case.  Jonathan, please confirm.
Comment 5 Eugene Chan CLA 2009-05-19 12:11:24 EDT
patch looks good. this bug is a regression to users and should be fixed.
Comment 6 Jonathan West CLA 2009-05-19 12:17:55 EDT
Indeed you are correct, Paul. 
Patched checked into HEAD w/ PMC approval.
Comment 7 Paul Slauenwhite CLA 2009-05-19 12:28:42 EDT
(In reply to comment #4)
> PMC approved assuming the scenario is covered by an existing test case. 
> Jonathan, please confirm.
> 

Re-submitting my approval.
Comment 8 Kathy Chan CLA 2009-05-20 09:44:05 EDT
Created attachment 136489 [details]
Patch for updating copyright year to 2009

A few of the files need copyright years updated to 2009.  Here's a patch with the changes.  I'm checking it into the TPTP-4_5_2_1 branch.
Comment 9 Kathy Chan CLA 2009-10-16 15:52:36 EDT
Verified on M2.