[patch] additional temp directory?

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[patch] additional temp directory?

DJ Delorie-2

When selinux is active, libffi might need a libffi-specific tmp
directory that allows both write and exec maps.  Can we add such a
variable?  Plus or minus any bikesheeding about its name or position
in the list?  ;-)

diff --git a/src/closures.c b/src/closures.c
index adb60bb..337b22b 100644
--- a/src/closures.c
+++ b/src/closures.c
@@ -664,6 +664,7 @@ static struct
   const char *arg;
   int repeat;
 } open_temp_exec_file_opts[] = {
+  { open_temp_exec_file_env, "LIBFFITMPDIR", 0 },
   { open_temp_exec_file_env, "TMPDIR", 0 },
   { open_temp_exec_file_dir, "/tmp", 0 },
   { open_temp_exec_file_dir, "/var/tmp", 0 },

See also https://bugzilla.redhat.com/show_bug.cgi?id=1722756
Reply | Threaded
Open this post in threaded view
|

Re: [patch] additional temp directory?

Sourceware - libffi-discuss mailing list

Ping?

DJ Delorie <[hidden email]> writes:

> When selinux is active, libffi might need a libffi-specific tmp
> directory that allows both write and exec maps.  Can we add such a
> variable?  Plus or minus any bikesheeding about its name or position
> in the list?  ;-)
>
> diff --git a/src/closures.c b/src/closures.c
> index adb60bb..337b22b 100644
> --- a/src/closures.c
> +++ b/src/closures.c
> @@ -664,6 +664,7 @@ static struct
>    const char *arg;
>    int repeat;
>  } open_temp_exec_file_opts[] = {
> +  { open_temp_exec_file_env, "LIBFFITMPDIR", 0 },
>    { open_temp_exec_file_env, "TMPDIR", 0 },
>    { open_temp_exec_file_dir, "/tmp", 0 },
>    { open_temp_exec_file_dir, "/var/tmp", 0 },
>
> See also https://bugzilla.redhat.com/show_bug.cgi?id=1722756