Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[OpenGEX] Crash when attempt load bundled test OpenGEX 3D model files #6057

Open
tellypresence opened this issue Mar 17, 2025 · 2 comments
Open
Labels
Bug Global flag to mark a deviation from expected behaviour OpenGEX BUgs related to the OpenGEX format

Comments

@tellypresence
Copy link
Contributor

tellypresence commented Mar 17, 2025

Getting crashes when attempting to load bundled OpenGEX models. Reason appears to be that when iterating in OpenGEXImporter::resolveReferences(), this line is failing:

    node->mMeshes[i] = meshIdx;

because i greater than or equal to value of node->mNumMeshes.

Can easily observe this problem by adding logging around that line in OpenGEXImporter::resolveReferences() e.g.

    if (i < node->mNumMeshes) {
        node->mMeshes[i] = meshIdx;
        // Log success...
    } else {
        // Log failure...
    }

Affected models:

Model
test/models/OpenGEX/animation_example.ogex
test/models/OpenGEX/camera.ogex
test/models/OpenGEX/collada.ogex
@tellypresence tellypresence added the Bug Global flag to mark a deviation from expected behaviour label Mar 17, 2025
@tellypresence tellypresence changed the title [OpenGEX] Errors loading bundled test 3D model files [OpenGEX] Crash when attempt load bundled test OpenGEX 3D model files Mar 18, 2025
@tellypresence
Copy link
Contributor Author

tellypresence commented Mar 27, 2025

Note that many errors were "masked" due to early non-catastrophic termination during load due to other factors

Assimp commitModelError
12 Jun 2019
a4907e5
test/models/OpenGEX/animation_example.ogexOpenGEX: Not enough values to fill 4-element color, only 3
test/models/OpenGEX/camera.ogexValidation failed: The mesh contains no vertices
test/models/OpenGEX/collada.ogexValidation failed: aiScene::mLights[3] has no corresponding node in the scene graph (light3)
15 Nov 2017
98da53e
test/models/OpenGEX/animation_example.ogexCrash on attempt load
test/models/OpenGEX/camera.ogexaiScene::mLights[1] has no corresponding node in the scene graph (light1)
test/models/OpenGEX/collada.ogexValidation failed: aiScene::mLights[3] has no corresponding node in the scene graph (light3)

@tellypresence
Copy link
Contributor Author

Please add "OpenGEX" label to this issue

@turol turol added the OpenGEX BUgs related to the OpenGEX format label Apr 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Global flag to mark a deviation from expected behaviour OpenGEX BUgs related to the OpenGEX format
Projects
None yet
Development

No branches or pull requests

2 participants