[vbox-dev] Null pointer access in VMEmt.cpp by vmR3EmulationThreadWithId

[Alexander Boettcher]

Hello,

On 20.03.2017 08:43, Frank Mehnert wrote:
> On Freitag, 17. März 2017 16:00:11 CET Knut St. Osmundsen wrote:
>> thanks for identifying a race.  Looking at the code, I can only imagine
>> this happening if the other EMTs (not 0) are late into the loop due to
>> busy host / scheduling fun.  The only alternative I can see would be
>> some wait error in vmR3BootstrapWait and causing it to return.
>>
>> I've committed a different fix than what you suggest in your patch,
>> VMEmt.cpp should check that both pVM and pVCpu are sane before selecting
>> the non-bootstrap code path.  The fix should appear in the public SVN soon.
> 
> for reference:
> 
>   https://www.virtualbox.org/changeset/66096/vbox and
>   https://www.virtualbox.org/changeset/66097/vbox

Thanks.

Alex.

-- 
Alexander Boettcher
Genode Labs

http://www.genode-labs.com - http://www.genode.org

Genode Labs GmbH - Amtsgericht Dresden - HRB 28424 - Sitz Dresden
Geschäftsführer: Dr.-Ing. Norman Feske, Christian Helmuth