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:44:53 UTC

svn commit: r783400 - in /apr/apr/branches/1.3.x: ./ threadproc/unix/proc.c

Author: rpluem
Date: Wed Jun 10 15:44:50 2009
New Revision: 783400

URL: http://svn.apache.org/viewvc?rev=783400&view=rev
Log:
Merge r783398 from trunk:

* We need to disable inheritance in the case of success like in the
  cases for stdout and stdin.

Submitted by: rpluem
Reviewed by: rpluem

Modified:
    apr/apr/branches/1.3.x/   (props changed)
    apr/apr/branches/1.3.x/threadproc/unix/proc.c

Propchange: apr/apr/branches/1.3.x/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Wed Jun 10 15:44:50 2009
@@ -1 +1 @@
-/apr/apr/trunk:712674,733052,742752,782838
+/apr/apr/trunk:712674,733052,742752,782838,783398

Modified: apr/apr/branches/1.3.x/threadproc/unix/proc.c
URL: http://svn.apache.org/viewvc/apr/apr/branches/1.3.x/threadproc/unix/proc.c?rev=783400&r1=783399&r2=783400&view=diff
==============================================================================
--- apr/apr/branches/1.3.x/threadproc/unix/proc.c (original)
+++ apr/apr/branches/1.3.x/threadproc/unix/proc.c Wed Jun 10 15:44:50 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;