diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2014-12-01 12:25:24 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-12-03 11:02:41 +0100 |
| commit | b4f87b42b18888c396e44e7359f7aafb092221bf (patch) | |
| tree | 0a59730a50e310e7d5cc8aaaa6b8adc06841065f /src/providers/ipa/ipa_selinux.c | |
| parent | e00c2b5ac4963de9521599c88597b7fb97339d0e (diff) | |
| download | sssd-b4f87b42b18888c396e44e7359f7aafb092221bf.tar.gz sssd-b4f87b42b18888c396e44e7359f7aafb092221bf.tar.xz sssd-b4f87b42b18888c396e44e7359f7aafb092221bf.zip | |
Add extra_args to exec_child()
Related:
https://fedorahosted.org/sssd/ticket/2503
Currently all child processes use the same arguments, the construction
of argv[] is even hardcoded in exec_child(). Add an extra_args[] array
that extends the common set of argvs so that we can have child-specific
arguments. Also adds a unit test.
Reviewed-by: Sumit Bose <sbose@redhat.com>
Diffstat (limited to 'src/providers/ipa/ipa_selinux.c')
| -rw-r--r-- | src/providers/ipa/ipa_selinux.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/providers/ipa/ipa_selinux.c b/src/providers/ipa/ipa_selinux.c index 30ad6f0a7..531258dac 100644 --- a/src/providers/ipa/ipa_selinux.c +++ b/src/providers/ipa/ipa_selinux.c @@ -1036,7 +1036,8 @@ static errno_t selinux_fork_child(struct selinux_child_state *state) if (pid == 0) { /* child */ ret = exec_child(state, pipefd_to_child, pipefd_from_child, - SELINUX_CHILD, selinux_child_debug_fd); + SELINUX_CHILD, selinux_child_debug_fd, + NULL); DEBUG(SSSDBG_CRIT_FAILURE, "Could not exec selinux_child: [%d][%s].\n", ret, sss_strerror(ret)); return ret; |