Differential D4277

Casper new architecture.
ClosedPublic
Actions

Authored by oshogbo on Nov 25 2015, 9:20 PM.

Tags

Referenced Files

	Unknown Object (File)
	Sun, Dec 8, 9:41 PM

	Unknown Object (File)
	Tue, Dec 3, 12:40 PM

	Unknown Object (File)
	Sat, Nov 30, 9:47 AM

	Unknown Object (File)
	Fri, Nov 29, 7:11 AM

	Unknown Object (File)
	Oct 3 2024, 2:16 PM

	Unknown Object (File)
	Oct 3 2024, 7:56 AM

	Unknown Object (File)
	Oct 1 2024, 1:17 PM

	Unknown Object (File)
	Sep 27 2024, 11:28 PM

Subscribers

drysdale_google.com

Details

Reviewers

ed
jonathan
bdrewery

Commits

rS296047: Convert casperd(8) daemon to the libcasper.

Summary

Threads:
https://lists.freebsd.org/pipermail/freebsd-arch/2015-November/017519.html
https://lists.cam.ac.uk/pipermail/cl-capsicum-discuss/2015-November/msg00004.html

Code is also available at:
https://github.com/oshogbo/freebsd/tree/libcasper

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

oshogbo updated this revision to Diff 10503.Nov 25 2015, 9:20 PM

oshogbo retitled this revision from to Casper new architecture..

oshogbo updated this object.

oshogbo edited the test plan for this revision. (Show Details)

oshogbo added a reviewer: jonathan.

oshogbo added a project: capsicum.

Herald added subscribers: emaste, bdrewery. · View Herald TranscriptNov 25 2015, 9:20 PM

drysdale_google.com added a subscriber: drysdale_google.com.Dec 8 2015, 4:01 PM

I *think* that nvlist_add_string() and friends don't cope with a NULL value, which means that all of the places where optional strings turn up in an API need to cope with absence. I've commented on a few such places, but there's probably more...

lib/libcasper/services/cap_dns/cap_dns.c
198 ↗	(On Diff #10503)	Should this be prefixed with a if (nvlist_exists_string(nvl, "ai_canonname")), plus an initialization of canonname to NULL above?
228–229 ↗	(On Diff #10503)	Either one of hostname/servname can be NULL, which I don't think nvlist_add_string() copes with.
299 ↗	(On Diff #10503)	add && nvlist_exists_string(nvl, "host") ?
301 ↗	(On Diff #10503)	add && nvlist_exists_string(nvl, "serv") ?
517 ↗	(On Diff #10503)	Need to initialize both of these to NULL I think
577 ↗	(On Diff #10503)	check that host/serv are != NULL ?
597 ↗	(On Diff #10503)	Check for ai->ai_canonname == NULL?
616 ↗	(On Diff #10503)	Can hostname/servname be absent?
tools/regression/capsicum/libcasper/dns.c
49 ↗	(On Diff #10503)	Presumably these tests haven't been updated yet?

Update according to David suggestion.
I forgot to commit changes about test, thanks!
You make very good points.

oshogbo marked 9 inline comments as done.Dec 14 2015, 12:10 AM

oshogbo added inline comments.

lib/libcasper/services/cap_dns/cap_dns.c
198 ↗	(On Diff #10503)	dnvlist will be bather here but you are right we should check this value.
517 ↗	(On Diff #10503)	It's done. Just before memset.

Code is also available at https://github.com/oshogbo/freebsd/tree/libcasper

oshogbo updated this object.Jan 8 2016, 3:37 PM

oshogbo added a reviewer: ed.

oshogbo added subscribers: rwatson, pjd.

Please also update these as needed:

targets/pseudo/userland/Makefile.depend:DIRDEPS+= sbin/casperd
targets/pseudo/userland/lib/Makefile.depend: lib/libcasper \
targets/pseudo/userland/libexec/Makefile.depend: libexec/casper/dns \
targets/pseudo/userland/libexec/Makefile.depend: libexec/casper/grp \
targets/pseudo/userland/libexec/Makefile.depend: libexec/casper/pwd \
targets/pseudo/userland/libexec/Makefile.depend: libexec/casper/random \
targets/pseudo/userland/libexec/Makefile.depend: libexec/casper/sysctl \

Makefile.inc1
551 ↗	(On Diff #11216)	This change should not be needed. The change in etc/mtree/BSD.root.dist will create the directory.

This revision now requires changes to proceed.Jan 8 2016, 6:55 PM

I updated Makefile.depnd which was pointed out byt @bdrewery.
I update also some dependencies in libnames and BSD.debug.dist.

Closed by commit rS296047: Convert casperd(8) daemon to the libcasper. (authored by oshogbo). · Explain WhyFeb 26 2016, 11:06 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

8 lines

ObsoleteFiles.inc

19 lines

contrib/

mdocml/

2 lines

tcpdump/

14 lines

2 lines

2 lines

2 lines

63 lines

etc/

defaults/

1 line

mtree/

2 lines

BSD.include.dist

2 lines

4 lines

rc.d/

5 lines

19 lines

gnu/

usr.bin/

groff/

tmac/

2 lines

lib/

5 lines

libc/

posix1e/

4 lines

libcapsicum/

46 lines

Makefile.depend

19 lines

115 lines

300 lines

266 lines

libcapsicum_dns.h

57 lines

libcapsicum_dns.c

365 lines

libcapsicum_grp.h

57 lines

libcapsicum_grp.c

438 lines

libcapsicum_impl.h

39 lines

libcapsicum_pwd.h

57 lines

libcapsicum_pwd.c

391 lines

libcapsicum_random.h

37 lines

libcapsicum_random.c

80 lines

libcapsicum_service.h

40 lines

libcapsicum_service.c

97 lines

libcapsicum_sysctl.h

43 lines

libcapsicum_sysctl.c

86 lines

libcasper/

19 lines

70 lines

441 lines

libcasper/

38 lines

115 lines

295 lines

337 lines

libcasper_impl.h

82 lines

libcasper_impl.c

44 lines

libcasper_service.h

60 lines

libcasper_service.c

277 lines

394 lines

41 lines

223 lines

libcasper_impl.h

46 lines

services/

9 lines

cap_dns/

20 lines

57 lines

761 lines

cap_grp/

20 lines

57 lines

787 lines

cap_pwd/

20 lines

57 lines

783 lines

cap_random/

20 lines

37 lines

117 lines

cap_sysctl/

20 lines

43 lines

293 lines

libexec/

5 lines

casper/

11 lines

dns/

21 lines

436 lines

grp/

21 lines

390 lines

pwd/

21 lines

430 lines

random/

21 lines

82 lines

sysctl/

21 lines

249 lines

rtld-elf/

2 lines

sbin/

1 line

casperd/

17 lines

132 lines

721 lines

40 lines

227 lines

ping/

5 lines

Makefile.depend

3 lines

34 lines

share/

man/

man4/

5 lines

mk/

bsd.libnames.mk

1 line

src.libnames.mk

33 lines

targets/

pseudo/

userland/

Makefile.depend

4 lines

lib/

Makefile.depend

12 lines

libexec/

Makefile.depend

5 lines

tools/

regression/

capsicum/

libcapsicum/

33 lines

588 lines

1550 lines

1536 lines

1510 lines

libcasper/

32 lines

588 lines

1550 lines

1536 lines

1510 lines

usr.bin/

kdump/

6 lines

31 lines

usr.sbin/

tcpdump/

tcpdump/

5 lines

2 lines

Diff 13786

head/Makefile.inc1

Loading...

head/ObsoleteFiles.inc

Loading...

head/contrib/mdocml/lib.in

Loading...

head/contrib/tcpdump/addrtoname.c

Loading...

head/contrib/tcpdump/config.h.in

Loading...

head/contrib/tcpdump/configure

Loading...

head/contrib/tcpdump/configure.in

Loading...

head/contrib/tcpdump/tcpdump.c

Loading...

head/etc/defaults/rc.conf

Loading...

head/etc/mtree/BSD.debug.dist

Loading...

head/etc/mtree/BSD.include.dist

Loading...

head/etc/mtree/BSD.root.dist

Loading...

head/etc/rc.d/Makefile

Loading...

head/etc/rc.d/casperd

Loading...

head/gnu/usr.bin/groff/tmac/mdoc.local.in

Loading...

head/lib/Makefile

Loading...

head/lib/libc/posix1e/posix1e.3

Loading...

head/lib/libcapsicum/Makefile

Loading...

head/lib/libcapsicum/Makefile.depend

Loading...

head/lib/libcapsicum/libcapsicum.h

Loading...

head/lib/libcapsicum/libcapsicum.3

Loading...

head/lib/libcapsicum/libcapsicum.c

Loading...

head/lib/libcapsicum/libcapsicum_dns.h

Loading...

head/lib/libcapsicum/libcapsicum_dns.c

Loading...

head/lib/libcapsicum/libcapsicum_grp.h

Loading...

head/lib/libcapsicum/libcapsicum_grp.c

Loading...

head/lib/libcapsicum/libcapsicum_impl.h

Loading...

head/lib/libcapsicum/libcapsicum_pwd.h

Loading...

head/lib/libcapsicum/libcapsicum_pwd.c

Loading...

head/lib/libcapsicum/libcapsicum_random.h

Loading...

head/lib/libcapsicum/libcapsicum_random.c

Loading...

head/lib/libcapsicum/libcapsicum_service.h

Loading...

head/lib/libcapsicum/libcapsicum_service.c

Loading...

head/lib/libcapsicum/libcapsicum_sysctl.h

Loading...

head/lib/libcapsicum/libcapsicum_sysctl.c

Loading...

head/lib/libcasper/Makefile

Loading...

head/lib/libcasper/libcasper.h

Loading...

head/lib/libcasper/libcasper.c

Loading...

head/lib/libcasper/libcasper/Makefile

Loading...

head/lib/libcasper/libcasper/libcasper.h

Loading...

head/lib/libcasper/libcasper/libcasper.3

Loading...

head/lib/libcasper/libcasper/libcasper.c

Loading...

head/lib/libcasper/libcasper/libcasper_impl.h

Loading...

head/lib/libcasper/libcasper/libcasper_impl.c

Loading...

head/lib/libcasper/libcasper/libcasper_service.h

Loading...

head/lib/libcasper/libcasper/libcasper_service.c

Loading...

head/lib/libcasper/libcasper/service.c

Loading...

head/lib/libcasper/libcasper/zygote.h

Loading...

head/lib/libcasper/libcasper/zygote.c

Loading...

head/lib/libcasper/libcasper_impl.h

Loading...

head/lib/libcasper/services/Makefile

Loading...

head/lib/libcasper/services/cap_dns/Makefile

Loading...

head/lib/libcasper/services/cap_dns/cap_dns.h

Loading...

head/lib/libcasper/services/cap_dns/cap_dns.c

Loading...

head/lib/libcasper/services/cap_grp/Makefile

Loading...

head/lib/libcasper/services/cap_grp/cap_grp.h

Loading...

head/lib/libcasper/services/cap_grp/cap_grp.c

Loading...

head/lib/libcasper/services/cap_pwd/Makefile

Loading...

head/lib/libcasper/services/cap_pwd/cap_pwd.h

Loading...

head/lib/libcasper/services/cap_pwd/cap_pwd.c

Loading...

head/lib/libcasper/services/cap_random/Makefile

Loading...

head/lib/libcasper/services/cap_random/cap_random.h

Loading...

head/lib/libcasper/services/cap_random/cap_random.c

Loading...

head/lib/libcasper/services/cap_sysctl/Makefile

Loading...

head/lib/libcasper/services/cap_sysctl/cap_sysctl.h

Loading...

head/lib/libcasper/services/cap_sysctl/cap_sysctl.c

Loading...

head/libexec/Makefile

Loading...

head/libexec/casper/Makefile

Loading...

head/libexec/casper/dns/Makefile

Loading...

head/libexec/casper/dns/dns.c

Loading...

head/libexec/casper/grp/Makefile

Loading...

head/libexec/casper/grp/grp.c

Loading...

head/libexec/casper/pwd/Makefile

Loading...

head/libexec/casper/pwd/pwd.c

Loading...

head/libexec/casper/random/Makefile

Loading...

head/libexec/casper/random/random.c

Loading...

head/libexec/casper/sysctl/Makefile

Loading...

head/libexec/casper/sysctl/sysctl.c

Loading...

head/libexec/rtld-elf/paths.h

Loading...

head/sbin/Makefile

Loading...

head/sbin/casperd/Makefile

Loading...

head/sbin/casperd/casperd.8

Loading...

head/sbin/casperd/casperd.c

Loading...

head/sbin/casperd/zygote.h

Loading...

head/sbin/casperd/zygote.c

Loading...

head/sbin/ping/Makefile

Loading...

head/sbin/ping/Makefile.depend

Loading...

head/sbin/ping/ping.c

Loading...

head/share/man/man4/capsicum.4

Loading...

head/share/mk/bsd.libnames.mk

Loading...

head/share/mk/src.libnames.mk

Loading...

head/targets/pseudo/userland/Makefile.depend

Loading...

head/targets/pseudo/userland/lib/Makefile.depend

Loading...

head/targets/pseudo/userland/libexec/Makefile.depend

Loading...

head/tools/regression/capsicum/libcapsicum/Makefile

Loading...

head/tools/regression/capsicum/libcapsicum/dns.c

Loading...

head/tools/regression/capsicum/libcapsicum/grp.c

Loading...

head/tools/regression/capsicum/libcapsicum/pwd.c

Loading...

head/tools/regression/capsicum/libcapsicum/sysctl.c

Loading...

head/tools/regression/capsicum/libcasper/Makefile

Loading...

head/tools/regression/capsicum/libcasper/dns.c

Loading...

head/tools/regression/capsicum/libcasper/grp.c

Loading...

head/tools/regression/capsicum/libcasper/pwd.c

Loading...

head/tools/regression/capsicum/libcasper/sysctl.c

Loading...

head/usr.bin/kdump/Makefile

Loading...

head/usr.bin/kdump/kdump.c

Loading...

head/usr.sbin/tcpdump/tcpdump/Makefile

Loading...

head/usr.sbin/tcpdump/tcpdump/config.h

Loading...