id,summary,reporter,owner,description,type,status,milestone,component,version,severity,resolution,keywords,cc
4685,boost::interprocess::winapi::get_last_bootup_time( std::wstring&),Jim Bell <jim@…>,Ion Gaztañaga,"boost::interprocess::winapi::get_last_bootup_time( std::wstring& strValue )

This line is wrong and is crashing most of the win32 regression tests (both release and trunk):
{{{
strValue.erase(strValue.find(L'+'));
}}}


strValue doesn't have '+', so .find() returns std::wstring::npos, but that causes std::wstring::erase() to crash. npos isn't a valid parameter to erase() and will cause an out_of_range exception to be thrown. [http://stdcxx.apache.org/doc/stdlibref/basic-string.html#idx349]

I think this is causing many regression test failures both the release and trunk branches for win32.

The fix would be to split it out:

{{{
std::wstring::size_type plusPos = strValue.find(L'+');
if (plusPos != std::wstring::npos)
   strValue.erase(plusPos);
}}}

I can't speak to why you're looking for a '+' and the significance of not finding it.
",Bugs,closed,To Be Determined,interprocess,Boost 1.44.0,Problem,fixed,,