summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2006-05-19 08:23:43 +0000
committerUlrich Drepper <drepper@redhat.com>2006-05-19 08:23:43 +0000
commit882bfe73fd0d3ee9e625ff841bd0c1e0815cff90 (patch)
treed93a017c9302cfda1b94b8757be966a2e0e75cd6
parent429bb1833efe87af5b75153b6c7fa9349cfff501 (diff)
* nis/nis_callback.c (__nis_create_callback): Calls tocvs/fedora-glibc-20060519T1550
svcudp_bufcreate and svctcp_create can fail. Free ->xprt if asprintf call fails.
-rw-r--r--ChangeLog4
-rw-r--r--nis/nis_callback.c16
2 files changed, 17 insertions, 3 deletions
diff --git a/ChangeLog b/ChangeLog
index cab8422e52..944e6bcd6e 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
2006-05-19 Ulrich Drepper <drepper@redhat.com>
+ * nis/nis_callback.c (__nis_create_callback): Calls to
+ svcudp_bufcreate and svctcp_create can fail. Free ->xprt if
+ asprintf call fails.
+
* sysdeps/unix/sysv/linux/ifaddrs.c (__netlink_sendreq): Make sure
no uninitialized memory is passed to sendto.
diff --git a/nis/nis_callback.c b/nis/nis_callback.c
index 86d2a5faeb..6582315c70 100644
--- a/nis/nis_callback.c
+++ b/nis/nis_callback.c
@@ -276,6 +276,7 @@ __nis_create_callback (int (*callback) (const_nis_name, const nis_object *,
struct sockaddr_in sin;
socklen_t len = sizeof (struct sockaddr_in);
unsigned short port;
+ int nomsg = 0;
cb = (struct nis_cb *) calloc (1,
sizeof (struct nis_cb) + sizeof (nis_server));
@@ -324,8 +325,14 @@ __nis_create_callback (int (*callback) (const_nis_name, const nis_object *,
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))
goto failed;
- cb->xprt = (flags & USE_DGRAM) ? svcudp_bufcreate (sock, 100, 8192) :
- svctcp_create (sock, 100, 8192);
+ cb->xprt = ((flags & USE_DGRAM)
+ ? svcudp_bufcreate (sock, 100, 8192)
+ : svctcp_create (sock, 100, 8192));
+ if (cb->xprt == NULL)
+ {
+ nomsg = 1;
+ goto failed;
+ }
cb->sock = cb->xprt->xp_sock;
if (!svc_register (cb->xprt, CB_PROG, CB_VERS, cb_prog_1, 0))
{
@@ -359,11 +366,14 @@ __nis_create_callback (int (*callback) (const_nis_name, const nis_object *,
failed:
if (cb)
{
+ if (cb->xprt)
+ svc_destroy (cb->xprt);
if (cb->serv)
xdr_free ((xdrproc_t) _xdr_nis_server, (char *) cb->serv);
free (cb);
}
- syslog (LOG_ERR, "NIS+: out of memory allocating callback");
+ if (!nomsg)
+ syslog (LOG_ERR, "NIS+: out of memory allocating callback");
return NULL;
}