Paths

Table of Contentst

Differential D39350

fuse: Remove set but unused cr_gid variable.
ClosedPublic
Actions

Authored by jhb on Mar 30 2023, 11:33 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Apr 16, 7:06 AM

	Unknown Object (File)
	Thu, Apr 16, 7:01 AM

	Unknown Object (File)
	Sat, Mar 28, 12:35 PM

	Unknown Object (File)
	Fri, Mar 27, 11:27 PM

	Unknown Object (File)
	Feb 9 2026, 8:08 AM

	Unknown Object (File)
	Feb 2 2026, 11:52 PM

	Unknown Object (File)
	Dec 23 2025, 7:50 AM

	Unknown Object (File)
	Dec 12 2025, 2:57 PM

Subscribers

Details

Reviewers

Commits

rGcb750f7f5afb: fuse: Remove set but unused cr_gid variable.

Summary

Reported by: GCC

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 50687
Build 47578: arc lint + arc unit

Event Timeline

jhb created this revision.Mar 30 2023, 11:33 PM

Herald added subscribers: asomers, imp. · View Herald TranscriptMar 30 2023, 11:33 PM

jhb requested review of this revision.Mar 30 2023, 11:33 PM

jhb added a parent revision: D39349: linuxkpi: Appease -Wunused-but-set-variable warnings from GCC..

Harbormaster completed remote builds in B50687: Diff 119686.Mar 30 2023, 11:33 PM

jhb added a child revision: D39351: ipmi: Remove some dead code for unsupported BMCs..Mar 30 2023, 11:33 PM

asomers accepted this revision.Mar 31 2023, 12:42 AM

This revision is now accepted and ready to land.Mar 31 2023, 12:42 AM

jhb added inline comments.Mar 31 2023, 5:04 PM

sys/fs/fuse/fuse_vnops.c
2221	Btw, cr_gid is #define'd to `cr_groups[0]` in <sys/ucred.h> so I'm not really sure how this compiling without other warnings as it was expanding to things like `gid_t cr_groups[0];` and then in the assignments below `cr_groups[0] = cred->cr_groups[0];`. In theory -Warray-bounds should have caught that but didn't.

Closed by commit rGcb750f7f5afb: fuse: Remove set but unused cr_gid variable. (authored by jhb). · Explain WhyMar 31 2023, 5:59 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rGcb750f7f5afb: fuse: Remove set but unused cr_gid variable..

Revision Contents
Changeset List

Path

Size

sys/

fs/

fuse/

5 lines

Diff 119686

sys/fs/fuse/fuse_vnops.c

Loading...