From 4dd55864f09a38acad9dc9b922bbfdc36b90c712 Mon Sep 17 00:00:00 2001
From: Florian Schmaus <flow@cs.fau.de>
Date: Thu, 20 May 2021 15:36:53 +0200
Subject: [PATCH] [Future] Fix clang-tidy issue in MadviseFuture
---
emper/io/Future.hpp | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/emper/io/Future.hpp b/emper/io/Future.hpp
index 04999b92..efffd8e4 100644
--- a/emper/io/Future.hpp
+++ b/emper/io/Future.hpp
@@ -654,7 +654,11 @@ class CancelWrapper : public Future {
* @brief Request a madvise operation
*/
class MadviseFuture : public Future {
- void prepareSqe(io_uring_sqe* sqe) override { io_uring_prep_madvise(sqe, buf, len, flags); }
+ void prepareSqe(io_uring_sqe* sqe) override {
+ // TODO: Check that len is within the bounds of off_t.
+ auto len_off_t = static_cast<off_t>(len);
+ io_uring_prep_madvise(sqe, buf, len_off_t, flags);
+ }
public:
MadviseFuture(void* addr, size_t len, int advise)
--
GitLab