RE: [PATCH 2/2] msvcp90: Copy name in _Getctype (Valgrind) (try2)

[Daniel Lehman]

> On 06/24/15 16:46, Daniel Lehman wrote:
> > - name = ___lc_locale_name_func()[LC_COLLATE];
> > - size = wcslen(name)+1;
> > - ret->name = malloc(size*sizeof(*name));
> > - if (!ret->name) throw_exception(EXCEPTION_BAD_ALLOC, NULL);
> > - memcpy(ret->name, name, size*sizeof(*name));
>
> This will not work if ___lc_locale_name_func()[LC_COLLATE] is NULL.

I updated the patch to check for NULL.  But wouldn't the same be true for these calls?
- _Getdateorder -> GetLocaleInfoEx 
- _Strcoll -> LocaleNameToLCID -> parse_locale_name

Thanks
daniel