build_system: Fix some warnings highlighted by clang

Fix some warnings found with clang.

Change-Id: I5195b6189b148c2ee3ed4a19d015a6d4ef3e77bd

diff --git a/addons/res_config_mysql.c b/addons/res_config_mysql.c
index 6e6224b8d92f83702a56bc57ed2c518f76f61b8b..be5d00284ae303bb196f05c3bb4a9aaf21fac7cf 100644 (file)
--- 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);

diff --git a/apps/app_fax.c b/apps/app_fax.c
index 7822a34e80061f9a42c5d7c9731e9d91ce23bd71..88aa6ad1ae1c61846641dfcdf3a5200153b5a6cd 100644 (file)
--- a/apps/app_fax.c
+++ b/apps/app_fax.c
@@ -356,8 +356,8 @@ static int fax_generator_generate(struct ast_channel *chan, void *data, int len,
 }
 
 static struct ast_generator generator = {
-       alloc:          fax_generator_alloc,
-       generate:       fax_generator_generate,
+       .alloc = fax_generator_alloc,
+       .generate = fax_generator_generate,
 };
 
 

diff --git a/apps/app_meetme.c b/apps/app_meetme.c
index b2131ff7e8c70869459364b0273b65f2ab9560a2..7903b1d5f1c0c2eb50fc2d4c23f19947d9be7579 100644 (file)
--- a/apps/app_meetme.c
+++ b/apps/app_meetme.c
@@ -2936,7 +2936,7 @@ static void meetme_menu_admin(enum menu_modes *menu_mode, int *dtmf, struct ast_
                tweak_talk_volume(user, VOL_UP);
                break;
        default:
-               menu_mode = MENU_DISABLED;
+               *menu_mode = MENU_DISABLED;
                /* Play an error message! */
                if (!ast_streamfile(chan, "conf-errormenu", ast_channel_language(chan))) {
                        ast_waitstream(chan, "");

diff --git a/apps/app_minivm.c b/apps/app_minivm.c
index 45d04d8ba6e4752489743b81ec2e3a05fe28da04..fb7c22aa4c813cb16366b0305e930cbe13d1daea 100644 (file)
--- a/apps/app_minivm.c
+++ b/apps/app_minivm.c
@@ -2998,10 +2998,10 @@ static char *handle_minivm_list_templates(struct ast_cli_entry *e, int cmd, stru
        ast_cli(a->fd, HVLT_OUTPUT_FORMAT, "-------------", "-------", "------", "------------", "-------");
        AST_LIST_TRAVERSE(&message_templates, this, list) {
                ast_cli(a->fd, HVLT_OUTPUT_FORMAT, this->name, 
-                       this->charset ? this->charset : "-", 
-                       this->locale ? this->locale : "-",
+                       S_OR(this->charset, "-"),
+                       S_OR(this->locale, "-"),
                        this->attachment ? "Yes" : "No",
-                       this->subject ? this->subject : "-");
+                       S_OR(this->subject, "-"));
                count++;
        }
        AST_LIST_UNLOCK(&message_templates);
@@ -3069,10 +3069,10 @@ static char *handle_minivm_show_users(struct ast_cli_entry *e, int cmd, struct a
                if ((a->argc == 3) || ((a->argc == 5) && !strcmp(a->argv[4], vmu->domain))) {
                        count++;
                        snprintf(tmp, sizeof(tmp), "%s@%s", vmu->username, vmu->domain);
-                       ast_cli(a->fd, HMSU_OUTPUT_FORMAT, tmp, vmu->etemplate ? vmu->etemplate : "-", 
-                               vmu->ptemplate ? vmu->ptemplate : "-",
-                               vmu->zonetag ? vmu->zonetag : "-", 
-                               vmu->attachfmt ? vmu->attachfmt : "-",
+                       ast_cli(a->fd, HMSU_OUTPUT_FORMAT, tmp, S_OR(vmu->etemplate, "-"),
+                               S_OR(vmu->ptemplate, "-"),
+                               S_OR(vmu->zonetag, "-"),
+                               S_OR(vmu->attachfmt, "-"),
                                vmu->fullname);
                }
        }

diff --git a/channels/chan_dahdi.c b/channels/chan_dahdi.c
index ef0ad651a4c92872db4c6f2b2b51ccd7e3219696..e9d0b5bd3b9f13dad155ff8b144a077db2ff3113 100644 (file)
--- a/channels/chan_dahdi.c
+++ b/channels/chan_dahdi.c
@@ -4705,7 +4705,7 @@ static int drc_sample(int sample, float drc)
        neg = (sample < 0 ? -1 : 1);
        steep = drc*sample;
        shallow = neg*(max-max/drc)+(float)sample/drc;
-       if (abs(steep) < abs(shallow)) {
+       if (fabsf(steep) < fabsf(shallow)) {
                sample = steep;
        }
        else {

diff --git a/channels/chan_motif.c b/channels/chan_motif.c
index 24575fae014a39766f15a3b89db8012e8345cc3a..640976f096bd9b85bee7ceed43da228db5328dec 100644 (file)
--- a/channels/chan_motif.c
+++ b/channels/chan_motif.c
@@ -320,7 +320,6 @@ struct jingle_session {
        struct ast_callid *callid;            /*!< Bound session call-id */
 };
 
-static const char desc[] = "Motif Jingle Channel";
 static const char channel_type[] = "Motif";
 
 struct jingle_config {

diff --git a/channels/sig_analog.c b/channels/sig_analog.c
index 1d44a29b9b99cae4457619b78364e8823ea0ac0f..467d548e436da6fb08b3d73c6cd24524ac983a8e 100644 (file)
--- a/channels/sig_analog.c
+++ b/channels/sig_analog.c
@@ -70,7 +70,7 @@ static char analog_defaultozz[64] = "";
 
 static const struct {
        enum analog_sigtype sigtype;
-       const char const *name;
+       const char *name;
 } sigtypes[] = {
        { ANALOG_SIG_FXOLS, "fxo_ls" },
        { ANALOG_SIG_FXOKS, "fxo_ks" },
@@ -97,7 +97,7 @@ static const struct {
 
 static const struct {
        unsigned int cid_type;
-       const char const *name;
+       const char *name;
 } cidtypes[] = {
        { CID_SIG_BELL,   "bell" },
        { CID_SIG_V23,    "v23" },
@@ -2671,7 +2671,7 @@ static struct ast_frame *__analog_handle_event(struct analog_pvt *p, struct ast_
 {
        int res, x;
        int mysig;
-       enum analog_sub idx;
+       int idx;
        char *c;
        pthread_t threadid;
        struct ast_channel *chan;

diff --git a/main/manager.c b/main/manager.c
index 8478730c87dc0fdd3d523000d1e3462573e35d88..de003813c93457e61a5f23fdeec4b81317ec0a9f 100644 (file)
--- a/main/manager.c
+++ b/main/manager.c
@@ -2334,7 +2334,7 @@ static char *handle_showmanager(struct ast_cli_entry *e, int cmd, struct ast_cli
                "        write perm: %s\n"
                "   displayconnects: %s\n"
                "allowmultiplelogin: %s\n",
-               (user->username ? user->username : "(N/A)"),
+               S_OR(user->username, "(N/A)"),
                (user->secret ? "<Set>" : "(N/A)"),
                ((user->acl && !ast_acl_list_is_empty(user->acl)) ? "yes" : "no"),
                user_authority_to_str(user->readperm, &rauthority),

diff --git a/res/res_pjproject.c b/res/res_pjproject.c
index 2e092d76e6f8279ba00dc8b321471dbee057bc2f..7bf9003f9763b25c7c08915941bac948dec24a00 100644 (file)
--- a/res/res_pjproject.c
+++ b/res/res_pjproject.c
@@ -125,6 +125,7 @@ static void capture_buildopts_cb(int level, const char *data, int len)
        AST_VECTOR_ADD_SORTED(&buildopts, ast_strdup(ast_skip_blanks(data)), strcmp);
 }
 
+#pragma GCC diagnostic ignored "-Wformat-nonliteral"
 int ast_pjproject_get_buildopt(char *option, char *format_string, ...)
 {
        int res = 0;
@@ -146,6 +147,7 @@ int ast_pjproject_get_buildopt(char *option, char *format_string, ...)
 
        return res;
 }
+#pragma GCC diagnostic warning "-Wformat-nonliteral"
 
 void ast_pjproject_log_intercept_begin(int fd)
 {