Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 16 Dec 2011 21:57:44 +0000 (UTC)
From:      Dimitry Andric <dim@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r228599 - head/libexec/ypxfr
Message-ID:  <201112162157.pBGLviCL028558@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: dim
Date: Fri Dec 16 21:57:44 2011
New Revision: 228599
URL: http://svn.freebsd.org/changeset/base/228599

Log:
  Revert r228592, as the non-messy way of fixing ypxfr breaks other yp
  programs (e.g. usr.sbin/rpc.yppasswdd).
  
  Spotted by:	np
  MFC after:	1 week

Modified:
  head/libexec/ypxfr/yp_dbwrite.c
  head/libexec/ypxfr/ypxfr_extern.h
  head/libexec/ypxfr/ypxfr_getmap.c
  head/libexec/ypxfr/ypxfr_main.c

Modified: head/libexec/ypxfr/yp_dbwrite.c
==============================================================================
--- head/libexec/ypxfr/yp_dbwrite.c	Fri Dec 16 21:48:09 2011	(r228598)
+++ head/libexec/ypxfr/yp_dbwrite.c	Fri Dec 16 21:57:44 2011	(r228599)
@@ -58,10 +58,10 @@ yp_open_db_rw(const char *domain, const 
 	char buf[1025];
 
 
-	yp_errno = YPXFR_SUCC;
+	yp_errno = YP_TRUE;
 
 	if (map[0] == '.' || strchr(map, '/')) {
-		yp_errno = YPXFR_BADARGS;
+		yp_errno = YP_BADARGS;
 		return (NULL);
 	}
 
@@ -73,13 +73,13 @@ yp_open_db_rw(const char *domain, const 
 	if (dbp == NULL) {
 		switch (errno) {
 		case ENOENT:
-			yp_errno = YPXFR_NOMAP;
+			yp_errno = YP_NOMAP;
 			break;
 		case EFTYPE:
-			yp_errno = YPXFR_MADDR;
+			yp_errno = YP_BADDB;
 			break;
 		default:
-			yp_errno = YPXFR_YPERR;
+			yp_errno = YP_YPERR;
 			break;
 		}
 	}

Modified: head/libexec/ypxfr/ypxfr_extern.h
==============================================================================
--- head/libexec/ypxfr/ypxfr_extern.h	Fri Dec 16 21:48:09 2011	(r228598)
+++ head/libexec/ypxfr/ypxfr_extern.h	Fri Dec 16 21:57:44 2011	(r228599)
@@ -46,7 +46,7 @@ extern BTREEINFO openinfo_b;
 
 extern char	*yp_dir;
 extern int	debug;
-extern ypxfrstat yp_errno;
+extern enum ypstat	yp_errno;
 extern void	yp_error(const char *, ...);
 extern int	_yp_check(char **);
 extern const char *ypxfrerr_string(ypxfrstat);

Modified: head/libexec/ypxfr/ypxfr_getmap.c
==============================================================================
--- head/libexec/ypxfr/ypxfr_getmap.c	Fri Dec 16 21:48:09 2011	(r228598)
+++ head/libexec/ypxfr/ypxfr_getmap.c	Fri Dec 16 21:57:44 2011	(r228599)
@@ -46,6 +46,7 @@ extern bool_t xdr_ypresp_all_seq(XDR *, 
 int (*ypresp_allfn)();
 void *ypresp_data;
 extern DB *specdbp;
+extern enum ypstat yp_errno;
 
 /*
  * This is largely the same as yp_all() except we do the transfer

Modified: head/libexec/ypxfr/ypxfr_main.c
==============================================================================
--- head/libexec/ypxfr/ypxfr_main.c	Fri Dec 16 21:48:09 2011	(r228598)
+++ head/libexec/ypxfr/ypxfr_main.c	Fri Dec 16 21:57:44 2011	(r228599)
@@ -88,7 +88,7 @@ ypxfr_exit(ypxfrstat retval, char *temp)
 			exit(1);
 		}
 
-		ypxfr_resp.status = (yppush_status)retval;
+		ypxfr_resp.status = retval;
 
 		if (yppushproc_xfrresp_1(&ypxfr_resp, clnt) == NULL) {
 			yp_error("%s", clnt_sperror(clnt, "callback failed"));



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201112162157.pBGLviCL028558>