]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
ja3: fix minor format string warning
authorVictor Julien <vjulien@oisf.net>
Wed, 27 Apr 2022 09:38:37 +0000 (11:38 +0200)
committerVictor Julien <vjulien@oisf.net>
Wed, 4 May 2022 16:57:09 +0000 (18:57 +0200)
cppcheck:

src/util-ja3.c:197:28: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint]
        (*buffer)->used += snprintf((*buffer)->data, (*buffer)->size, "%d",
                           ^
src/util-ja3.c:201:28: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint]
        (*buffer)->used += snprintf((*buffer)->data + (*buffer)->used,
                           ^

Bug: #5291.
(cherry picked from commit 1e13f7278585555623ff0be2d3a98476617b1219)

src/util-ja3.c

index c937920009717ccbca0cee129731dc710fab4575..564e919b5d04e9fa7c4c62732277f2a189178b36 100644 (file)
@@ -194,13 +194,11 @@ int Ja3BufferAddValue(JA3Buffer **buffer, uint32_t value)
     }
 
     if ((*buffer)->used == 0) {
-        (*buffer)->used += snprintf((*buffer)->data, (*buffer)->size, "%d",
-                                    value);
+        (*buffer)->used += snprintf((*buffer)->data, (*buffer)->size, "%u", value);
     }
     else {
-        (*buffer)->used += snprintf((*buffer)->data + (*buffer)->used,
-                                    (*buffer)->size - (*buffer)->used, "-%d",
-                                    value);
+        (*buffer)->used += snprintf(
+                (*buffer)->data + (*buffer)->used, (*buffer)->size - (*buffer)->used, "-%u", value);
     }
 
     return 0;