Static some variables and clean up extra new lines to make the code more readable.

[araujo]

Summary

I'm gonna start to update the NIS softwares, this is the first update to make the code a bit more readable.

Also, some tools are from OpenBSD, and I intend to update those for the next couple weeks.

The NIS Server(ypserver), was wrote by wpaul@ and the implementation is a bit different than the one on OpenBSD, in this case, I will rewrite few things, instead of sync with the OpenBSD project.

NOTE: I have contacted wpaul@ and asked him if I can change the license from 4 clauses BSD to a more modern one 2 CLAUSES.

Test Plan

The test were done as following:

1. Create a NIS server.
2. Use a NIS client to cat some mapped files such like:

root@srcCODE:~ # ypwhich
10.0.1.11
root@srcCODE:~ # ypwhich -x
"passwd" is an alias for "passwd.byname"
"master.passwd" is an alias for "master.passwd.byname"
"shadow" is an alias for "shadow.byname"
"group" is an alias for "group.byname"
"networks" is an alias for "networks.byaddr"
"hosts" is an alias for "hosts.byaddr"
"protocols" is an alias for "protocols.bynumber"
"services" is an alias for "services.byname"
"aliases" is an alias for "mail.aliases"
"ethers" is an alias for "ethers.byname"

passwd:
root@srcCODE:~ # ypcat -d freebsd passwd
smmsp:*:25:25:Sendmail Submission User:/var/spool/clientmqueue:/usr/sbin/nologin
hast:*:845:845:HAST unprivileged user:/var/empty:/usr/sbin/nologin
bin:*:3:7:Binaries Commands and Source:/:/usr/sbin/nologin
tty:*:4:65533:Tty Sandbox:/:/usr/sbin/nologin
kmem:*:5:65533:KMem Sandbox:/:/usr/sbin/nologin
sshd:*:22:22:Secure Shell Daemon:/var/empty:/usr/sbin/nologin
proxy:*:62:62:Packet Filter pseudo-user:/nonexistent:/usr/sbin/nologin
unbound:*:59:59:Unbound DNS Resolver:/var/unbound:/usr/sbin/nologin
_pflogd:*:64:64:pflogd privsep user:/var/empty:/usr/sbin/nologin
nobody:*:65534:65534:Unprivileged user:/nonexistent:/usr/sbin/nologin
root:*:0:0:Charlie &:/root:/bin/csh
toor:*:0:0:Bourne-again Superuser:/root:
daemon:*:1:1:Owner of many system processes:/root:/usr/sbin/nologin
operator:*:2:5:System &:/:/usr/sbin/nologin
man:*:9:9:Mister Man Pages:/usr/share/man:/usr/sbin/nologin
games:*:7:13:Games pseudo-user:/usr/games:/usr/sbin/nologin
news:*:8:8:News Subsystem:/:/usr/sbin/nologin
bind:*:53:53:Bind Sandbox:/:/usr/sbin/nologin
uucp:*:66:66:UUCP pseudo-user:/var/spool/uucppublic:/usr/local/libexec/uucp/uucico
www:*:80:80:World Wide Web Owner:/nonexistent:/usr/sbin/nologin
mailnull:*:26:26:Sendmail Default User:/var/spool/mqueue:/usr/sbin/nologin
_dhcp:*:65:65:dhcp programs:/var/empty:/usr/sbin/nologin
pop:*:68:6:Post Office Owner:/nonexistent:/usr/sbin/nologin
auditdistd:*:78:77:Auditdistd unprivileged user:/var/empty:/usr/sbin/nologin
git_daemon:*:964:964:git daemon:/nonexistent:/usr/sbin/nologin

hosts:
root@srcCODE:~ # ypcat -d freebsd hosts
::1 localhost localhost.my.domain
127.0.0.1 localhost localhost.my.domain

[araujo]

Forgot to update some other yp* tools.

[araujo]

Fix small space/tab for a comment.

[kib]

Tests should include retrieval of all yp maps. Do not limit this to the most common, lik passwd and group.

As the very important testing step, compare the symbol table of the pre- and post-patch libc. Diff the sorted output of nm --dynamic on libc.so.7 for start; you might need to strip the first column which contains the symbol address.

usr.bin/ypcat/ypcat.c

52 ↗

(On Diff #7339)

While there, could you (try to) constify all the tables ? I think both the array itself could be constifyied, and the char * members could be tried to convert to const char *.

[araujo]

usr.bin/ypcat/ypcat.c

52 ↗

(On Diff #7339)

It is possible to constify all the tables, but I can't constify the char * members because of the function:

/usr/include/rpcsvc/ypclnt.h:86:34
int yp_all(char *indomain, char *inmap, struct ypall_callback *incallback);

It is possible to convert to const, but then I need to convert that function and touch in other parts of the system.

[araujo]

In D3166#64420, @kib wrote:

Tests should include retrieval of all yp maps. Do not limit this to the most common, lik passwd and group.

As the very important testing step, compare the symbol table of the pre- and post-patch libc. Diff the sorted output of nm --dynamic on libc.so.7 for start; you might need to strip the first column which contains the symbol address.

I checked using nm(1) and the result I can see all yp_* function that should be visible externally. Here is a wc(1) to count the lines with/without the patch.

root@srcCODE:/usr/src # cat /root/without_patch.txt | wc -l

3116

root@srcCODE:/usr/src # cat /root/with_patch.txt | wc -l

3116

[araujo]

Constify all tables, suggested by @kib.

[araujo]

In D3166#64420, @kib wrote:

Tests should include retrieval of all yp maps. Do not limit this to the most common, lik passwd and group.

I made the test with all maps exposed by ypwhich -x, I just didn't past here because the output is a bit big.

[kib]

In D3166#64621, @araujo wrote:

I checked using nm(1) and the result I can see all yp_* function that should be visible externally. Here is a wc(1) to count the lines with/without the patch.

Do diff, it is more reliable than line number calculations, due to pecularities of nm (it could list duplicates, it lists file and section names).

Otherwise I am fine with this.