<AWT Dev> [12] RFR : JDK-8198000 : java/awt/List/EmptyListEventTest/EmptyListEventTest.java debug assert on Windows

Ambarish Rapte ambarish.rapte at oracle.com
Wed Oct 31 19:49:45 UTC 2018

Hi Phil & Sergey,


This issue was introduced with the fix for HYPERLINK "https://bugs.openjdk.java.net/browse/JDK-6806217"JDK-6806217, in 7b55, which modified AwtList::HandleEvent(), so It was not observed with JDK6.

                Please review the updated change as discussed offline: http://cr.openjdk.java.net/~arapte/8198000/webrev.01/





From: Phil Race 
Sent: Tuesday, October 30, 2018 2:09 AM
To: Ambarish Rapte <ambarish.rapte at oracle.com>; awt-dev at openjdk.java.net
Subject: Re: <AWT Dev> [12] RFR : JDK-8198000 : java/awt/List/EmptyListEventTest/EmptyListEventTest.java debug assert on Windows



On 10/29/18 7:31 AM, Ambarish Rapte wrote:


                Please review this Windows only fix,

                JBS: https://bugs.openjdk.java.net/browse/JDK-8198000

                Webrev: http://cr.openjdk.java.net/~arapte/8198000/webrev.00/



Test asserts with debug build jdk, only on windows.

Only the debug build turns on asserts.
But I think JDK 6 always turned on asserts, and this test was introduced in 6,
so something must have changed else we'd have seen this test fail a long time ago.
Can you identify what it was ?

Also same comment as the other bug - you need to add the bug id to the test.

Assert at Line no 77, awt_List.h :: IsItemSelected()
awt_List.cpp  ::  AwtList::HandleEvent() calls IsItemSelected() with an incorrect index value. 

Why ? 

In AwtList::HandleEvent() , the call SendListMessage(LB_ITEMFROMPOINT, 0, msg->lParam)  returns an arbitrary value 131071, which gets passed to IsItemSelected().
Could not find any relevance to the value 131071, from LB_ITEMFROMPOINT doc.

That is (128*1024)-1, so it is probably not "arbitrary".

Please add the eval above to the bug report .. once we have a complete understanding.




Fix:  Index should be verified before making call to IsItemSelected() : http://cr.openjdk.java.net/~arapte/8198000/webrev.00/


Verification:  All list tests pass.




-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20181031/5bcf5f12/attachment-0001.html>

More information about the awt-dev mailing list