Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- WebContents*
- PrintPreviewDialogController::CreatePrintPreviewDialog(
- WebContents* initiator)
- {
- ....
- Browser* current_browser = new Browser(
- Browser::CreateParams(Browser::TYPE_POPUP, profile,
- chrome::GetActiveDesktop()));
- if (!current_browser) {
- NOTREACHED() << "Failed to create popup browser window";
- return NULL;
- }
- ....
- }
- This suspicious code was found in Chromium project by PVS-Studio static code analyzer.
- Warning message is:
- V668 There is no sense in testing the 'current_browser' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. print_preview_dialog_controller.cc 403
- 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