]> 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, 27 Apr 2022 10:21:45 +0000 (12:21 +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.

src/util-ja3.c

index a2fee4efb3c7bde7116dc4892672272d4f062144..f3b455d7c44b8b7396695456cdb29be40200257e 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;