You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@apr.apache.org by rp...@apache.org on 2009/06/10 17:41:41 UTC

svn commit: r783398 - /apr/apr/trunk/threadproc/unix/proc.c

Author: rpluem
Date: Wed Jun 10 15:41:40 2009
New Revision: 783398

URL: http://svn.apache.org/viewvc?rev=783398&view=rev
Log:
* We need to disable inheritance in the case of success like in the
  cases for stdout and stdin.

Modified:
    apr/apr/trunk/threadproc/unix/proc.c

Modified: apr/apr/trunk/threadproc/unix/proc.c
URL: http://svn.apache.org/viewvc/apr/apr/trunk/threadproc/unix/proc.c?rev=783398&r1=783397&r2=783398&view=diff
==============================================================================
--- apr/apr/trunk/threadproc/unix/proc.c (original)
+++ apr/apr/trunk/threadproc/unix/proc.c Wed Jun 10 15:41:40 2009
@@ -78,7 +78,7 @@
 
     if ((err != APR_NO_PIPE) && (err != APR_NO_FILE)) {
         if ((rv = apr_file_pipe_create_ex(&attr->parent_err, &attr->child_err,
-                                          err, attr->pool)) != APR_SUCCESS)
+                                          err, attr->pool)) == APR_SUCCESS)
             rv = apr_file_inherit_unset(attr->parent_err);
         if (rv != APR_SUCCESS)
             return rv;