Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fuse: do not serialize headers until after adjustment #539

Closed
wants to merge 1 commit into from

Conversation

creachadair
Copy link
Contributor

Older kernel versions do not support all the fields of the current INIT header. Adjust the header size before marshaling the header payload.

Updates #538

@hanwen
Copy link
Owner

hanwen commented Oct 27, 2024

thanks! I actually use gerrit for code review (see CONTRIBUTING) but if that is too onerous, I can could do the gerrit upload for you. Thanks for the sleuthing.

@creachadair
Copy link
Contributor Author

thanks! I actually use gerrit for code review (see CONTRIBUTING) but if that is too onerous, I can could do the gerrit upload for you. Thanks for the sleuthing.

Oh, I'm sorry, I was just neglectful about reading the doc. I will reissue this in the approved manner.

Older kernel versions do not support all the fields of the current INIT header.
Adjust the header size before marshaling the header payload.

Updates #538

Change-Id: I06933c281af5c46bde6bfecaf5269274d20b8ad6
@creachadair
Copy link
Contributor Author

Closing this in favour of https://review.gerrithub.io/c/hanwen/go-fuse/+/1203164, and apologies for not reading first.

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.

2 participants