about summary refs log tree commit diff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2005-12-08 20:32:12 +0000
committerUlrich Drepper <drepper@redhat.com>2005-12-08 20:32:12 +0000
commit8c058eec1e5ea88c64056fe2edef6a11b174756c (patch)
treec3d18cf9c0e670c0d5bc1c2a97ad9915424c7692
parente975f14ee90a2ddb80afd9517670d61a80c00a02 (diff)
downloadglibc-8c058eec1e5ea88c64056fe2edef6a11b174756c.tar.gz
glibc-8c058eec1e5ea88c64056fe2edef6a11b174756c.tar.xz
glibc-8c058eec1e5ea88c64056fe2edef6a11b174756c.zip
* nis/nis_call.c (__nisbind_create): Remove __nisbind_destroy,
	->clnt cannot be != NULL.
	(__do_niscall): No need to use __nisbind_destroy, __nisbind_next
	did it.
	(__nisbind_connect): use_auth is already TRUE, otherwise we would
	not be here.

	* nis/nis_lookup.c (nis_lookup): Remove unnecessary
	__nisbind_destroy calls.  __nisbind_next does all that.
-rw-r--r--ChangeLog10
-rw-r--r--nis/nis_call.c9
-rw-r--r--nis/nis_lookup.c2
3 files changed, 12 insertions, 9 deletions
diff --git a/ChangeLog b/ChangeLog
index e7a2187843..7bfbb3003c 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,15 @@
 2005-12-08  Jakub Jelinek  <jakub@redhat.com>
 
+	* nis/nis_call.c (__nisbind_create): Remove __nisbind_destroy,
+	->clnt cannot be != NULL.
+	(__do_niscall): No need to use __nisbind_destroy, __nisbind_next
+	did it.
+	(__nisbind_connect): use_auth is already TRUE, otherwise we would
+	not be here.
+
+	* nis/nis_lookup.c (nis_lookup): Remove unnecessary
+	__nisbind_destroy calls.  __nisbind_next does all that.
+
 	* nis/ypclnt.c (yp_update): Before trying default authentication,
 	free DES descriptor.
 
diff --git a/nis/nis_call.c b/nis/nis_call.c
index 14041a160c..07f95f041c 100644
--- a/nis/nis_call.c
+++ b/nis/nis_call.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 1997, 1998, 2001, 2004 Free Software Foundation, Inc.
+/* Copyright (C) 1997, 1998, 2001, 2004, 2005 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Thorsten Kukuk <kukuk@vt.uni-paderborn.de>, 1997.
 
@@ -179,7 +179,6 @@ __nisbind_connect (dir_binding *dbp)
 	}
       else
 	dbp->clnt->cl_auth = authunix_create_default ();
-      dbp->use_auth = TRUE;
     }
 
   return NIS_SUCCESS;
@@ -215,10 +214,7 @@ __nisbind_create (dir_binding *dbp, const nis_server *serv_val,
 
   dbp->class = -1;
   if (__nis_findfastest (dbp) < 1)
-    {
-      __nisbind_destroy (dbp);
-      return NIS_NAMEUNREACHABLE;
-    }
+    return NIS_NAMEUNREACHABLE;
 
   return NIS_SUCCESS;
 }
@@ -585,7 +581,6 @@ __do_niscall (const_nis_name name, u_long prog, xdrproc_t xargs,
 	  if (__nisbind_next (&bptr) != NIS_SUCCESS)
 	    {
 	      nis_free_directory (dir);
-	      __nisbind_destroy (&bptr);
 	      return NIS_NAMEUNREACHABLE;
 	    }
 	}
diff --git a/nis/nis_lookup.c b/nis/nis_lookup.c
index 5f2fb59d79..4cb34dd1a8 100644
--- a/nis/nis_lookup.c
+++ b/nis/nis_lookup.c
@@ -81,7 +81,6 @@ nis_lookup (const_nis_name name, const unsigned int flags)
 	{
 	  if (__nisbind_next (&bptr) != NIS_SUCCESS)
 	    {
-	      __nisbind_destroy (&bptr);
 	      nis_free_directory (dir);
 	      NIS_RES_STATUS (res) = NIS_NAMEUNREACHABLE;
 	      return res;
@@ -167,7 +166,6 @@ nis_lookup (const_nis_name name, const unsigned int flags)
 		      {
 			if (__nisbind_next (&bptr) != NIS_SUCCESS)
 			  {
-			    __nisbind_destroy (&bptr);
 			    nis_free_directory (dir);
 			    return res;
 			  }