From 8083a56fe0d38549f816818ea276ec4a388402bf Mon Sep 17 00:00:00 2001 From: David Smith Date: Tue, 21 May 2013 15:51:43 -0500 Subject: [PATCH] Removed overly-paranoid comment. --- runtime/dyninst/transport.c | 9 --------- 1 file changed, 9 deletions(-) diff --git a/runtime/dyninst/transport.c b/runtime/dyninst/transport.c index eec6f6dbe..4a088bba3 100644 --- a/runtime/dyninst/transport.c +++ b/runtime/dyninst/transport.c @@ -240,15 +240,6 @@ static void __stp_d_t_run_command(char *command) { /* - * NB: Calling system() or posix_spawn from a thread could be - * dangerous (since we're calling fork()/exec() from a - * thread). If odd things start happenning, we'll need to come - * up with an alternate solution. One possible solution would - * be to fork a new process before starting the thread (and - * creating any mutexes, opening files, etc.). Then this - * thread would communicate with that new process and have it - * call system()/posix_spawn() and return the result. - * * FIXME: We'll need to make sure the output from system goes * to the correct file descriptor. We may need some posix file * actions to pass to posix_spawnp(). -- 2.43.5