about summary refs log tree commit diff
path: root/locale
diff options
context:
space:
mode:
authorAurelien Jarno <aurelien@aurel32.net>2013-12-03 11:16:16 +0100
committerAurelien Jarno <aurelien@aurel32.net>2013-12-03 11:16:16 +0100
commit18d1425127e5712dda888bb280d5d1a038a45c7f (patch)
tree1633427d7b3f7ea9abac01473faa517788839268 /locale
parent520d437b9455560d099fe6bd9664be1f9f76868b (diff)
downloadglibc-18d1425127e5712dda888bb280d5d1a038a45c7f.tar.gz
glibc-18d1425127e5712dda888bb280d5d1a038a45c7f.tar.xz
glibc-18d1425127e5712dda888bb280d5d1a038a45c7f.zip
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.
Diffstat (limited to 'locale')
-rw-r--r--locale/loadarchive.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/locale/loadarchive.c b/locale/loadarchive.c
index 70136dcf95..f723780ce1 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);