Fix for regression in Gantt Project jnlp

[Deepak Bhole]

* Omair Majid <omajid at redhat.com> [2012-08-24 18:12]:
> Hi Deepak,
> 
> On 08/24/2012 04:34 PM, Deepak Bhole wrote:
> > The old patch seems to broad and does not catch the circularity error
> > anyway :( Attached is a refined patch that catches just the error we are
> > seeing.
> 
> I am fine with the patch going in into 1.3, assuming there are no other
> regressions. But please avoid this for HEAD.
> 
> Is it possible to add a reproducer along with the fix?
> 
> > Once again, ignoring this error is OK because ignoring it implies
> > permission denied, which is acceptable. This is only a stop-gap measure
> > and we are working on a proper fix for 1.4 (and perhaps a backport to
> > 1.3 later).
> 
> The error can probably be ignored safely here (though it may break an
> application if it really needs the permission). But I am afraid this is
> really an indication of a deeper bug in the classloader. This fix may
> make this particular application work, but other applications may remain
> broken if the circularity error is detected in some other part of code.
> I would prefer to fix the root cause, if at all possible, instead.
> 

Agreed. I have pushed the temporary fix and opened a bug:
http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=1145

Thanks!

Deepak