]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
src: fix coverity warning about sizeof
authorPhilippe Antoine <pantoine@oisf.net>
Fri, 27 Jan 2023 12:28:57 +0000 (13:28 +0100)
committerVictor Julien <vjulien@oisf.net>
Sat, 28 Jan 2023 07:12:04 +0000 (08:12 +0100)
CID: 1520601
CID: 1520602

> In this particular case sizeof (char **) happens to be equal to
sizeof (char *), but this is not a portable assumption.

src/suricata.c

index ad44e129aac0113442c09707c709e84dadbdbbb8..6a655a1e85b7b5f97713045ca52e6eae61162076 100644 (file)
@@ -1772,7 +1772,7 @@ static TmEcode ParseCommandLine(int argc, char** argv, SCInstance *suri)
                 }
             } else if (strcmp((long_opts[option_index]).name, "include") == 0) {
                 if (suri->additional_configs == NULL) {
-                    suri->additional_configs = SCCalloc(2, sizeof(char **));
+                    suri->additional_configs = SCCalloc(2, sizeof(char *));
                     if (suri->additional_configs == NULL) {
                         FatalError(
                                 "Failed to allocate memory for additional configuration files: %s",
@@ -1783,7 +1783,7 @@ static TmEcode ParseCommandLine(int argc, char** argv, SCInstance *suri)
                     for (int i = 0;; i++) {
                         if (suri->additional_configs[i] == NULL) {
                             const char **additional_configs =
-                                    SCRealloc(suri->additional_configs, (i + 2) * sizeof(char **));
+                                    SCRealloc(suri->additional_configs, (i + 2) * sizeof(char *));
                             if (additional_configs == NULL) {
                                 FatalError("Failed to allocate memory for additional configuration "
                                            "files: %s",