From: Laszlo Boszormenyi (GCS) <gcs@debian.org>
Date: Sun, 16 Feb 2025 08:07:36 +0000 (+0100)
Subject: Fix build of example/memfs_ll.cc on 32 bit architectures
X-Git-Tag: fuse-3.17.1-rc1~17
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=6adee44e78b70f1708b5d129281e958a2dfca33a;p=qemu-gpiodev%2Flibfuse.git

Fix build of example/memfs_ll.cc on 32 bit architectures

The code uses std::min() which expects its arguments to be size_t. Two
times it uses an offset declared as off_t. While both size_t and off_t
are 32-bit integers, the latter is signed. On 64 bit architectures
the conversation of off_t -> size_t performed automatically. On 32 bit
architectures it needs a type cast.

Signed-off-by: Laszlo Boszormenyi (GCS) <gcs@debian.org>
---

diff --git a/example/memfs_ll.cc b/example/memfs_ll.cc
index c7b9663..9898f25 100644
--- a/example/memfs_ll.cc
+++ b/example/memfs_ll.cc
@@ -150,7 +150,7 @@ class Inode {
 
 	void read_content(char *buf, size_t size, off_t offset) const
 	{
-		size_t bytes_to_read = std::min(size, content.size() - offset);
+		size_t bytes_to_read = std::min(size, content.size() - (size_t)offset);
 		std::copy(content.begin() + offset,
 			  content.begin() + offset + bytes_to_read, buf);
 	}
@@ -613,7 +613,7 @@ static void memfs_read(fuse_req_t req, fuse_ino_t ino, size_t size,
 	}
 
 	std::vector<char> content(
-		std::min(size, inode->content_size() - offset));
+		std::min(size, inode->content_size() - (size_t)offset));
 	inode->read_content(content.data(), content.size(), offset);
 
 	inode->unlock();