<AWT Dev> [14] RFR 8230782: Robot.createScreenCapture() fails if “awt.robot.gtk” is set to false

[Sergey Bylokhov]

On 9/20/19 3:54 am, Dmitry Markov wrote:
> Hi Sergey,
> 
> That’s right, this one and JDK-4392080 are devoted to the same problem. I have opened the bug against “xorg.app.xwd” - https://gitlab.freedesktop.org/xorg/app/xwd/issues/3

Thank you, looks fine.

> 
> Thanks,
> Dmitry
> 
>> On 19 Sep 2019, at 20:26, Sergey Bylokhov <Sergey.Bylokhov at oracle.com <mailto:Sergey.Bylokhov at oracle.com>> wrote:
>>
>> Hi, Dmitry.
>>
>> It is interesting that previously this bug was fixed loooong time ago:
>> https://bugs.openjdk.java.net/browse/JDK-4392080
>>
>> Looks like the evaluation of that bug is the same as the current one, right?
>>
>> Can we report this bug upstream, and provide the patch with a proper description of the bug?
>>
>>
>> On 9/19/19 11:25 am, Dmitry Markov wrote:
>>> Thank you for the review, Phil!
>>> I am afraid we cannot create a regression test for this since the issue takes place only in VNC environment. I have added “noreg-hard” to the bug.
>>> Thanks,
>>> Dmitry
>>>> On 19 Sep 2019, at 17:58, Phil Race <philip.race at oracle.com <mailto:philip.race at oracle.com><mailto:philip.race at oracle.com>> wrote:
>>>>
>>>>
>>>> +1 to the fix but any ideas for a regression test.
>>>> There are no related keywords on the bug ..
>>>>
>>>> -phil.
>>>>
>>>>
>>>> On 9/19/19 9:12 AM, Dmitry Markov wrote:
>>>>> Hello,
>>>>>
>>>>> Could you review a fix for jdk14, please?
>>>>>
>>>>>  bug: https://bugs.openjdk.java.net/browse/JDK-8230782
>>>>>  webrev: http://cr.openjdk.java.net/~dmarkov/8230782/webrev.00/
>>>>>
>>>>> Problem description:
>>>>> After integration of JDK-8210776 [1] we started to use malloc() function for allocation of XColor array in QueryColorMap() (see multiVis.c). However malloc() does not initialise allocated memory. So if the memory contains some “garbage” we may get incorrect data inside the array. As a result Robot.createsScreenCapture() retrieves incorrect colours.
>>>>>
>>>>> Fix:
>>>>> It is necessary to invoke calloc() for allocation of the array as it used to be before XWD upgrade
>>>>>
>>>>> Thanks,
>>>>> Dmitry
>>>>>
>>>>> [1] - https://bugs.openjdk.java.net/browse/JDK-8210776
>>>>
>>
>>
>> --
>> Best regards, Sergey.
> 


-- 
Best regards, Sergey.