2012-10-06 16:10:38

by sven.vermeulen

Subject: [refpolicy] [PATCH v2 1/1] Allow search within postgresql var directory for the stream connect interface

Domains that are granted postgresql_stream_connect() need to be able to search
through the postgresql_var_run_t directory (in which the socket is located).

Update the interface to use the stream_connect_pattern definition to simplify
the interface and make it more readable.

Signed-off-by: Sven Vermeulen <[email protected]>
---
policy/modules/services/postgresql.if | 7 +++----
1 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/policy/modules/services/postgresql.if b/policy/modules/services/postgresql.if
index ecef19f..9d2f311 100644
--- a/policy/modules/services/postgresql.if
+++ b/policy/modules/services/postgresql.if
@@ -428,11 +428,10 @@ interface(`postgresql_stream_connect',`
type postgresql_t, postgresql_var_run_t, postgresql_tmp_t;
')

+ stream_connect_pattern($1, { postgresql_var_run_t postgresql_tmp_t }, { postgresql_var_run_t postgresql_tmp_t }, postgresql_t)
+
files_search_pids($1)
- allow $1 postgresql_t:unix_stream_socket connectto;
- allow $1 postgresql_var_run_t:sock_file write;
- # Some versions of postgresql put the sock file in /tmp
- allow $1 postgresql_tmp_t:sock_file write;
+ files_search_tmp($1)
')

########################################
--
1.7.8.6

2012-10-09 14:21:37

by cpebenito

Subject: [refpolicy] [PATCH v2 1/1] Allow search within postgresql var directory for the stream connect interface

On 10/06/12 12:10, Sven Vermeulen wrote:
> Domains that are granted postgresql_stream_connect() need to be able to search
> through the postgresql_var_run_t directory (in which the socket is located).
>
> Update the interface to use the stream_connect_pattern definition to simplify
> the interface and make it more readable.
>
> Signed-off-by: Sven Vermeulen <[email protected]>
> ---
> policy/modules/services/postgresql.if | 7 +++----
> 1 files changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/policy/modules/services/postgresql.if b/policy/modules/services/postgresql.if
> index ecef19f..9d2f311 100644
> --- a/policy/modules/services/postgresql.if
> +++ b/policy/modules/services/postgresql.if
> @@ -428,11 +428,10 @@ interface(`postgresql_stream_connect',`
> type postgresql_t, postgresql_var_run_t, postgresql_tmp_t;
> ')
>
> + stream_connect_pattern($1, { postgresql_var_run_t postgresql_tmp_t }, { postgresql_var_run_t postgresql_tmp_t }, postgresql_t)
> +
> files_search_pids($1)
> - allow $1 postgresql_t:unix_stream_socket connectto;
> - allow $1 postgresql_var_run_t:sock_file write;
> - # Some versions of postgresql put the sock file in /tmp
> - allow $1 postgresql_tmp_t:sock_file write;
> + files_search_tmp($1)
> ')
>
> ########################################

Merged.

--
Chris PeBenito
Tresys Technology, LLC
http://www.tresys.com | oss.tresys.com