CesiumGS / cdb-to-3dtiles

Convert CDB to 3D Tiles
Apache License 2.0
76 stars 28 forks source link

The converter crashes on exit when using static linking OpenSceneGraph #12

Closed baothientran closed 3 years ago

baothientran commented 3 years ago

The converter works correctly with GSModel and produces the correct tileset.json. However, it will seg fault when exiting the program. This only happens in the release build. The command line to produce the error:

./CDBConverter -i CDB_san_diego_Lite -o San-Diego

CDB_san_diego_Lite example only contains the GSFeature and GSModelGeometry in the Tiles directory. I've removed all other datasets for easier to debug

ErixenCruz commented 3 years ago

I am able to reproduce on the OSG static branch. I will build with debug option and go from there

ErixenCruz commented 3 years ago

Is there a way to validate CDB_san_diego_Lite? I want to rule out that we are feeding the converter bad data @baothientran

baothientran commented 3 years ago

@ErixenCruz currently there is no validator for CDB, but I can replicate the error with other sample as well

baothientran commented 3 years ago

For the static library seg fault, compiling with release with debug info allows me to see the seg fault in the OpenFlight loader destructor. It's still not better, but at least we know where to look next

Screenshot_20201104_092131

the command to compile with debug info is cmake ../cdb-to-3dtiles -DCMAKE_BUILD_TYPE=RelWithDebInfo

ErixenCruz commented 3 years ago

just going to put this here for reference https://groups.google.com/g/osg-users/c/_RN5Eld1G7I

lilleyse commented 3 years ago

Fixed in https://github.com/CesiumGS/cdb-to-3dtiles/commit/92e6383ed431627ea8042f2ac95cb88b78a62409