Re: [cp-patches] Patch: SecurityManager.checkAccess broken

[Mark Wielaard]

Hi Gary,

On Thu, 2005-11-17 at 16:24 +0000, Gary Benson wrote:
> The two SecurityManager.checkAccess functions are supposed to check
> permissions when called on system threads and thread groups, but
> neither function identifies them correctly so the checks never occur.
> The attached patch fixes.

Nice catch. The comments were correct! Just our code had the logic
reversed. Doh.

> +2005-11-17  Gary Benson  <address@hidden>
> +
> +       * java/lang/SecurityManager.java (checkAccess): Correctly
> +       identify system threads (and thus check permissions for them).

Committed,

Mark

BTW If you are going to more work like this and want developer access
please sent register a user name on savannah and sent me an email to add
that user to the group.

signature.asc
Description: This is a digitally signed message part