Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- STDMETHODIMP SessionMachine::EndTakingSnapshot(BOOL aSuccess)
- {
- ....
- if (fOnline)
- //no need to test for whether the saved state file is shared:
- //an online snapshot means that a new saved state file was
- //created, which we must clean up now
- RTFileDelete(mConsoleTaskData.mSnapshot->....);
- machineLock.acquire(); //<==
- mConsoleTaskData.mSnapshot->uninit();
- machineLock.release();
- ....
- }
- This suspicious code was found in VirtualBox project by PVS-Studio static code analyzer.
- Warning message is:
- V640 The code's operational logic does not correspond with its formatting. The statement is indented to the right, but it is always executed. It is possible that curly brackets are missing. snapshotimpl.cpp 1649
- PVS-Studio is a static analyzer for detecting bugs in the source code of applications written in C, C++, C++11, C++/CX. Site: http://www.viva64.com/en/pvs-studio/
Advertisement
Add Comment
Please, Sign In to add comment