Add check for nan in Nurbs. Add check for IfcTokenType::LINE_END in IfcLoader to prevent near infinite loop

Author: ifcapps

src/cpp/web-ifc/geometry/nurbs.cpp

@@ -178,6 +178,11 @@ namespace webifc::geometry{
 		this->pth = tinynurbs::surfacePoint(*this->nurbs, 1.0, 0.0);
 		this->ptv = tinynurbs::surfacePoint(*this->nurbs, 0.0, 1.0);
 
+        if (std::isnan(pth.x) || std::isnan(pth.y) ||std::isnan(pth.z) ) {
+			spdlog::error("pth isnan: ({}/{}/{})", pth.x, pth.y, pth.z);
+			return;
+		}
+
 		// Compute distances for further use.
 		this->dh = glm::distance(ptc, pth);
 		this->dv = glm::distance(ptc, ptv);
@@ -313,6 +318,12 @@ namespace webifc::geometry{
 						double rads = (i / rotations) * pi2;
 						double incU = glm::sin(rads) / mul_divisor;
 						double incV = glm::cos(rads) / mul_divisor;
+
+                        if (std::isnan(pr)) {
+                            spdlog::error("Invalid pr {} ",pr);
+                            break;
+                        }
+
 						if (pr > 1) incV *= pr;
 						else incU /= pr;
 						while (true)

src/cpp/web-ifc/parsing/IfcLoader.cpp

@@ -507,6 +507,11 @@ namespace webifc::parsing {
        {
          depth--;
        }
+       else if (t == IfcTokenType::LINE_END)
+       {
+           spdlog::error("[GetSetArgument[{}]) unexpected LINE_END token", GetCurrentLineExpressID());
+           break;
+       }
        else
        {
          tapeOffsets.push_back(offset);
@@ -522,7 +527,7 @@ namespace webifc::parsing {
          }
          else
          {
-           spdlog::error("[GetSetArgument[]) unexpected token", GetCurrentLineExpressID());
+           spdlog::error("[GetSetArgument[{}]) unexpected token", GetCurrentLineExpressID());
          }
        }