diff options
| author | David Lamparter <equinox@opensourcerouting.org> | 2024-04-30 14:18:03 +0200 |
|---|---|---|
| committer | David Lamparter <equinox@opensourcerouting.org> | 2024-05-02 22:26:53 +0200 |
| commit | 552a0a3a284194dc4f9dac3d72259e9d622813e1 (patch) | |
| tree | 54fb1c29e0989083012932f0c32d984a5e3817ea /lib/northbound_oper.c | |
| parent | Merge pull request #15837 from acooks-at-bda/dev/fix-deprecated-cares-api (diff) | |
| download | frr-552a0a3a284194dc4f9dac3d72259e9d622813e1.tar.xz frr-552a0a3a284194dc4f9dac3d72259e9d622813e1.zip | |
lib: fix `time_t` print without cast
The gcc plugin wasn't warning about printing `suseconds_t` (which is
`time_t`, but in `struct timeval`.) It needs to be printed with a cast,
just like `time_t`. Luckily there is only one such usage.
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
Diffstat (limited to 'lib/northbound_oper.c')
| -rw-r--r-- | lib/northbound_oper.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/northbound_oper.c b/lib/northbound_oper.c index 7e7190f5a..5f38c970c 100644 --- a/lib/northbound_oper.c +++ b/lib/northbound_oper.c @@ -1556,8 +1556,9 @@ static enum nb_error nb_op_yield(struct nb_op_yield_state *ys) unsigned long min_us = MAX(1, NB_OP_WALK_INTERVAL_US / 50000); struct timeval tv = { .tv_sec = 0, .tv_usec = min_us }; - DEBUGD(&nb_dbg_events, "NB oper-state: yielding %s for %lus (should_batch %d)", - ys->xpath, tv.tv_usec, ys->should_batch); + DEBUGD(&nb_dbg_events, + "NB oper-state: yielding %s for %lldus (should_batch %d)", + ys->xpath, (long long)tv.tv_usec, ys->should_batch); if (ys->should_batch) { /* |