diff options
author | Csaba Henk <csaba@gluster.com> | 2010-09-21 09:12:31 +0000 |
---|---|---|
committer | Vijay Bellur <vijay@dev.gluster.com> | 2010-09-21 10:41:18 -0700 |
commit | 78a09306fe1bd955616b8dd56e85f3af954c96b0 (patch) | |
tree | ba24112eac9b3507ecdb4996ae32b39aa28d1735 /Makefile.am | |
parent | eb8561e302bd83305fb093eb8cca42b4f9f62b22 (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 'Makefile.am')
0 files changed, 0 insertions, 0 deletions