]> git.ipfire.org Git - thirdparty/freeradius-server.git/commitdiff
clang scan fixes
authorArran Cudbard-Bell <a.cudbardb@freeradius.org>
Wed, 5 Apr 2023 01:31:06 +0000 (19:31 -0600)
committerArran Cudbard-Bell <a.cudbardb@freeradius.org>
Wed, 5 Apr 2023 13:48:53 +0000 (07:48 -0600)
src/lib/curl/io.c
src/lib/server/log.c
src/lib/util/event.c
src/modules/rlm_rest/rest.c

index 0486326ab32f8783ca42cf541702f91cf60d9e25..b1cba5fa6860c1899b3f96af14cb8f3e04687a31 100644 (file)
@@ -575,7 +575,7 @@ fr_curl_handle_t *fr_curl_io_init(TALLOC_CTX *ctx,
        CURLMcode               ret;
        CURLM                   *mandle;
        fr_curl_handle_t        *mhandle;
-       char const              *option = "unknown";
+       char const              *option;
 
        mandle = curl_multi_init();
        if (!mandle) {
index 98d4e193a507af388d6e3340fb6ed25aee89b8ae..b5909a864d509ba1876106fd745c949a32bdfafc 100644 (file)
@@ -297,7 +297,7 @@ void vlog_request(fr_log_type_t type, fr_log_lvl_t lvl, request_t *request,
        char const      *fmt_location = "";
        char const      *fmt_prefix = "";
        char const      *fmt_module = "";
-       char const      *fmt_exp = "";
+       char const      *fmt_exp;
 
        fr_log_t        *log_dst = uctx;
        TALLOC_CTX      *pool;
index 42fe96339693e8e236d8457c1fb4069e33fa1501..5639b386471ade02e3b8582c1d373057ff16fcf9 100644 (file)
@@ -1389,14 +1389,18 @@ static int _event_timer_free(fr_event_timer_t *ev)
                (void) fr_dlist_remove(&el->ev_to_add, ev);
        } else {
                int             ret = fr_lst_extract(el->times, ev);
-               char const      *err_file = "not-available";
-               int             err_line = 0;
+               char const      *err_file;
+               int             err_line;
 
 #ifndef NDEBUG
                err_file = ev->file;
                err_line = ev->line;
+#else
+               err_file = "not-available";
+               err_line = 0;
 #endif
 
+
                /*
                 *      Events MUST be in the lst (or the insertion list).
                 */
@@ -1504,14 +1508,17 @@ int _fr_event_timer_at(NDEBUG_LOCATION_ARGS
                 */
                if (!fr_dlist_entry_in_list(&ev->entry)) {
                        int             ret;
-                       char const      *err_file = "not-available";
-                       int             err_line = 0;
+                       char const      *err_file;
+                       int             err_line;
 
                        ret = fr_lst_extract(el->times, ev);
 
 #ifndef NDEBUG
                        err_file = ev->file;
                        err_line = ev->line;
+#else
+                       err_file = "not-available";
+                       err_line = 0;
 #endif
 
                        /*
index 9bf2faea9f3782d1a91f66786151e08f720e66ef..c878d09e904439ef2b99b32cd20d731e752e2f80 100644 (file)
@@ -1736,7 +1736,7 @@ int rest_request_config(module_ctx_t const *mctx, rlm_rest_section_t const *sect
        http_auth_type_t        auth = section->auth;
 
        CURLcode                ret = CURLE_OK;
-       char const              *option = "unknown";
+       char const              *option;
 
        char                    buffer[512];
        bool                    content_type_set = false;