diff options
author | Andrew Hsieh <andrewhsieh@google.com> | 2013-03-08 20:59:13 +0800 |
---|---|---|
committer | Andrew Hsieh <andrewhsieh@google.com> | 2013-03-08 20:59:13 +0800 |
commit | d3029b5a1a770d75806370618f7311a991107119 (patch) | |
tree | f71acae487aad631c3b8eb726b2cfa14d27a4cc7 | |
parent | 45ba0cf7812270b97cce13afcd15f3e669336ffb (diff) | |
download | toolchain_gcc-d3029b5a1a770d75806370618f7311a991107119.tar.gz toolchain_gcc-d3029b5a1a770d75806370618f7311a991107119.tar.bz2 toolchain_gcc-d3029b5a1a770d75806370618f7311a991107119.zip |
More fix to handle NULL return value of setlocale() in libstdc++/GCC4.7
Bionic setlocale() always returns NULL. Check return value before passing
it to other libraries which don't expect NULL in argument.
See 20ffc596a2771a96ac9b4a4838edc7779a48f0af
https://code.google.com/p/android/issues/detail?id=46718
Change-Id: Ie2bf30d5f54c77401519ecb27cb97fb905639bba
-rw-r--r-- | gcc-4.7/libstdc++-v3/config/locale/generic/c_locale.cc | 24 | ||||
-rw-r--r-- | gcc-4.7/libstdc++-v3/config/locale/generic/time_members.cc | 12 |
2 files changed, 24 insertions, 12 deletions
diff --git a/gcc-4.7/libstdc++-v3/config/locale/generic/c_locale.cc b/gcc-4.7/libstdc++-v3/config/locale/generic/c_locale.cc index a89c16fcb..783bf30ef 100644 --- a/gcc-4.7/libstdc++-v3/config/locale/generic/c_locale.cc +++ b/gcc-4.7/libstdc++-v3/config/locale/generic/c_locale.cc @@ -121,10 +121,14 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION { // Assumes __s formatted for "C" locale. char* __old = setlocale(LC_ALL, 0); - const size_t __len = strlen(__old) + 1; - char* __sav = new char[__len]; - memcpy(__sav, __old, __len); - setlocale(LC_ALL, "C"); + char* __sav = 0; + if (__old) + { + const size_t __len = strlen(__old) + 1; + __sav = new char[__len]; + memcpy(__sav, __old, __len); + setlocale(LC_ALL, "C"); + } char* __sanity; #if !__DBL_HAS_INFINITY__ @@ -166,10 +170,14 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION { // Assumes __s formatted for "C" locale. char* __old = setlocale(LC_ALL, 0); - const size_t __len = strlen(__old) + 1; - char* __sav = new char[__len]; - memcpy(__sav, __old, __len); - setlocale(LC_ALL, "C"); + char* __sav = 0; + if (__old) + { + const size_t __len = strlen(__old) + 1; + __sav = new char[__len]; + memcpy(__sav, __old, __len); + setlocale(LC_ALL, "C"); + } #if !__LDBL_HAS_INFINITY__ errno = 0; diff --git a/gcc-4.7/libstdc++-v3/config/locale/generic/time_members.cc b/gcc-4.7/libstdc++-v3/config/locale/generic/time_members.cc index a260904f0..724dd26b7 100644 --- a/gcc-4.7/libstdc++-v3/config/locale/generic/time_members.cc +++ b/gcc-4.7/libstdc++-v3/config/locale/generic/time_members.cc @@ -134,10 +134,14 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION const tm* __tm) const throw() { char* __old = setlocale(LC_ALL, 0); - const size_t __llen = strlen(__old) + 1; - char* __sav = new char[__llen]; - memcpy(__sav, __old, __llen); - setlocale(LC_ALL, _M_name_timepunct); + char* __sav = 0; + if (__old) + { + const size_t __llen = strlen(__old) + 1; + __sav = new char[__llen]; + memcpy(__sav, __old, __llen); + setlocale(LC_ALL, _M_name_timepunct); + } const size_t __len = wcsftime(__s, __maxlen, __format, __tm); setlocale(LC_ALL, __sav); delete [] __sav; |