[rfc][icedtea-web][policyeditor] Fix for duplicate codebases when launched from dialog
Lukasz Dracz
ldracz at redhat.com
Tue Jun 17 18:59:14 UTC 2014
Hello
> When launching PolicyEditor from a security dialog, the applet's
> codebase may be added to the UI twice. This only affects the list UI -
> each entry shares the same checkbox state, custom permissions, etc., and
> will be written to file as one entry. This happens partially because
> there was some missing validation that the element doesn't already exist
> before it is re-added to the list, and partially because in another
> place where validation was done, the check was performed outside of a
> SwingUtilities.invokeLater block which would later update the UI with
> the new element, without re-checking its presence. This could cause a
> race condition where the element was not yet in the list when it was
> checked but could have already been added by the time it the invokeLater
> ran. This patch fixes these two errors.
>
> http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=1850
Looks good to me, +0.5!
Thank you,
Lukasz D
More information about the distro-pkg-dev
mailing list