runmodes: minor format string fixes

cppcheck:

src/util-runmodes.c:210:9: warning: %u in format string (no. 2) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
        snprintf(tname, sizeof(tname), "%s#%02u", thread_name_workers, thread+1);
        ^
src/util-runmodes.c:211:9: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
        snprintf(qname, sizeof(qname), "pickup%u", thread+1);
        ^
src/util-runmodes.c:455:9: warning: %u in format string (no. 2) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
        snprintf(tname, sizeof(tname), "%s#%02u", thread_name_workers, thread+1);
        ^
src/util-runmodes.c:457:9: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
        snprintf(qname, sizeof(qname), "pickup%u", thread+1);
        ^

src/runmode-erf-file.c:188:9: warning: %u in format string (no. 2) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
        snprintf(tname, sizeof(tname), "%s#%02u", thread_name_workers, thread+1);
        ^
src/runmode-erf-file.c:189:9: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
        snprintf(qname, sizeof(qname), "pickup%u", thread+1);
        ^
src/runmode-pcap-file.c:201:9: warning: %u in format string (no. 2) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
        snprintf(tname, sizeof(tname), "%s#%02u", thread_name_workers, thread+1);
        ^
src/runmode-pcap-file.c:202:9: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
        snprintf(qname, sizeof(qname), "pickup%u", thread+1);
        ^

Bug: #5291.
pull/7321/head
Victor Julien 4 years ago
parent a8d3cd6eb4
commit 2965d809a4

@ -185,8 +185,8 @@ int RunModeErfFileAutoFp(void)
} }
for (thread = 0; thread < (uint16_t)thread_max; thread++) { for (thread = 0; thread < (uint16_t)thread_max; thread++) {
snprintf(tname, sizeof(tname), "%s#%02u", thread_name_workers, thread+1); snprintf(tname, sizeof(tname), "%s#%02d", thread_name_workers, thread + 1);
snprintf(qname, sizeof(qname), "pickup%u", thread+1); snprintf(qname, sizeof(qname), "pickup%d", thread + 1);
SCLogDebug("tname %s, qname %s", tname, qname); SCLogDebug("tname %s, qname %s", tname, qname);

@ -198,8 +198,8 @@ int RunModeFilePcapAutoFp(void)
} }
for (thread = 0; thread < (uint16_t)thread_max; thread++) { for (thread = 0; thread < (uint16_t)thread_max; thread++) {
snprintf(tname, sizeof(tname), "%s#%02u", thread_name_workers, thread+1); snprintf(tname, sizeof(tname), "%s#%02d", thread_name_workers, thread + 1);
snprintf(qname, sizeof(qname), "pickup%u", thread+1); snprintf(qname, sizeof(qname), "pickup%d", thread + 1);
SCLogDebug("tname %s, qname %s", tname, qname); SCLogDebug("tname %s, qname %s", tname, qname);
SCLogDebug("Assigning %s affinity to cpu %u", tname, cpu); SCLogDebug("Assigning %s affinity to cpu %u", tname, cpu);

@ -207,8 +207,8 @@ int RunModeSetLiveCaptureAutoFp(ConfigIfaceParserFunc ConfigParser,
} }
for (int thread = 0; thread < thread_max; thread++) { for (int thread = 0; thread < thread_max; thread++) {
snprintf(tname, sizeof(tname), "%s#%02u", thread_name_workers, thread+1); snprintf(tname, sizeof(tname), "%s#%02d", thread_name_workers, thread + 1);
snprintf(qname, sizeof(qname), "pickup%u", thread+1); snprintf(qname, sizeof(qname), "pickup%d", thread + 1);
SCLogDebug("tname %s, qname %s", tname, qname); SCLogDebug("tname %s, qname %s", tname, qname);
@ -452,9 +452,9 @@ int RunModeSetIPSAutoFp(ConfigIPSParserFunc ConfigParser,
} }
for (int thread = 0; thread < thread_max; thread++) { for (int thread = 0; thread < thread_max; thread++) {
snprintf(tname, sizeof(tname), "%s#%02u", thread_name_workers, thread+1); snprintf(tname, sizeof(tname), "%s#%02d", thread_name_workers, thread + 1);
char qname[TM_QUEUE_NAME_MAX]; char qname[TM_QUEUE_NAME_MAX];
snprintf(qname, sizeof(qname), "pickup%u", thread+1); snprintf(qname, sizeof(qname), "pickup%d", thread + 1);
SCLogDebug("tname %s, qname %s", tname, qname); SCLogDebug("tname %s, qname %s", tname, qname);

Loading…
Cancel
Save