Page MenuHomeFreeBSD
Differential D56343

tunefs: Fix pointer arithmetic
ClosedPublic
Actions

Authored by des on Apr 10 2026, 3:15 PM.
Tags
None
Referenced Files
F158650356: D56343.diff
Thu, Jun 4, 8:30 AM
F158650346: D56343.diff
Thu, Jun 4, 8:30 AM
Unknown Object (File)
Sun, May 17, 7:16 PM
Unknown Object (File)
Sun, May 17, 7:08 PM
Unknown Object (File)
Sun, May 17, 4:19 PM
Unknown Object (File)
Sun, May 17, 4:12 PM
Unknown Object (File)
Sat, May 16, 4:01 PM
Unknown Object (File)
Sat, May 16, 4:00 PM
View All 43 Files
Subscribers
imp
olce

Details

Reviewers
ivy
kevans
mckusick
Commits
rG9d95d806691e: tunefs: Fix pointer arithmetic
Summary

While here, remove a bogus const which has been there for years.

MFC after: 1 week
Reported by: ivy@
Fixes: 1b83e8a3f840 ("Constify string pointers.")
Fixes: 8244dd326265 ("tunefs: Better fix for arm64 alignment issues")

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 72110
Build 68993: arc lint + arc unit

Event Timeline

des created this revision.Apr 10 2026, 3:15 PM
Herald added a subscriber: imp. · View Herald TranscriptApr 10 2026, 3:15 PM
des requested review of this revision.Apr 10 2026, 3:15 PM
Harbormaster completed remote builds in B72110: Diff 175234.Apr 10 2026, 3:15 PM
des updated this revision to Diff 175235.Apr 10 2026, 3:28 PM

->

Harbormaster completed remote builds in B72111: Diff 175235.Apr 10 2026, 3:28 PM
olce added a subscriber: olce.Apr 10 2026, 3:50 PM
olce added inline comments.
sbin/tunefs/tunefs.c
717

Could you please remove as well the (uintptr_t) conversion that I suspect was here just to circumvent the bogus const in the signature?

des marked an inline comment as done.Apr 10 2026, 3:52 PM
des added inline comments.
sbin/tunefs/tunefs.c
717

No, it was added last week to suppress an alignment warning and is still needed.

des mentioned this in D56245: tunefs: Better fix for arm64 alignment issues.Apr 10 2026, 3:56 PM
des marked an inline comment as done.Apr 10 2026, 4:06 PM
kevans accepted this revision.Apr 11 2026, 4:41 PM

Whoops-

This revision is now accepted and ready to land.Apr 11 2026, 4:41 PM
Closed by commit rG9d95d806691e: tunefs: Fix pointer arithmetic (authored by des). · Explain WhyApr 11 2026, 7:23 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rG9d95d806691e: tunefs: Fix pointer arithmetic.
des mentioned this in rG84e7cd491fdf: tunefs: Better fix for arm64 alignment issues.Apr 13 2026, 5:39 PM
des mentioned this in rGc4266d84c56f: tunefs: Better fix for arm64 alignment issues.

Revision Contents
Changeset List

PathSize
sbin/
tunefs/
4 lines

Diff 175234

View Options

sbin/tunefs/tunefs.c

Loading...