Commit Graph

15 Commits

Author SHA1 Message Date
christos 473ea2c4e7 add explicit casts to avoid parameter conversion warnings. 2000-07-06 03:14:05 +00:00
mycroft 605490369c Delint.
Remove trailing ; from uses of __weak_alias().  The macro inserts this if
needed.
2000-01-22 22:19:07 +00:00
lukem b48252f365 * use _DIAGASSERT() to check pointer arguments against NULL and file
descriptors against -1 (as appropriate).
* add actual checks which to detect stuff that would trigger_DIAGASSERT(),
  and attempt to return a sane error condition.
* knf some code
* remove some `register' decls.

the first two items result in the addition of code similar to the
following in various functions:

		_DIAGASSERT(path != NULL)
	#ifdef _DIAGNOSTIC
		if (path == NULL) {
			errno = EFAULT;
			return (-1);
		}
	#endif
1999-09-16 11:44:54 +00:00
christos b2a14ab2d3 xdrproc_t casts 1999-01-31 20:46:12 +00:00
christos f4c1479174 delint 1998-11-15 17:10:30 +00:00
kleink 8d6b3ab02a Need <time.h> for time() prototype. 1998-03-30 15:05:20 +00:00
jtc 43fa6fe319 If port provides __weak_alias(), provide an Standard C and POSIX pure
identifier namespace by renaming non standard functions and variables
such that they have a leading underscore.  The library will use those
names internally.  Weak aliases are used to provide the original names
to the API.

This is only the first part of this change.  It is most of the functions
which are implemented in C for all NetBSD ports.  Subsequent changes are
to add the same support to the remaining C files, to assembly files, and
to the automagically generated assembly source used for system calls.
When all of the above is done, ports with weak alias support should add
a definition for __weak_alias to <sys/cdefs.h>.
1997-07-21 14:06:24 +00:00
christos e7cc550330 Fix RCSID's
printf -> warnx
Fix gcc warnings
Put locally declared functions in local.h
1997-07-13 20:28:09 +00:00
lukem 42736edb4d * implement _yp_invalid_domain, which returns non-zero if the given
domainname isn't RFC1035 compliant. Also ensure that the length <=
  YPMAXDOMAIN (which is less than what RFC1035 allows).
* use _yp_invalid_domain in the public front-ends, instead of using
  hand-rolled checks
* fix a typo
1997-07-07 02:00:30 +00:00
lukem db4fd8d56f Don't return allocated memory in an error condition in a char **outXXX argument
(set them to the null pointer instead).
Thus, code that doesn't specificallly attempt to clean up allocated
memory after an error result is returned from yp_...() won't have an
unexpected memory leak (i.e, most 3rd party code)
1997-05-21 06:55:25 +00:00
lukem 220410601a yp_match() should return errors of the form YPERR_xxx not RPC_xxx.
return YPERR_RESRC instead of RPC_SYSTEMERROR
1997-05-21 01:48:46 +00:00
lukem 301e6e6c32 Some cleanups for [lib/3655]
* Ensure that all 'char **outXXX' pointers aren't the null pointer before use.
* Set *outXXX=NULL before checking any other arguments.
* Document that *outXXX will always be NULL or a malloc(3)ed string
  unless outXXX was NULL (in which case YPERR_BADARGS will be returned
  and the caller shouldn't attempt to free(*outXXX) if *outXXX != NULL;

These changes should prevent most occurances of coredumps when a bad
argument was given to a yp client function and the caller attempts to
free an outvalue that isn't the null pointer. To be really safe, the
caller probably should set the *outvalue=NULL anyway (ref: PR [lib/3580])
1997-05-20 15:25:38 +00:00
christos 409a9590f3 Avoid using clnt_perror() on the first rpc failure, since it is going to be
retried anyway. Only report it every _yplib_nerrs times.
1996-05-23 13:48:59 +00:00
jtc 0724069f60 Use a constant struct timeval for yp timeouts instead of creating
a new struct timeval in each yp function at runtime.
Check arguments and return YPERR_BADARGS if invalid.
1996-05-18 19:01:19 +00:00
jtc 5d8adb683d Split yperr_match out of yplib.c; makes many static executables a bit smaller. 1996-05-15 05:27:50 +00:00