diff options
| author | Martin Schwenke <martin@meltin.net> | 2014-03-17 13:28:14 +1100 |
|---|---|---|
| committer | Amitay Isaacs <amitay@samba.org> | 2014-03-23 04:20:14 +0100 |
| commit | a947cf6c0c3e1453ec833033dcd2edaa9490a55b (patch) | |
| tree | 614df72a698391eeaf86dd248f5a70b5e0b031e9 /ctdb/tools | |
| parent | 7fdb21cc321bc0b9a759393467fe42f26cdc812a (diff) | |
| download | samba-a947cf6c0c3e1453ec833033dcd2edaa9490a55b.tar.gz | |
ctdb-tools-ctdb: Don't close stderr when running without_daemon commands
It looks like the original without_daemon code still tried to
establish a client connection to the daemon. Closing stderr looks to
be a cheap way of hiding the errors when this failed.
However, later cleanups avoid the client connection altogether, so do
not close stderr. Now debug output from without_daemon commands
actually appears.
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Diffstat (limited to 'ctdb/tools')
| -rw-r--r-- | ctdb/tools/ctdb.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/ctdb/tools/ctdb.c b/ctdb/tools/ctdb.c index d44c3476afe..5ad5d1aadee 100644 --- a/ctdb/tools/ctdb.c +++ b/ctdb/tools/ctdb.c @@ -6433,7 +6433,6 @@ int main(int argc, const char *argv[]) DEBUG(DEBUG_ERR, ("Can't specify node(s) with \"ctdb %s\"\n", control)); exit(1); } - close(2); return ctdb_commands[i].fn(NULL, extra_argc-1, extra_argv+1); } |