Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 356712

Summary:

Monitor and stack depth are read from JDWP interchanged

Product:

[Eclipse Project] JDT

Reporter:

aicas <eclipse-bugs>

Component:

Debug

Assignee:

Michael Rennie <Michael_Rennie>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

eclipse-bugs

Version:

3.8

Target Milestone:

3.8 M2

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
Read arguments of OwnedMonitorsStackDepthInfo in the correct order	none
patch	none

Description aicas

2011-09-05 04:40:40 EDT

Build Identifier: I20110613-1736

In ThreadReferenceImpl.ownedMonitorsAndFrames(), the stack depth and subsequently the monitor is read from JVMTI via OwnedMonitorsStackDepthInfo. The specification says that the monitor and subsequently the stack depth is written however.

See: http://download.oracle.com/javase/6/docs/platform/jpda/jdwp/jdwp-protocol.html#JDWP_ThreadReference_OwnedMonitorsStackDepthInfo

Reproducible: Always

Comment 1 aicas

2011-09-05 04:43:42 EDT

Created attachment 202739 [details]
Read arguments of OwnedMonitorsStackDepthInfo in the correct order

Comment 2 Michael Rennie

2011-09-06 11:17:23 EDT

Created attachment 202812 [details]
patch

This is simply an updated of the original patch because I could not get the other one to apply.

Comment 3 Michael Rennie

2011-09-06 12:10:50 EDT

Applied patch to HEAD.

Comment 4 aicas

2011-09-06 12:19:24 EDT

(In reply to comment #3)
> Applied patch to HEAD.

Cool. Thanx!