Re: [PATCH] libselinux: add support for /contexts/postgresql_contexts

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

 



On Mon, 2008-06-09 at 12:07 +0900, KaiGai Kohei wrote:
> In the attached patch, the following points are changed from [1].
> 
>   [1] http://marc.info/?l=selinux&m=120999566809541&w=2
> 
> - type_transition rule of a newly created database got being described
>   as a relation ship between client and itself, like:
>   | type_transition sepgsql_client_type sepgsql_client_type:db_database sepgsql_db_t;
> 
> - neverallow rule has gone.
> 
> - allow sepgsql_unconfined_type postgresql_t:db_blob { import export }; is removed.
> 
> - A comment is added to explain dontaudit rule in row-level denied logs.
>  | # NOTE:
>  | # The purpose of the dontaudit rule in row-level access control is to prevent a flood of logs.
>  | # If a client tries to SELECT a table including violated tuples, these are filtered from
>  | # the result set as if not exist, but its access denied longs can be recorded within log files.
>  | # In generally, the number of tuples are much larger than the number of columns, tables and so on.
>  | # So, it makes a flood of logs when many tuples are violated.
>  | #
>  | # The default policy does not prevent anything for sepgsql_client_type sepgsql_unconfined_type,
>  | # so we don't need "dontaudit" rules in Type-Enforcement. However, MLS/MCS can prevent them
>  | # to access classified tuples and can make a audit record.
>  | #
>  | # Therefore, the following rule is applied for any domains which can connect SE-PostgreSQL.

I merged this, but I was thinking about some revisions that we should
consider:

1. in the unpriv client interface, we have these type transitions:
type_transition $1 sepgsql_database_type:db_table sepgsql_table_t;
type_transition $1 sepgsql_database_type:db_procedure sepgsql_proc_t;
type_transition $1 sepgsql_database_type:db_blob sepgsql_blob_t;

The client can only access the system database, not all databases, so it
seems that sepgsql_database_type should be replaced with sepgsql_db_t.

2. the stored procedure type names have been in the back of my mind for
long time but I couldn't come up with a good naming scheme.  This
especially bugged me for the sepgsql_trusted_domain_t and
sepgsql_trusted_proc_t.  Why not just go with what we do with regular
domains and executables: sepgsql_trusted_proc_t and
sepgsql_trusted_proc_exec_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