<AWT Dev> [9] Review request for 8068886: IDEA IntelliJ crashes in objc_msgSend when an accessibility tool is enabled

Anton Nashatyrev anton.nashatyrev at oracle.com
Fri May 15 14:20:20 UTC 2015

Hi Sergey,

     thanks for review (and for the fix idea itself)!

     I've added some comments, you may take a look here: 

> Also please confirm that all our code follow this rule.

     I've sent you two suspects, but if this needs to fixed as well I 
suggest to make it under another bug.


On 12.05.2015 21:32, Sergey Bylokhov wrote:
> Hi, Anton.
> The fix looks fine, but I suggest to document the new resources 
> deallocation policy. I suppose documentation of loops, 
> CFRetainedResource, etc should be updated.
> Also please confirm that all our code follow this rule.
> On 08.05.15 19:52, Anton Nashatyrev wrote:
>> Hello,
>>     could you please review the following fix:
>> fix: http://cr.openjdk.java.net/~anashaty/8068886/webrev.00/ 
>> <http://cr.openjdk.java.net/%7Eanashaty/8068886/webrev.00/>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8068886
>>     Problem: the native window pointer is disposed in the inner 
>> AppKit loop, while it is still referenced on the stack in the native 
>> Cocoa method which caused the mentioned inner loop. When the inner 
>> loop is exited Cocoa crashes while dereferencing this window pointer
>>     Fix: dispose resources in the main AppKit loop only. For this a 
>> 'dispose message' posted which is processed only on the main loop.
>> Thanks!
>> Anton.
> -- 
> Best regards, Sergey.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20150515/01e76464/attachment.html>

More information about the awt-dev mailing list