PVS: V522 There might be dereferencing of a potential null pointer

This commit is contained in:
wmayer
2019-03-13 12:47:05 +01:00
parent f611102db3
commit e392fc2f07

View File

@@ -274,9 +274,11 @@ void CmdRaytracingWriteView::activated(int)
Gui::ViewProvider* vp = getActiveGuiDocument()->getViewProvider(*it);
if (vp && vp->isVisible()) {
App::PropertyColor *pcColor = dynamic_cast<App::PropertyColor *>(vp->getPropertyByName("ShapeColor"));
App::Color col = pcColor->getValue();
doCommand(Doc,"content += Raytracing.getPartAsPovray('%s',App.activeDocument().%s.Shape,%f,%f,%f)",
(*it)->getNameInDocument(),(*it)->getNameInDocument(),col.r,col.g,col.b);
if (pcColor) {
App::Color col = pcColor->getValue();
doCommand(Doc,"content += Raytracing.getPartAsPovray('%s',App.activeDocument().%s.Shape,%f,%f,%f)",
(*it)->getNameInDocument(),(*it)->getNameInDocument(),col.r,col.g,col.b);
}
}
}
doCommand(Doc,"result = result.replace('//RaytracingContent',content)");