Don't fail on parse errors while resolving inverse attributes

aothms · aothms · commit 6ee283dbae2f · 2018-05-04T10:25:57.000+02:00
diff --git a/src/ifcparse/IfcParse.cpp b/src/ifcparse/IfcParse.cpp
@@ -1896,15 +1896,20 @@ IfcEntityList::ptr IfcFile::getInverse(int instance_id, IfcSchema::Type::Enum ty
 	for(IfcEntityList::it it = all->begin(); it != all->end(); ++it) {
 		bool valid = type == IfcSchema::Type::UNDEFINED || (*it)->is(type);
 		if (valid && attribute_index >= 0) {
-			Argument* arg = (*it)->entity->getArgument(attribute_index);
-			if (arg->type() == IfcUtil::Argument_ENTITY_INSTANCE) {
-				valid = instance == *arg;
-			} else if (arg->type() == IfcUtil::Argument_AGGREGATE_OF_ENTITY_INSTANCE) {
-				IfcEntityList::ptr li = *arg;
-				valid = li->contains(instance);
-			} else if (arg->type() == IfcUtil::Argument_AGGREGATE_OF_AGGREGATE_OF_ENTITY_INSTANCE) {
-				IfcEntityListList::ptr li = *arg;
-				valid = li->contains(instance);
+			try {
+				Argument* arg = (*it)->entity->getArgument(attribute_index);
+				if (arg->type() == IfcUtil::Argument_ENTITY_INSTANCE) {
+					valid = instance == *arg;
+				} else if (arg->type() == IfcUtil::Argument_AGGREGATE_OF_ENTITY_INSTANCE) {
+					IfcEntityList::ptr li = *arg;
+					valid = li->contains(instance);
+				} else if (arg->type() == IfcUtil::Argument_AGGREGATE_OF_AGGREGATE_OF_ENTITY_INSTANCE) {
+					IfcEntityListList::ptr li = *arg;
+					valid = li->contains(instance);
+				}
+			} catch (IfcException& e) {
+				valid = false;
+				Logger::Error(e);
 			}
 		}
 		if (valid) {
