OpenJDK6 regression, javax.swing.UIDefaults broken

[Alexander Potochkin]

Hello Phil
Hello Clement

Indeed, it was an urgent fix,
to make it work CR #6795356 has to be backported to 6-open

when it is there, the problem will be gone

I am not sure who takes care of 6-open,
Phil could you help me to pass this information to the 6-open team?

Thanks
alexp

> SFAIK bugzilla is still officially just for tracking patches.
> I've cc'd the engineer who created the 6 open fix. Hopefully that
> should be all that's needed.
> The fix looks identical to the 6u22 one but the problem may be
> that 6-open was forked off an early jdk7 build so that might mean
> there's an issue resulting from that divergence, but that's just a guess.
>
> BTW there is a specific 6-dev list and I think any further discussion
> ought to move there
> after this reply: So I've cc'd jdk6-dev at openjdk.java.net
>
> -phil.
>
> Clemens Eisserer wrote:
>> Hi,
>>
>> After updating OpenJDK6 recently an application I've written is
>> broken, when running it as Applet in an AppletViewer.
>> The app displays boolean values in a JTable, using
>> JTable.getDefaultRenderer(Class) to get the CellRenderer.
>> After digging a bit deeper, I found the problem is the
>> checkAccess()-call when loading "javax.swing.JTable$BooleanRenderer"
>> in javax.swing.UIDefaults.java : 1103.
>>
>> The code there looks like this:
>>> c = Class.forName(className, true, (ClassLoader)cl);
>>> checkAccess(c.getModifiers()); //Fails, unfourtunatly
>>> Exception-Handler kills Exception, so no idea what it is.
>>> if (methodName != null) {
>>
>> In a 2-3 weeks old OpenJDK7 repo, I didn't find the
>> checkAccess()-Call, it simply says:
>>> c = Class.forName(className, true, (ClassLoader)cl);
>>> if (methodName != null) {
>>
>> Looks like some kind of emergency security fix ;)
>> Oracle's JDK6u22 works fine however.
>>
>> Should I report this against bugs.sun.com or the OpenJDK bugzilla?
>> I know for now only OpenJDK seems affected, but I wonder if the oracle
>> guys look at bugzilla at all?
>>
>> Thanks, Clemens
>>
>> PS: Tetscase attached.
>