summaryrefslogtreecommitdiffstats
path: root/booster/src
diff options
context:
space:
mode:
authorCsaba Henk <csaba@gluster.com>2010-09-21 09:12:31 +0000
committerVijay Bellur <vijay@dev.gluster.com>2010-09-21 10:41:18 -0700
commit78a09306fe1bd955616b8dd56e85f3af954c96b0 (patch)
treeba24112eac9b3507ecdb4996ae32b39aa28d1735 /booster/src
parenteb8561e302bd83305fb093eb8cca42b4f9f62b22 (diff)
libglusterfs: handle failure properly in gf_system()
- Child must be terminated if execve(2) fails, otherwise it will deadlock parent - Status, as of given by waitpid(2), is aggregated data, should not be returned as is. In fact, there is not much point in threading up the exitvalue: callers usually return gf_system result as is, and according to glusterfs conventions, they should return -1 on failure. Therefore now we check only for success/fail, and return 0/-1. Signed-off-by: Csaba Henk <csaba@gluster.com> Signed-off-by: Vijay Bellur <vijay@dev.gluster.com> BUG: 1665 (glusterd hangs if spawning a child is failed) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1665
Diffstat (limited to 'booster/src')
0 files changed, 0 insertions, 0 deletions