minor cleanup for code scan

.github/workflows/ci.yml

@@ -41,7 +41,9 @@ jobs:
       uses: actions/checkout@v4
 
     - name: Run clang-format
-      run: ./scripts/ci-run-clang-format.sh
+      run: |
+        sudo ln -sf /usr/bin/clang-format-15 /usr/bin/clang-format
+        ./scripts/ci-run-clang-format.sh
 
   ci:
     name: ci

app/rm.c

@@ -95,7 +95,7 @@ int ZSV_MAIN_NO_OPTIONS_FUNC(ZSV_COMMAND)(int argc, const char *argv[]) {
           printf("Are you sure you want to remove the file %s%s?\n", filepath,
                  remove_cache ? " and all of its cache contents" : "");
         char buff[64];
-        if (fscanf(stdin, "%60s", buff) && strchr("Yy", buff[0]))
+        if (fscanf(stdin, "%60s", buff) == 1 && strchr("Yy", buff[0]))
           ok = 1;
       }
 #endif

app/select-pull.c

@@ -24,6 +24,7 @@
 #include <zsv/utils/utf8.h>
 #include <zsv/utils/string.h>
 #include <zsv/utils/mem.h>
+#include <zsv/utils/arg.h>
 
 struct zsv_select_search_str {
   struct zsv_select_search_str *next;
@@ -63,7 +64,7 @@ struct zsv_select_data {
     struct {         // merge data: only used with --merge
       struct zsv_select_uint_list *indexes, **last_index;
     } merge;
-  } * out2in; // array of .output_cols_count length; out2in[x] = y where x = output ix, y = input info
+  } *out2in; // array of .output_cols_count length; out2in[x] = y where x = output ix, y = input info
 
   unsigned int output_cols_count; // total count of output columns
 
@@ -343,23 +344,23 @@ static enum zsv_select_column_index_selection_type zsv_select_column_index_selec
                                                                                      unsigned *lo, unsigned *hi) {
   enum zsv_select_column_index_selection_type result = zsv_select_column_index_selection_type_none;
 
-  unsigned int i, j, k;
-  int n = 0;
-  k = sscanf((const char *)arg, "%u-%u%n", &i, &j, &n);
+  unsigned int i = 0;
+  unsigned int j = 0;
+  size_t n = 0;
+  int k = sscanf((const char *)arg, "%u-%u%n", &i, &j, &n);
   if (k == 2) {
-    if (n == (int)strlen((const char *)arg) && i > 0 && j >= i)
+    if (n == strlen((const char *)arg) && i > 0 && j >= i)
       result = zsv_select_column_index_selection_type_range;
   } else {
     k = sscanf((const char *)arg, "%u%n", &i, &n);
-    if (k && n == (int)strlen((const char *)arg)) {
+    if (k == 1 && n == strlen((const char *)arg)) {
       if (i > 0)
         result = zsv_select_column_index_selection_type_single;
     } else {
       k = sscanf((const char *)arg, "%u-%n", &i, &n);
-      if (k && n == (int)strlen((const char *)arg)) {
+      if (k == 1 && n == strlen((const char *)arg)) {
         if (i > 0) {
           result = zsv_select_column_index_selection_type_lower_bounded;
-          j = 0;
         }
       }
     }
@@ -616,6 +617,7 @@ int ZSV_MAIN_FUNC(ZSV_COMMAND)(int argc, const char *argv[], struct zsv_opts *op
     return zsv_status_ok;
   }
 
+  int err = 0;
   struct zsv_select_data data = {0};
   data.opts = opts;
   const char *input_path = NULL;
@@ -708,14 +710,11 @@ int ZSV_MAIN_FUNC(ZSV_COMMAND)(int argc, const char *argv[], struct zsv_opts *op
           -1, "--sample-pct value should be a number between 0 and 100 (e.g. 1.5 for a sample of 1.5%% of the data");
       else
         data.sample_pct = d;
-    } else if (!strcmp(argv[arg_i], "--prepend-header")) {
-      if (!(arg_i + 1 < argc))
-        stat = zsv_printerr(1, "%s option requires a value");
-      else
-        data.prepend_header = argv[++arg_i];
-    } else if (!strcmp(argv[arg_i], "--no-header")) {
+    } else if (!strcmp(argv[arg_i], "--prepend-header"))
+      data.prepend_header = zsv_next_arg(++arg_i, argc, argv, &err);
+    else if (!strcmp(argv[arg_i], "--no-header"))
       data.no_header = 1;
-    } else if (!strcmp(argv[arg_i], "-H") || !strcmp(argv[arg_i], "--head")) {
+    else if (!strcmp(argv[arg_i], "-H") || !strcmp(argv[arg_i], "--head")) {
       if (!(arg_i + 1 < argc && atoi(argv[arg_i + 1]) >= 0))
         stat = zsv_printerr(1, "%s option value invalid: should be positive integer; got %s", argv[arg_i],
                             arg_i + 1 < argc ? argv[arg_i + 1] : "");

app/select.c

@@ -24,6 +24,7 @@
 #include <zsv/utils/utf8.h>
 #include <zsv/utils/string.h>
 #include <zsv/utils/mem.h>
+#include <zsv/utils/arg.h>
 
 struct zsv_select_search_str {
   struct zsv_select_search_str *next;
@@ -69,7 +70,7 @@
     struct {         // merge data: only used with --merge
       struct zsv_select_uint_list *indexes, **last_index;
     } merge;
-  } * out2in; // array of .output_cols_count length; out2in[x] = y where x = output ix, y = input info
+  } *out2in; // array of .output_cols_count length; out2in[x] = y where x = output ix, y = input info
 
   unsigned int output_cols_count; // total count of output columns
 
@@ -353,20 +354,21 @@
                                                                                      unsigned *lo, unsigned *hi) {
   enum zsv_select_column_index_selection_type result = zsv_select_column_index_selection_type_none;
 
-  unsigned int i, j, k;
-  int n = 0;
-  k = sscanf((const char *)arg, "%u-%u%n", &i, &j, &n);
+  unsigned int i = 0;
+  unsigned int j = 0;
+  size_t n = 0;
+  int k = sscanf((const char *)arg, "%u-%u%n", &i, &j, &n);
   if (k == 2) {
-    if (n == (int)strlen((const char *)arg) && i > 0 && j >= i)
+    if (n == strlen((const char *)arg) && i > 0 && j >= i)
       result = zsv_select_column_index_selection_type_range;
   } else {
     k = sscanf((const char *)arg, "%u%n", &i, &n);
-    if (k && n == (int)strlen((const char *)arg)) {
+    if (k == 1 && n == strlen((const char *)arg)) {
       if (i > 0)
         result = zsv_select_column_index_selection_type_single;
     } else {
       k = sscanf((const char *)arg, "%u-%n", &i, &n);
-      if (k && n == (int)strlen((const char *)arg)) {
+      if (k == 1 && n == strlen((const char *)arg)) {
         if (i > 0) {
           result = zsv_select_column_index_selection_type_lower_bounded;
           j = 0;
@@ -755,6 +757,7 @@
     return zsv_status_ok;
   }
 
+  int err = 0;
   char fixed_auto = 0;
   struct zsv_select_data data = {0};
   data.opts = opts;
@@ -783,12 +786,15 @@
           if (*s == ',')
             data.fixed.count++;
         free(data.fixed.offsets);
-        data.fixed.offsets = malloc(data.fixed.count * sizeof(*data.fixed.offsets));
+        if (!(data.fixed.offsets = malloc(data.fixed.count * sizeof(*data.fixed.offsets)))) {
+          stat = zsv_printerr(1, "Out of memory!\n");
+          break;
+        }
         size_t count = 0;
         const char *start = argv[arg_i];
         for (const char *end = argv[arg_i];; end++) {
           if (*end == ',' || *end == '\0') {
-            if (!sscanf(start, "%zu,", &data.fixed.offsets[count++])) {
+            if (sscanf(start, "%zu,", &data.fixed.offsets[count++]) != 1) {
               stat = zsv_printerr(1, "Invalid offset: %.*s\n", end - start, start);
               break;
             } else if (*end == '\0')
@@ -850,17 +856,14 @@
         stat = zsv_printerr(1, "--sample-pct option requires a value");
       else if (!(d = atof(argv[arg_i])) && d > 0 && d < 100)
         stat = zsv_printerr(
-          -1, "--sample-pct value should be a number between 0 and 100 (e.g. 1.5 for a sample of 1.5% of the data");
+          -1, "--sample-pct value should be a number between 0 and 100 (e.g. 1.5 for a sample of 1.5%% of the data");
       else
         data.sample_pct = d;
-    } else if (!strcmp(argv[arg_i], "--prepend-header")) {
-      if (!(arg_i + 1 < argc))
-        stat = zsv_printerr(1, "%s option requires a value");
-      else
-        data.prepend_header = argv[++arg_i];
-    } else if (!strcmp(argv[arg_i], "--no-header")) {
+    } else if (!strcmp(argv[arg_i], "--prepend-header"))
+      data.prepend_header = zsv_next_arg(++arg_i, argc, argv, &err);
+    else if (!strcmp(argv[arg_i], "--no-header"))
       data.no_header = 1;
-    } else if (!strcmp(argv[arg_i], "-H") || !strcmp(argv[arg_i], "--head")) {
+    else if (!strcmp(argv[arg_i], "-H") || !strcmp(argv[arg_i], "--head")) {
       if (!(arg_i + 1 < argc && atoi(argv[arg_i + 1]) >= 0))
         stat = zsv_printerr(1, "%s option value invalid: should be positive integer; got %s", argv[arg_i],
                             arg_i + 1 < argc ? argv[arg_i + 1] : "");
@@ -898,33 +901,37 @@
       input_path = argv[arg_i];
   }
 
-  if (data.sample_pct)
-    srand(time(0));
+  if (stat == zsv_status_ok) {
+    if (data.sample_pct)
+      srand(time(0));
 
-  if (data.use_header_indexes && stat == zsv_status_ok)
-    stat = zsv_select_check_exclusions_are_indexes(&data);
+    if (data.use_header_indexes && stat == zsv_status_ok)
+      stat = zsv_select_check_exclusions_are_indexes(&data);
+  }
 
-  if (!data.opts->stream) {
+  if (stat == zsv_status_ok) {
+    if (!data.opts->stream) {
 #ifdef NO_STDIN
-    stat = zsv_printerr(1, "Please specify an input file");
+      stat = zsv_printerr(1, "Please specify an input file");
 #else
-    data.opts->stream = stdin;
+      data.opts->stream = stdin;
 #endif
-  }
+    }
 
-  if (stat == zsv_status_ok && fixed_auto) {
-    if (data.fixed.offsets)
-      stat = zsv_printerr(zsv_status_error, "Please specify either --fixed-auto or --fixed, but not both");
-    else if (data.opts->insert_header_row)
-      stat = zsv_printerr(zsv_status_error, "--fixed-auto can not be specified together with --header-row");
-    else {
-      size_t buffsize = 1024 * 256; // read the first
-      preview_buff = calloc(buffsize, sizeof(*preview_buff));
-      if (!preview_buff)
-        stat = zsv_printerr(zsv_status_memory, "Out of memory!");
-      else
-        stat = auto_detect_fixed_column_sizes(&data.fixed, data.opts, preview_buff, buffsize, &preview_buff_len,
-                                              opts->verbose);
+    if (stat == zsv_status_ok && fixed_auto) {
+      if (data.fixed.offsets)
+        stat = zsv_printerr(zsv_status_error, "Please specify either --fixed-auto or --fixed, but not both");
+      else if (data.opts->insert_header_row)
+        stat = zsv_printerr(zsv_status_error, "--fixed-auto can not be specified together with --header-row");
+      else {
+        size_t buffsize = 1024 * 256; // read the first
+        preview_buff = calloc(buffsize, sizeof(*preview_buff));
+        if (!preview_buff)
+          stat = zsv_printerr(zsv_status_memory, "Out of memory!");
+        else
+          stat = auto_detect_fixed_column_sizes(&data.fixed, data.opts, preview_buff, buffsize, &preview_buff_len,
+                                                opts->verbose);
+      }
     }
   }
 

app/sql.c

@@ -392,7 +392,7 @@ int ZSV_MAIN_FUNC(ZSV_COMMAND)(int argc, const char *argv[], struct zsv_opts *op
           for (char *ix_str = data.join_indexes; !err && ix_str && *ix_str && *(++ix_str);
                ix_str = strchr(ix_str + 1, ',')) {
             unsigned int next_ix;
-            if (sscanf(ix_str, "%u,", &next_ix)) {
+            if (sscanf(ix_str, "%u,", &next_ix) == 1) {
               if (next_ix == 0)
                 fprintf(stderr, "--join-indexes index must be greater than zero\n");
               else if (next_ix > (unsigned)col_count)

scripts/ci-run-clang-format.sh

@@ -6,7 +6,7 @@ echo "[INF] Running $0"
 
 VERSION=$(clang-format --version | cut -d ' ' -f4 | tr -d '\n')
 MAJOR_VERSION=$(echo "$VERSION" | cut -d '.' -f1)
-REQUIRED_VERSION="14"
+REQUIRED_VERSION="15"
 
 if [ "$VERSION" = "" ]; then
   echo "[ERR] clang-format is not installed!"
@@ -16,7 +16,7 @@ else
   echo "[INF] clang-format version [$VERSION]"
   if [ "$MAJOR_VERSION" -lt "$REQUIRED_VERSION" ]; then
     echo "[ERR] Installed clang-format version is $VERSION."
-    echo "[ERR] clang-format $REQUIRED_VERSION or later is required!"
+    echo "[ERR] clang-format-$REQUIRED_VERSION or later is required!"
     exit 1
   fi
 fi

src/zsv_internal.c

@@ -183,7 +183,7 @@ struct zsv_scanner {
     union {
       struct zsv_scan_delim_regs delim;
       struct zsv_scan_fixed_regs fixed;
-    } * regs;
+    } *regs;
     enum zsv_status stat; // last status
     unsigned char *buff;
     size_t bytes_read;