Removed remaining linter warnings

diff --git a/.clang-tidy b/.clang-tidy
index e8c0c331da05c6fa26bdb96e6d70fb932ca3d81b..049204a7d1b3ed4c6b64de4bb5a6ab72532a5fce 100644 (file)
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -21,7 +21,9 @@ Checks: "*,
   -altera-id-dependent-backward-branch,
   -clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling,
   -misc-no-recursion,
-  -concurrency-mt-unsafe
+  -concurrency-mt-unsafe,
+  -clang-analyzer-unix.Malloc,
+  -modernize-macro-to-enum
 "
 WarningsAsErrors: ''
 HeaderFilterRegex: ''

diff --git a/makefile b/makefile
index da26467bd2221a98e33942e67ac7111e2147c833..0fed56feac8c529f57c0db637e9793e799038ae4 100644 (file)
--- a/makefile
+++ b/makefile
@@ -55,6 +55,10 @@ install:
 tags:
        ctags --recurse=yes --exclude=.git --exclude=bin --exclude=obj --exclude=scripts
 
+.PHONY:
+ccmds:
+       bear -- make SHELL="sh -x -e" --always-make
+
 .PHONY: format
 format:
        clang-format -i ./src/*.c ./include/*.h
@@ -63,5 +67,6 @@ format:
 lint:
        clang-tidy ./include/*.h ./src/*.c
 
+.PHONY: runtest
 runtest:
        ./$(BDIR)/$(TEST_BNAME)

diff --git a/src/main.c b/src/main.c
index 48c959fbd454fea8cd00799e86128c704a0e1f3f..bb38dee0bea1b8e388864552c48a0a55b410c867 100644 (file)
--- a/src/main.c
+++ b/src/main.c
@@ -71,5 +71,9 @@ int main(int argc, char **argv) {
 
   int res = ulas_main(cfg);
 
+  if (cfg.output_path) {
+    free(cfg.output_path);
+  }
+
   return res;
 }

diff --git a/src/ulas.c b/src/ulas.c
index 5bb481d951fc719acf4583aea201894c9af942ab..55102273f7b9270ea95dc6298fd21a3dea81b3df 100644 (file)
--- a/src/ulas.c
+++ b/src/ulas.c
@@ -56,7 +56,6 @@ int ulas_main(struct ulas_config cfg) {
     ulasout = fopen(cfg.output_path, "we");
     if (!ulasout) {
       fprintf(ulaserr, "%s: %s\n", cfg.output_path, strerror(errno));
-      free(cfg.output_path);
       return -1;
     }
   }
@@ -95,7 +94,6 @@ cleanup:
 
   if (cfg.output_path) {
     fclose(ulasout);
-    free(cfg.output_path);
   }
 
   if (cfg.argc > 0) {