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)
	Sun, Mar 9, 10:30 AM

	Unknown Object (File)
	Feb 16 2025, 9:31 PM

	Unknown Object (File)
	Feb 2 2025, 6:51 PM

	Unknown Object (File)
	Jan 24 2025, 10:16 AM

	Unknown Object (File)
	Jan 21 2025, 12:03 PM

	Unknown Object (File)
	Jan 9 2025, 12:24 PM

	Unknown Object (File)
	Jan 8 2025, 9:57 PM

	Unknown Object (File)
	Jan 8 2025, 2:06 PM

View All 60 Files

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...