aboutsummaryrefslogtreecommitdiff
path: root/nis/nis_callback.c
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2000-04-27 05:42:19 +0000
committerUlrich Drepper <drepper@redhat.com>2000-04-27 05:42:19 +0000
commit316518d610789256841f929e3c5757d1e385fcd5 (patch)
treecf53931201515683b375870c359e5ec81ffe6088 /nis/nis_callback.c
parentcd201e387c261f9684a76a51e63f194bf49faa3f (diff)
downloadglibc-316518d610789256841f929e3c5757d1e385fcd5.tar
glibc-316518d610789256841f929e3c5757d1e385fcd5.tar.gz
glibc-316518d610789256841f929e3c5757d1e385fcd5.tar.bz2
glibc-316518d610789256841f929e3c5757d1e385fcd5.zip
Update.
2000-04-26 Ulrich Drepper <drepper@redhat.com> * iconv/gconv_simple.c (utf8_internal_loop): Correctly reconstruct stored character in state in UNPACK_BYTES macro. * iconv/loop.c (SINGLE(LOOPFCT)): Make it actually work. Correct test for available characters, handle result of BODY code correctly. * localedata/Makefile (test-srcs): Add tst-mbswcs1. (distribute): Add tst-mbswcs.sh. Add rule to run tst-mbswcs.sh. * localedata/tst-mbswcs.sh: New file. * localedata/tst-mbswcs1.c: New file. 2000-04-26 Jakub Jelinek <jakub@redhat.com> * nis/nis_callback.c (__nis_create_callback): Do failed memory allocation fixups centrally, fix __builtin_expect call, return NULL on failure, not NIS_NOMEMORY. 2000-04-27 Bruno Haible <haible@clisp.cons.org>
Diffstat (limited to 'nis/nis_callback.c')
-rw-r--r--nis/nis_callback.c48
1 files changed, 23 insertions, 25 deletions
diff --git a/nis/nis_callback.c b/nis/nis_callback.c
index 8f59b587fa..dca870127f 100644
--- a/nis/nis_callback.c
+++ b/nis/nis_callback.c
@@ -279,29 +279,21 @@ __nis_create_callback (int (*callback) (const_nis_name, const nis_object *,
unsigned short port;
cb = (struct nis_cb *) calloc (1, sizeof (struct nis_cb));
- if (__builtin_expect (cb == NULL, ))
- {
- syslog (LOG_ERR, "NIS+: out of memory allocating callback");
- return NULL;
- }
-
+ if (__builtin_expect (cb == NULL, 0))
+ goto failed;
cb->serv = (nis_server *) calloc (1, sizeof (nis_server));
if (__builtin_expect (cb->serv == NULL, 0))
- {
- free (cb);
- syslog (LOG_ERR, "NIS+: out of memory allocating callback");
- return NULL;
- }
+ goto failed;
cb->serv->name = strdup (nis_local_principal ());
if (__builtin_expect (cb->serv->name == NULL, 0))
- return NIS_NOMEMORY;
+ goto failed;
cb->serv->ep.ep_val = (endpoint *) calloc (2, sizeof (endpoint));
if (__builtin_expect (cb->serv->ep.ep_val == NULL, 0))
- return NIS_NOMEMORY;
+ goto failed;
cb->serv->ep.ep_len = 1;
cb->serv->ep.ep_val[0].family = strdup ("inet");
if (__builtin_expect (cb->serv->ep.ep_val[0].family == NULL, 0))
- return NIS_NOMEMORY;
+ goto failed;
cb->callback = callback;
cb->userdata = userdata;
@@ -331,18 +323,11 @@ __nis_create_callback (int (*callback) (const_nis_name, const nis_object *,
#endif
}
- if (flags & USE_DGRAM)
- {
- cb->serv->ep.ep_val[0].proto = strdup ("udp");
- cb->xprt = svcudp_bufcreate (sock, 100, 8192);
- }
- else
- {
- cb->serv->ep.ep_val[0].proto = strdup ("tcp");
- cb->xprt = svctcp_create (sock, 100, 8192);
- }
+ cb->serv->ep.ep_val[0].proto = strdup ((flags & USE_DGRAM) ? "udp" : "tcp");
if (__builtin_expect (cb->serv->ep.ep_val[0].proto == NULL, 0))
- return NIS_NOMEMORY;
+ goto failed;
+ cb->xprt = (flags & USE_DGRAM) ? svcudp_bufcreate (sock, 100, 8192) :
+ svctcp_create (sock, 100, 8192);
cb->sock = cb->xprt->xp_sock;
if (!svc_register (cb->xprt, CB_PROG, CB_VERS, cb_prog_1, 0))
{
@@ -372,6 +357,19 @@ __nis_create_callback (int (*callback) (const_nis_name, const nis_object *,
cb->serv->ep.ep_val[0].uaddr = strdup (addr);
return cb;
+
+ failed:
+ if (cb)
+ {
+ if (cb->serv)
+ {
+ xdr_free ((xdrproc_t) _xdr_nis_server, (char *) cb->serv);
+ free (cb->serv);
+ }
+ free (cb);
+ }
+ syslog (LOG_ERR, "NIS+: out of memory allocating callback");
+ return NULL;
}
nis_error