appFs - Initialize fuse_buf struct to avoid use of uninitialized values

Pavel Siska · Pavel Siska · commit e0dabcf58e6f · 2025-11-10T16:07:53.000+01:00
Previously, m_buffer members were manually set in the constructor,
but not all fields were guaranteed to be initialized, leading to
valgrind warnings about uninitialized reads inside libfuse.
diff --git a/src/appFs/appFs.cpp b/src/appFs/appFs.cpp
@@ -365,8 +365,6 @@ class AppFsFuseBuffer {
 public:
 	AppFsFuseBuffer()
 	{
-		m_buffer.mem = nullptr;
-		m_buffer.size = 0;
 	}
 
 	~AppFsFuseBuffer()
@@ -378,7 +376,7 @@ class AppFsFuseBuffer {
 	fuse_buf* getBuffer() { return &m_buffer; }
 
 private:
-	fuse_buf m_buffer;
+	fuse_buf m_buffer {};
 };
 
 static void fuseLoop(struct fuse_session* session)

Original file line number	Diff line number	Diff line change
`@@ -365,8 +365,6 @@ class AppFsFuseBuffer {`
`365`	`365`	`public:`
`366`	`366`	`AppFsFuseBuffer()`
`367`	`367`	`{`
`368`		`- m_buffer.mem = nullptr;`
`369`		`- m_buffer.size = 0;`
`370`	`368`	`}`
`371`	`369`
`372`	`370`	`~AppFsFuseBuffer()`
`@@ -378,7 +376,7 @@ class AppFsFuseBuffer {`
`378`	`376`	`fuse_buf* getBuffer() { return &m_buffer; }`
`379`	`377`
`380`	`378`	`private:`
`381`		`- fuse_buf m_buffer;`
	`379`	`+ fuse_buf m_buffer {};`
`382`	`380`	`};`
`383`	`381`
`384`	`382`	`static void fuseLoop(struct fuse_session* session)`