356712 – Monitor and stack depth are read from JDWP interchanged

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

Bug 356712 - Monitor and stack depth are read from JDWP interchanged

Summary: Monitor and stack depth are read from JDWP interchanged

Status:	RESOLVED FIXED

Alias:	None

Product:	JDT
Classification:	Eclipse Project
Component:	Debug (show other bugs)
Version:	3.8
Hardware:	All All

Importance:	P3 normal (vote)
Target Milestone:	3.8 M2
Assignee:	Michael Rennie
QA Contact:

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2011-09-05 04:40 EDT by aicas
Modified:	2011-09-06 12:19 EDT (History)
CC List:	1 user (show)

See Also:

Attachments
Read arguments of OwnedMonitorsStackDepthInfo in the correct order (3.54 KB, patch) 2011-09-05 04:43 EDT, aicas	no flags	Details \| Diff
patch (1.66 KB, patch) 2011-09-06 11:17 EDT, Michael Rennie	no flags	Details \| Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

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!