Check for nullptr before resolving inverses

aothms · aothms · commit 352e361ea51a · 2024-12-19T10:40:59.000+01:00
diff --git a/src/ifcparse/IfcUtil.cpp b/src/ifcparse/IfcUtil.cpp
@@ -231,6 +231,9 @@ AttributeValue IfcUtil::IfcBaseEntity::get(const std::string& name) const
 }
 
 aggregate_of_instance::ptr IfcUtil::IfcBaseEntity::get_inverse(const std::string& name) const {
+    if (file_ == nullptr) {
+        throw IfcParse::IfcException("Instance not added to file");
+    }
     const std::vector<const IfcParse::inverse_attribute*> attrs = declaration().as_entity()->all_inverse_attributes();
     std::vector<const IfcParse::inverse_attribute*>::const_iterator iter = attrs.begin();
     for (; iter != attrs.end(); ++iter) {

Original file line number	Diff line number	Diff line change
`@@ -231,6 +231,9 @@ AttributeValue IfcUtil::IfcBaseEntity::get(const std::string& name) const`
`231`	`231`	`}`
`232`	`232`
`233`	`233`	`aggregate_of_instance::ptr IfcUtil::IfcBaseEntity::get_inverse(const std::string& name) const {`
	`234`	`+ if (file_ == nullptr) {`
	`235`	`+ throw IfcParse::IfcException("Instance not added to file");`
	`236`	`+ }`
`234`	`237`	`const std::vector<const IfcParse::inverse_attribute*> attrs = declaration().as_entity()->all_inverse_attributes();`
`235`	`238`	`std::vector<const IfcParse::inverse_attribute*>::const_iterator iter = attrs.begin();`
`236`	`239`	`for (; iter != attrs.end(); ++iter) {`