Re: [patch 03/35] logrotate policy update

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2008-08-04 at 14:34 +0200, david@xxxxxxxxxxx wrote:
> plain text document attachment (policy_modules_admin_logrotate.patch)
> Perhaps controversial changes
> 
> First sent on Jul 19, dwalsh described the need for the patch here:
> http://marc.info/?l=selinux&m=121726318902852&w=2
> 
> Refreshed to apply cleanly to SVN repo
> 
> Index: refpolicy/policy/modules/admin/logrotate.te
> ===================================================================
> --- refpolicy.orig/policy/modules/admin/logrotate.te	2008-08-03 16:47:00.000000000 +0200
> +++ refpolicy/policy/modules/admin/logrotate.te	2008-08-03 16:59:10.000000000 +0200
> @@ -97,6 +97,7 @@
>  files_read_etc_files(logrotate_t)
>  files_read_etc_runtime_files(logrotate_t)
>  files_read_all_pids(logrotate_t)
> +files_search_all(logrotate_t)
>  # Write to /var/spool/slrnpull - should be moved into its own type.
>  files_manage_generic_spool(logrotate_t)
>  files_manage_generic_spool_dirs(logrotate_t)

Still don't like this.

> @@ -142,9 +143,8 @@
>  ')
>  
>  optional_policy(`
> -	apache_read_config(logrotate_t)
> -	apache_domtrans(logrotate_t)
>  	apache_signull(logrotate_t)
> +	apache_manage_all_content(logrotate_t)
>  ')
>  
>  optional_policy(`

Definitely need justification for this; these aren't even logs.

> @@ -186,6 +186,5 @@
>  ')
>  
>  optional_policy(`
> -	# cjp: why?
> -	squid_domtrans(logrotate_t)
> +	squid_signal(logrotate_t)
>  ')
> 
-- 
Chris PeBenito
Tresys Technology, LLC
(410) 290-1411 x150


--
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.

[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux