[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [cp-patches] Patch: SecurityManager.checkAccess broken
From: |
Mark Wielaard |
Subject: |
Re: [cp-patches] Patch: SecurityManager.checkAccess broken |
Date: |
Fri, 18 Nov 2005 01:07:53 +0100 |
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