From 2a2244972082c026061a08c76d03ec2f19b0d438 Mon Sep 17 00:00:00 2001 From: Anand Avati Date: Thu, 23 May 2013 23:14:30 -0700 Subject: gfapi: ignore NULL and -1 parameters in glfs_set_logging() Permit changing either logfile or level by specifying the other as -1 or NULL respectively. Change-Id: I7df05db7b613c57e0106a5cce846ae26d2a160d4 BUG: 953694 Signed-off-by: Anand Avati Reviewed-on: http://review.gluster.org/5085 Reviewed-by: Vijay Bellur Tested-by: Gluster Build System --- api/src/glfs.c | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) (limited to 'api') diff --git a/api/src/glfs.c b/api/src/glfs.c index 3962a39d..82873fe7 100644 --- a/api/src/glfs.c +++ b/api/src/glfs.c @@ -499,13 +499,16 @@ glfs_set_volfile_server (struct glfs *fs, const char *transport, int glfs_set_logging (struct glfs *fs, const char *logfile, int loglevel) { - int ret = -1; + int ret = 0; - ret = gf_log_init (fs->ctx, logfile); - if (ret) - return ret; + if (logfile) { + ret = gf_log_init (fs->ctx, logfile); + if (ret) + return ret; + } - gf_log_set_loglevel (loglevel); + if (loglevel >= 0) + gf_log_set_loglevel (loglevel); return ret; } -- cgit