Stephen Smalley wrote:
> On Tue, 2008-08-12 at 13:59 -0400, Joshua Brindle wrote:
>> Send again with the right date and time ;)
>>
>> This removes the (apparently) unused expand_rule function
>>
>> Signed-off-by: Joshua Brindle <method@xxxxxxxxxxxxxxx>
>
> Have you checked the history to see when/why this function stopped being
> used? And whether any external users like setools are using it?
>
it looks like it came in with the original expand patch and wasn't even in use then. SETools also does not use it.
>> ---
>>
>> libsepol/src/expand.c | 29 -----------------------------
>> 1 files changed, 0 insertions(+), 29 deletions(-)
>>
>> diff --git a/libsepol/src/expand.c b/libsepol/src/expand.c
>> index 3194f8a..14dc4fc 100644
>> --- a/libsepol/src/expand.c
>> +++ b/libsepol/src/expand.c
>> @@ -1917,35 +1917,6 @@ int expand_convert_type_set(policydb_t * p, uint32_t * typemap,
>> return 0;
>> }
>>
>> -/* Expand a rule into a given avtab - checking for conflicting type
>> - * rules. Return 1 on success, 0 if the rule conflicts with something
>> - * (and hence was not added), or -1 on error. */
>> -int expand_rule(sepol_handle_t * handle,
>> - policydb_t * source_pol,
>> - avrule_t * source_rule, avtab_t * dest_avtab,
>> - cond_av_list_t ** cond, cond_av_list_t ** other, int enabled)
>> -{
>> - int retval;
>> - ebitmap_t stypes, ttypes;
>> -
>> - if (source_rule->specified & AVRULE_NEVERALLOW)
>> - return 1;
>> -
>> - ebitmap_init(&stypes);
>> - ebitmap_init(&ttypes);
>> -
>> - if (type_set_expand(&source_rule->stypes, &stypes, source_pol, 1))
>> - return -1;
>> - if (type_set_expand(&source_rule->ttypes, &ttypes, source_pol, 1))
>> - return -1;
>> - retval = expand_rule_helper(handle, source_pol, NULL,
>> - source_rule, dest_avtab,
>> - cond, other, enabled, &stypes, &ttypes);
>> - ebitmap_destroy(&stypes);
>> - ebitmap_destroy(&ttypes);
>> - return retval;
>> -}
>> -
>> int role_set_expand(role_set_t * x, ebitmap_t * r, policydb_t * p, uint32_t * rolemap)
>> {
>> unsigned int i;
>>
>>
>> --
>> This message was distributed to subscribers of the selinux mailing list.
>> If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with
>> the words "unsubscribe selinux" without quotes as the message.
--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with
the words "unsubscribe selinux" without quotes as the message.
