sepolgen typo fix

sepolgen typo fix

[Marshall Miller]

Commit b3b3f8186ed6d56c48c4e0f997d6e6b3fd90be37 attempted to fix a bug,
but didn't.  The following patch should do it.


Marshall Miller

---
diff --git a/sepolgen/src/sepolgen/access.py
b/sepolgen/src/sepolgen/access.py
index 6de919a..71121d7 100644
--- a/sepolgen/src/sepolgen/access.py
+++ b/sepolgen/src/sepolgen/access.py
@@ -313,7 +313,7 @@ class RoleTypeSet:
 
     def __len__(self):
         """Return the unique number of role allow statements."""
-        return len(self.role_type.keys())
+        return len(self.role_types.keys())
 
     def add(self, role, type):
         if self.role_types.has_key(role):


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

Re: sepolgen typo fix

[Chad Sellers]

On 3/31/09 3:42 PM, "Marshall Miller" <mmiller@tresys.com> wrote:

> Commit b3b3f8186ed6d56c48c4e0f997d6e6b3fd90be37 attempted to fix a bug,
> but didn't.  The following patch should do it.
> 
> 
> Marshall Miller
> 
> ---
> diff --git a/sepolgen/src/sepolgen/access.py
> b/sepolgen/src/sepolgen/access.py
> index 6de919a..71121d7 100644
> --- a/sepolgen/src/sepolgen/access.py
> +++ b/sepolgen/src/sepolgen/access.py
> @@ -313,7 +313,7 @@ class RoleTypeSet:
>  
>      def __len__(self):
>          """Return the unique number of role allow statements."""
> -        return len(self.role_type.keys())
> +        return len(self.role_types.keys())
>  
>      def add(self, role, type):
>          if self.role_types.has_key(role):
> 
Looks fine.

Acked-by: Chad Sellers <csellers@tresys.com>



--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

Re: sepolgen typo fix

[Joshua Brindle]

Chad Sellers wrote:
> On 3/31/09 3:42 PM, "Marshall Miller" <mmiller@tresys.com> wrote:
> 
>> Commit b3b3f8186ed6d56c48c4e0f997d6e6b3fd90be37 attempted to fix a bug,
>> but didn't.  The following patch should do it.
>>
>>
>> Marshall Miller
>>
>> ---
>> diff --git a/sepolgen/src/sepolgen/access.py
>> b/sepolgen/src/sepolgen/access.py
>> index 6de919a..71121d7 100644
>> --- a/sepolgen/src/sepolgen/access.py
>> +++ b/sepolgen/src/sepolgen/access.py
>> @@ -313,7 +313,7 @@ class RoleTypeSet:
>>  
>>      def __len__(self):
>>          """Return the unique number of role allow statements."""
>> -        return len(self.role_type.keys())
>> +        return len(self.role_types.keys())
>>  
>>      def add(self, role, type):
>>          if self.role_types.has_key(role):
>>
> Looks fine.
> 
> Acked-by: Chad Sellers <csellers@tresys.com>
> 

Merged in sepolgen-1.0.17

--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.