ensure that the retry struct is always initialized

before calling the sent() function

diff --git a/src/lib/bio/retry.c b/src/lib/bio/retry.c
index 47621f5efb168307aecb218b901f72359efe42c9..d2f1cbf0d7e278803abf0c25db5cd9aa310df47b 100644 (file)
--- a/src/lib/bio/retry.c
+++ b/src/lib/bio/retry.c
@@ -762,16 +762,20 @@ static ssize_t fr_bio_retry_write(fr_bio_t *bio, void *packet_ctx, void const *b
                .size = size,
        };
 
+       /*
+        *      Always initialize the retry timer.  That way the sent() callback doesn't have to call
+        *      fr_time().
+        *
+        *      The application can call fr_bio_retry_entry_init() to re-initialize it, but that's fine.
+        */
+       fr_retry_init(&item->retry, item->retry.start, &my->retry_config);
+
        /*
         *      Tell the application that we've saved the packet.  The "item" pointer allows the application
         *      to cancel this packet if necessary.
         */
        my->sent(bio, packet_ctx, buffer, size, item);
 
-       if (!item->retry.config) {
-               fr_retry_init(&item->retry, item->retry.start, &my->retry_config);
-       }
-
        /*
         *      This should never fail.
         */