summaryrefslogtreecommitdiff
path: root/lib/network.h
diff options
context:
space:
mode:
authorDavid Lamparter <equinox@diac24.net>2020-05-01 10:59:53 +0200
committerDavid Lamparter <equinox@diac24.net>2021-02-01 17:08:13 +0100
commit3c191fb1387ece2265b8ac7c1e102904148ebed9 (patch)
tree7f1ca88a86022782b7cb145dd39a2a0cfa969ca3 /lib/network.h
parent6968b038ebb0beeda99b7ed70f3f1c35383f1ee3 (diff)
lib: move frr_weak_random to header file
Makes more sense to have this as a static inline. Also I don't want to be forced to link network.o into clippy ;) Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
Diffstat (limited to 'lib/network.h')
-rw-r--r--lib/network.h18
1 files changed, 17 insertions, 1 deletions
diff --git a/lib/network.h b/lib/network.h
index 83c9e59e76..4a9666984f 100644
--- a/lib/network.h
+++ b/lib/network.h
@@ -45,7 +45,23 @@ extern int set_cloexec(int fd);
extern float htonf(float);
extern float ntohf(float);
-extern long frr_weak_random(void);
+/**
+ * Helper function that returns a random long value. The main purpose of
+ * this function is to hide a `random()` call that gets flagged by coverity
+ * scan and put it into one place.
+ *
+ * The main usage of this function should be for generating jitter or weak
+ * random values for simple purposes.
+ *
+ * See 'man 3 random' for more information.
+ *
+ * \returns random long integer.
+ */
+static inline long frr_weak_random(void)
+{
+ /* coverity[dont_call] */
+ return random();
+}
#ifdef __cplusplus
}