Page MenuHomeFreeBSD
Differential D17852

Use declared types for caddr_t arguments.
ClosedPublic
Actions

Authored by brooks on Nov 5 2018, 10:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 8, 8:00 PM
Unknown Object (File)
Nov 14 2024, 12:48 PM
Unknown Object (File)
Oct 1 2024, 9:29 PM
Unknown Object (File)
Sep 17 2024, 2:12 PM
Unknown Object (File)
Sep 8 2024, 6:01 AM
Unknown Object (File)
Sep 5 2024, 2:18 PM
Unknown Object (File)
Sep 1 2024, 11:19 PM
Unknown Object (File)
Sep 1 2024, 3:33 PM
View All 35 Files
Subscribers
imp

Details

Reviewers
kib
emaste
jhb
Commits
rS340200: Regen after r340199: Use declared types for caddr_t arguments.
rS340199: Use declared types for caddr_t arguments.
Summary

Leave ptrace(2) alone for the moment as it's defined to take a caddr_t.

Note: these changes don't compile without the followup regeneration
commit.

Diff Detail

Lint
Lint Warnings
SeverityLocationCodeMessage
Warningsys/fs/nfsclient/nfs_clvfsops.c:1241SPELL1Possible Spelling Mistake
Unit
No Test Coverage
Build Status
Buildable 20647
Build 20063: arc lint + arc unit

Event Timeline

brooks created this revision.Nov 5 2018, 10:29 PM
Harbormaster completed remote builds in B20647: Diff 50043.Nov 5 2018, 10:29 PM
kib accepted this revision.Nov 6 2018, 2:05 AM
kib added inline comments.
sys/compat/linux/linux_socket.c
1577

linux_to_bsd_sockaddr is an awful hack (not related to this change.) And deconst there demonstrates why it is also incorrect.

sys/kern/syscalls.master
339

What about struct pstats::struct uprof::pr_base ?

This revision is now accepted and ready to land.Nov 6 2018, 2:05 AM
brooks marked 2 inline comments as done.Nov 6 2018, 4:08 PM
brooks added inline comments.
sys/compat/linux/linux_socket.c
1577

Indeed. I created https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=232920 to document its awfulness when I found it. The shortest path is probably to use copyout_map rather than doing this translation in place.

sys/kern/syscalls.master
339

I went with char * because that's what the manpage and header say.

Closed by commit rS340199: Use declared types for caddr_t arguments. (authored by brooks). · Explain WhyNov 6 2018, 6:46 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptNov 6 2018, 6:46 PM

Revision Contents
Changeset List

PathSize
sys/
compat/
freebsd32/
52 lines
linux/
9 lines
fs/
nfsclient/
3 lines
kern/
54 lines
12 lines
sys/
3 lines
2 lines

Diff 50043

View Options

sys/compat/freebsd32/syscalls.master

Loading...
View Options

sys/compat/linux/linux_socket.c

Loading...
View Options

sys/fs/nfsclient/nfs_clvfsops.c

Loading...
View Options

sys/kern/syscalls.master

Loading...
View Options

sys/kern/uipc_syscalls.c

Loading...
View Options

sys/sys/socketvar.h

Loading...
View Options

sys/sys/syscallsubr.h

Loading...