Skip to content

Fix inconsistent type of readdir(plus) offsets#123

Open
progval wants to merge 1 commit intoSherlock-Holo:masterfrom
progval:offset-u64
Open

Fix inconsistent type of readdir(plus) offsets#123
progval wants to merge 1 commit intoSherlock-Holo:masterfrom
progval:offset-u64

Conversation

@progval
Copy link
Contributor

@progval progval commented Aug 17, 2025

u64 was used:

  • in raw::abi::fuse_read_in (the source of values for readdir and readdirplus arguments)
  • in raw::abi::fuse_dirent (what DirectoryEntry and DirectoryEntryPlus are converted to)
  • as readdirplus argument

i64 was used:

  • as readdir argument
  • in raw::reply::DirectoryEntry and raw::reply::DirectoryEntryPlus,

This fixes this inconsistency by making them all use u64.

`u64` was used:

* in `raw::abi::fuse_read_in` (the source of values for `readdir` and
  `readdirplus` arguments)
* in `raw::abi::fuse_dirent` (what `DirectoryEntry` and
  `DirectoryEntryPlus` are converted to)
* as `readdirplus` argument

`i64` was used:

* as `readdir` argument
* in `raw::reply::DirectoryEntry` and `raw::reply::DirectoryEntryPlus`,

This fixes this inconsistency by making them all use `u64`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant