summaryrefslogtreecommitdiffstats
path: root/src/tests
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2015-01-07 10:36:12 +0100
committerJakub Hrozek <jhrozek@redhat.com>2015-01-13 17:50:05 +0100
commitd31fba405f3392f27f0eea861834083cfaa0ef10 (patch)
tree54d87d664d934d3e8edc4403655b61febd894138 /src/tests
parent9740d8a555dee2e41294f6e4acf558e4998656d9 (diff)
downloadsssd-d31fba405f3392f27f0eea861834083cfaa0ef10.tar.gz
sssd-d31fba405f3392f27f0eea861834083cfaa0ef10.tar.xz
sssd-d31fba405f3392f27f0eea861834083cfaa0ef10.zip
UTIL: Allow dup-ing child pipe to a different FD
Related to: https://fedorahosted.org/sssd/ticket/2544 Adds a new function exec_child_ex and moves setting the extra_argv[] to exec_child_ex() along with specifying the input and output fds. Reviewed-by: Sumit Bose <sbose@redhat.com> (cherry picked from commit 16cb0969f0a9ea71524d852077d6a480740d4f12)
Diffstat (limited to 'src/tests')
-rw-r--r--src/tests/cmocka/test_child_common.c11
1 files changed, 6 insertions, 5 deletions
diff --git a/src/tests/cmocka/test_child_common.c b/src/tests/cmocka/test_child_common.c
index ee6e297f5..23e14ce49 100644
--- a/src/tests/cmocka/test_child_common.c
+++ b/src/tests/cmocka/test_child_common.c
@@ -89,7 +89,7 @@ void test_exec_child(void **state)
ret = exec_child(child_tctx,
child_tctx->pipefd_to_child,
child_tctx->pipefd_from_child,
- CHILD_DIR"/"TEST_BIN, 2, NULL);
+ CHILD_DIR"/"TEST_BIN, 2);
assert_int_equal(ret, EOK);
} else {
do {
@@ -128,10 +128,11 @@ void test_exec_child_extra_args(void **state)
child_pid = fork();
assert_int_not_equal(child_pid, -1);
if (child_pid == 0) {
- ret = exec_child(child_tctx,
- child_tctx->pipefd_to_child,
- child_tctx->pipefd_from_child,
- CHILD_DIR"/"TEST_BIN, 2, extra_args);
+ ret = exec_child_ex(child_tctx,
+ child_tctx->pipefd_to_child,
+ child_tctx->pipefd_from_child,
+ CHILD_DIR"/"TEST_BIN, 2, extra_args,
+ STDIN_FILENO, STDOUT_FILENO);
assert_int_equal(ret, EOK);
} else {
do {