aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndreas Schwab <schwab@suse.de>2018-11-06 16:12:07 +0100
committerFlorian Weimer <fweimer@redhat.com>2018-11-06 16:12:07 +0100
commitfc0e3393ff775aa795b523083bb0db7f18d3b91e (patch)
tree2db3acca3876c5822959c6f147f2dc1c86ab1a37
parent65010329f2c596bdd1204c1c9c9baac0193637af (diff)
downloadglibc-fc0e3393ff775aa795b523083bb0db7f18d3b91e.zip
glibc-fc0e3393ff775aa795b523083bb0db7f18d3b91e.tar.gz
glibc-fc0e3393ff775aa795b523083bb0db7f18d3b91e.tar.bz2
libanl: properly cleanup if first helper thread creation failed (bug 22927)
(cherry picked from commit bd3b0fbae33a9a4cc5e2daf049443d5cf03d4251)
-rw-r--r--ChangeLog6
-rw-r--r--NEWS1
-rw-r--r--resolv/gai_misc.c7
3 files changed, 12 insertions, 2 deletions
diff --git a/ChangeLog b/ChangeLog
index d558df5..db4ac3b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2018-11-05 Andreas Schwab <schwab@suse.de>
+
+ [BZ #22927]
+ * resolv/gai_misc.c (__gai_enqueue_request): Don't crash if
+ creating the first helper thread failed.
+
2018-10-23 Adhemerval Zanella <adhemerval.zanella@linaro.org>
[BZ #23709]
diff --git a/NEWS b/NEWS
index f300491..b85be4a 100644
--- a/NEWS
+++ b/NEWS
@@ -10,6 +10,7 @@ Version 2.28.1
The following bugs are resolved with this release:
[20209] localedata: Spelling mistake for Sunday in Greenlandic kl_GL
+ [22927] libanl: properly cleanup if first helper thread creation failed
[23400] stdlib/test-bz22786.c creates temporary files in glibc source tree
[23497] readdir64@GLIBC_2.1 cannot parse the kernel directory stream
[23521] nss_files aliases database file stream leak
diff --git a/resolv/gai_misc.c b/resolv/gai_misc.c
index e7c3b63..80a2cff 100644
--- a/resolv/gai_misc.c
+++ b/resolv/gai_misc.c
@@ -261,8 +261,11 @@ __gai_enqueue_request (struct gaicb *gaicbp)
/* We cannot create a thread in the moment and there is
also no thread running. This is a problem. `errno' is
set to EAGAIN if this is only a temporary problem. */
- assert (lastp->next == newp);
- lastp->next = NULL;
+ assert (requests == newp || lastp->next == newp);
+ if (lastp != NULL)
+ lastp->next = NULL;
+ else
+ requests = NULL;
requests_tail = lastp;
newp->next = freelist;