From: Aurelien Jarno <aurelien@aurel32.net>
Date: Tue, 3 Dec 2013 10:16:16 +0000 (+0100)
Subject: locale: don't crash if locale-archive contains all zeros
X-Git-Tag: upstream/2.30~8334
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=18d1425127e5712dda888bb280d5d1a038a45c7f;p=external%2Fglibc.git

locale: don't crash if locale-archive contains all zeros

In case of power failure followed by filesystem issues locale-archive
can end-up containing all zeros. In that case all calls to setlocale()
generate a SIGFPE. This renders a system with a default non-C locale
unbootable.

Avoid this by ignoring the locale instead of generating a SIGFPE.
---

diff --git a/ChangeLog b/ChangeLog
index 182051a..aec3cb2 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2013-12-03  Aurelien Jarno  <aurelien@aurel32.net>
+
+	* locale/loadarchive.c (_nl_load_locale_from_archive): Avoid
+	division by 0.
+
 2013-12-03  Siddhesh Poyarekar  <siddhesh@redhat.com>
 
 	[BZ #16195]
diff --git a/locale/loadarchive.c b/locale/loadarchive.c
index 70136dc..f723780 100644
--- a/locale/loadarchive.c
+++ b/locale/loadarchive.c
@@ -274,6 +274,10 @@ _nl_load_locale_from_archive (int category, const char **namep)
   namehashtab = (struct namehashent *) ((char *) head
 					+ head->namehash_offset);
 
+  /* Avoid division by 0 if the file is corrupted.  */
+  if (__glibc_unlikely (head->namehash_size == 0))
+    goto close_and_out;
+
   idx = hval % head->namehash_size;
   incr = 1 + hval % (head->namehash_size - 2);