1) The yp_bind/yp_unbind and internal _yp_dobind/_yp_unbind sequences shared
dom_binding structs between threads, which is unsafe -- example, dom_vers
signalled retry events, and structs+socket would

1) The yp_bind/yp_unbind and internal _yp_dobind/_yp_unbind sequences shared
dom_binding structs between threads, which is unsafe -- example, dom_vers
signalled retry events, and structs+socket would get deallocated in _yp_unbind.
Change all yp_first (and similar) functions to understand that _yp_dobind now
provides a private dom_binding and socket, which must be released using
_yp_unbind. Use similar methods in the one-step yp_all function.
2) domainname caching in get* is not neccessary now that the domainname cannot
change relative to ypconnect(2)'s decisions.
Many fields in dom_binding struct become unused, so delete them.
ok jmatthew, also tested by miod

show more ...

Wrap <rpcsvc/ypclnt.h> so that calls go direct and the symbols are all weak

Move to the <limits.h> universe.
review by millert, binary checking process with doug, concept with guenther

Two minor bug fixes rotting in my tree:
(1) When the second malloc in yp_next fails, do not leak the memory
allocated by the first one. Same fix as yp_first.c rev. 1.9.
(2) When compiled with YPMATC

Two minor bug fixes rotting in my tree:
(1) When the second malloc in yp_next fails, do not leak the memory
allocated by the first one. Same fix as yp_first.c rev. 1.9.
(2) When compiled with YPMATCHCACHE, do not fail the lookup when
reserving memory for the cache fails. Instead, just return the
correct result without caching it.
ok millert@

show more ...

In case of memory exhaustion, ypmatch_add may both leak memory and leave
invalid data on the list, inviting later NULL pointer access.
noticed by deraadt@, algorithm proposed by millert, implemented

In case of memory exhaustion, ypmatch_add may both leak memory and leave
invalid data on the list, inviting later NULL pointer access.
noticed by deraadt@, algorithm proposed by millert, implemented by me;
feedback and ok millert@

show more ...

malloc(3) failure should raise YPERR_RESRC, not YPERR_YPERR;
ok millert@

- remove rcsid
- put all the YPMATCHCACHE code into a #ifdef, proper
- remove all extra header files that are not needed

okay deraadt@

Move static funcion and variable declarations from header to the
.c file where they are used. Gets rid of YPMATCHCACHE define.
Sure deraadt@

remove terms 3 & 4 on my own software

if ypmatch_add sees short keys or values, abort early

ansi

for yp_match() and yp_next(), do not permit keys longer than YPMAXRECORD

never bitch about first failure

Remove dead code
Remove unused variables
Silence some warnings
lint(1) is your friend

Fix RCS ids
Make sure everything uses {SYS,}LIBC_SCCS properly

update rcsid

repair return values, more checking

yp_prot.h lies -> use yp.h (from yp.x) instead, change lots of structure
accesses to match. change xdr_domainname() and some other functions to
take "char **" instead of "char *", as rpcgen intended

yp_prot.h lies -> use yp.h (from yp.x) instead, change lots of structure
accesses to match. change xdr_domainname() and some other functions to
take "char **" instead of "char *", as rpcgen intended -- helps programs
linking against rpcgen output of yp.x. yp_all() should not free it's
return value. split xdryp.c and yplib.c into ~20 files so binaries like
"csh" need not link in functions like yp_maplist(). NIS+'s YP emulation
code lacks YPPROC_ORDER... attempt to deal nicely.

show more ...