diff options
author | George Joseph <george.joseph@fairview5.com> | 2016-02-01 12:04:06 -0700 |
---|---|---|
committer | George Joseph <george.joseph@fairview5.com> | 2016-02-01 18:20:05 -0700 |
commit | 8804d0973c9144da4b3af863a24a8dfaac22c04e (patch) | |
tree | 0662c58f11d7483f7980a116b225c6443c40c639 /addons | |
parent | 109b0aff6bcd73a634f440b6c8ec84a155e4ba66 (diff) |
build_system: Fix some warnings highlighted by clang
Fix some warnings found with clang.
Change-Id: I5195b6189b148c2ee3ed4a19d015a6d4ef3e77bd
Diffstat (limited to 'addons')
-rw-r--r-- | addons/res_config_mysql.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/addons/res_config_mysql.c b/addons/res_config_mysql.c index 6e6224b8d..be5d00284 100644 --- a/addons/res_config_mysql.c +++ b/addons/res_config_mysql.c @@ -1518,7 +1518,7 @@ static int load_mysql_config(struct ast_config *config, const char *category, st ast_debug(1, "MySQL RealTime database name: %s\n", conn->name); ast_debug(1, "MySQL RealTime user: %s\n", conn->user); ast_debug(1, "MySQL RealTime password: %s\n", conn->pass); - if(conn->charset) + if(!ast_strlen_zero(conn->charset)) ast_debug(1, "MySQL RealTime charset: %s\n", conn->charset); return 1; @@ -1533,13 +1533,13 @@ static int mysql_reconnect(struct mysql_conn *conn) /* mutex lock should have been locked before calling this function. */ reconnect_tryagain: - if ((!conn->connected) && (!ast_strlen_zero(conn->host) || conn->sock) && !ast_strlen_zero(conn->user) && !ast_strlen_zero(conn->name)) { + if ((!conn->connected) && (!ast_strlen_zero(conn->host) || !ast_strlen_zero(conn->sock)) && !ast_strlen_zero(conn->user) && !ast_strlen_zero(conn->name)) { if (!mysql_init(&conn->handle)) { ast_log(LOG_WARNING, "MySQL RealTime: Insufficient memory to allocate MySQL resource.\n"); conn->connected = 0; return 0; } - if(conn->charset && strlen(conn->charset) > 2){ + if(strlen(conn->charset) > 2){ char set_names[255]; char statement[512]; snprintf(set_names, sizeof(set_names), "SET NAMES %s", conn->charset); |