]> git.puffer.fish Git - matthieu/frr.git/commitdiff
isisd: fix OpenBSD pedantic format warnings
authorDavid Lamparter <equinox@diac24.net>
Tue, 14 Jul 2020 10:02:54 +0000 (12:02 +0200)
committerDavid Lamparter <equinox@diac24.net>
Tue, 14 Jul 2020 10:02:54 +0000 (12:02 +0200)
constants are int-typed, so adding something to an uint8_t yields an
int.  Nevermind the fact that varargs calling conventions require
upcasting everything smaller than an int to an int anyways...

Signed-off-by: David Lamparter <equinox@diac24.net>
isisd/isis_pdu.c
isisd/isis_tlvs.c

index 9a1ad38f8af7b59311293dff8f83fe4af1abc936..72548e0425239dbd4110341e215db4392b5cb3b8 100644 (file)
@@ -876,7 +876,7 @@ static int process_lsp(uint8_t pdu_type, struct isis_circuit *circuit,
        /* lsp is_type check */
        if ((hdr.lsp_bits & IS_LEVEL_1) != IS_LEVEL_1) {
                zlog_debug(
-                       "ISIS-Upd (%s): LSP %s invalid LSP is type 0x%hhx",
+                       "ISIS-Upd (%s): LSP %s invalid LSP is type 0x%x",
                        circuit->area->area_tag, rawlspid_print(hdr.lsp_id),
                        hdr.lsp_bits & IS_LEVEL_1_AND_2);
                /* continue as per RFC1122 Be liberal in what you accept, and
index e9b28489c3b56a8c1b02728a79767eb5712b6bd0..a1f9cc236ff75d4df146c99dca7d822410a7d333 100644 (file)
@@ -538,7 +538,7 @@ static int unpack_item_ext_subtlvs(uint16_t mtid, uint8_t len, struct stream *s,
                subtlv_type = stream_getc(s);
                subtlv_len = stream_getc(s);
                if (subtlv_len > len - sum) {
-                       sbuf_push(log, indent, "TLV %hhu: Available data %hhu is less than TLV size %u !\n",
+                       sbuf_push(log, indent, "TLV %hhu: Available data %u is less than TLV size %u !\n",
                                  subtlv_type, len - sum, subtlv_len);
                        return 1;
                }
@@ -1152,7 +1152,7 @@ static int unpack_item_area_address(uint16_t mtid, uint8_t len,
        rv->len = stream_getc(s);
 
        if (len < 1 + rv->len) {
-               sbuf_push(log, indent, "Not enough data left. (Expected %hhu bytes of address, got %hhu)\n",
+               sbuf_push(log, indent, "Not enough data left. (Expected %hhu bytes of address, got %u)\n",
                          rv->len, len - 1);
                goto out;
        }
@@ -1468,7 +1468,7 @@ static int unpack_item_extended_reach(uint16_t mtid, uint8_t len,
 
        if ((size_t)len < ((size_t)11) + subtlv_len) {
                sbuf_push(log, indent,
-                         "Not enough data left for subtlv size %hhu, there are only %hhu bytes left.\n",
+                         "Not enough data left for subtlv size %hhu, there are only %u bytes left.\n",
                          subtlv_len, len - 11);
                goto out;
        }