]> git.ipfire.org Git - thirdparty/freeradius-server.git/commitdiff
Quiet clang scan
authorArran Cudbard-Bell <a.cudbardb@freeradius.org>
Thu, 2 Mar 2017 20:24:43 +0000 (15:24 -0500)
committerArran Cudbard-Bell <a.cudbardb@freeradius.org>
Thu, 2 Mar 2017 20:24:43 +0000 (15:24 -0500)
src/main/dl.c

index c9a01fda7cf7376571317d05911d553cf40cab56..5c3bb9ce4b3c0e35baed585317715a9b9ef533dc 100644 (file)
@@ -101,11 +101,13 @@ static int dl_module_sym_init_cmp(void const *one, void const *two)
        dl_module_sym_init_t const *a = one;
        dl_module_sym_init_t const *b = two;
 
+       rad_assert(a && b);
+
        if (a->symbol && !b->symbol) return +1;
        if (!b->symbol && a->symbol) return -1;
-       if (!a->symbol && !b->symbol) return 0;
+       if (a->symbol && b->symbol) return strcmp(a->symbol, b->symbol);
 
-       return strcmp(a->symbol, b->symbol);
+       return 0;
 }
 
 static int dl_module_sym_free_cmp(void const *one, void const *two)
@@ -113,11 +115,13 @@ static int dl_module_sym_free_cmp(void const *one, void const *two)
        dl_module_sym_free_t const *a = one;
        dl_module_sym_free_t const *b = two;
 
+       rad_assert(a && b);
+
        if (a->symbol && !b->symbol) return +1;
        if (!b->symbol && a->symbol) return -1;
-       if (!a->symbol && !b->symbol) return 0;
+       if (a->symbol && b->symbol) strcmp(a->symbol, b->symbol);
 
-       return strcmp(a->symbol, b->symbol);
+       return 0;
 }
 
 static int dl_module_sym_cmp(void const *one, void const *two)