From 23d101d8eea76469b41c12b10a8e8ed13cf66bc6 Mon Sep 17 00:00:00 2001 From: Emilio Pozuelo Monfort Date: Tue, 20 Jul 2010 19:19:34 -0700 Subject: [PATCH] Hurd: fix sendmsg memory leak in error paths --- ChangeLog | 4 ++++ sysdeps/mach/hurd/sendmsg.c | 12 ++++++++++-- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index de7f38f8aa..fa670c5d54 100644 --- a/ChangeLog +++ b/ChangeLog @@ -5,6 +5,10 @@ ld.so.cache was broken. With it, there is no way to disable dsocaps like LD_HWCAP_MASK can disable hwcaps. +2010-06-02 Emilio Pozuelo Monfort + + * sysdeps/mach/hurd/sendmsg.c (__libc_sendmsg): Fix memory leaks. + 2010-07-16 Ulrich Drepper * sysdeps/x86_64/multiarch/strstr.c: Rewrite to avoid indirect function diff --git a/sysdeps/mach/hurd/sendmsg.c b/sysdeps/mach/hurd/sendmsg.c index a9d1c8c1d4..118fd59a5b 100644 --- a/sysdeps/mach/hurd/sendmsg.c +++ b/sysdeps/mach/hurd/sendmsg.c @@ -109,14 +109,22 @@ __libc_sendmsg (int fd, const struct msghdr *message, int flags) and talk to it with the ifsock protocol. */ file_t file = __file_name_lookup (addr->sun_path, 0, 0); if (file == MACH_PORT_NULL) - return -1; + { + if (dealloc) + __vm_deallocate (__mach_task_self (), data.addr, len); + return -1; + } err = __ifsock_getsockaddr (file, &aport); __mach_port_deallocate (__mach_task_self (), file); if (err == MIG_BAD_ID || err == EOPNOTSUPP) /* The file did not grok the ifsock protocol. */ err = ENOTSOCK; if (err) - return __hurd_fail (err); + { + if (dealloc) + __vm_deallocate (__mach_task_self (), data.addr, len); + return __hurd_fail (err); + } } else err = EIEIO; -- 2.11.4.GIT