aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-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 e7a2187..7bfbb30 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 14041a1..07f95f0 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 5f2fb59..4cb34dd 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;
}