diff options
author | Aurelien Jarno <aurelien@aurel32.net> | 2015-03-11 21:03:50 -0400 |
---|---|---|
committer | Carlos O'Donell <carlos@systemhalted.org> | 2015-03-11 21:07:32 -0400 |
commit | 6a1cf708dd5681b517744d6d4fac02e4e4a0aa2e (patch) | |
tree | a965b2d1be84995f55dbfdeb479b4a2b74f1ec69 /stdlib/tst-random2.c | |
parent | a2d4cf72c0ab07d4e58b42c01ac3ed2b95ca8d9b (diff) | |
download | glibc-6a1cf708dd5681b517744d6d4fac02e4e4a0aa2e.tar.gz glibc-6a1cf708dd5681b517744d6d4fac02e4e4a0aa2e.tar.xz glibc-6a1cf708dd5681b517744d6d4fac02e4e4a0aa2e.zip |
Fix ldconfig segmentation fault with corrupted cache (Bug 18093).
ldconfig is using an aux-cache to speed up the ld.so.cache update. It is read by mmaping the file to a structure which contains data offsets used as pointers. As they are not checked, it is not hard to get ldconfig to segfault with a corrupted file. This happens for instance if the file is truncated, which is common following a filesystem check following a system crash. This can be reproduced for example by truncating the file to roughly half of it's size. There is already some code in elf/cache.c (load_aux_cache) to check for a corrupted aux cache, but it happens to be broken and not enough. The test (aux_cache->nlibs >= aux_cache_size) compares the number of libs entry with the cache size. It's a non sense, as it basically assumes that each library entry is a 1 byte... Instead this commit computes the theoretical cache size using the headers and compares it to the real size.
Diffstat (limited to 'stdlib/tst-random2.c')
0 files changed, 0 insertions, 0 deletions