diff options
| author | Wieland Hagen <wieland.hagen@kdab.com> | 2017-09-22 12:21:09 +0200 |
|---|---|---|
| committer | Sean Harmer <sean.harmer@kdab.com> | 2017-09-22 10:29:41 +0000 |
| commit | ba9a38ceca15f9bc086a6c9c5d341001e9e73852 (patch) | |
| tree | bf8bc94d15772498491a974a14f51fae56213d9e /src | |
| parent | 75697d59cb6e3783bd1444816cfc90b4bc8b9aba (diff) | |
Don't crash objgeometryloader on ill-formed face, just ignore
For < 3 vertices, array accesses would fail, and face creation not
make sense anyways
Change-Id: I5eb746dfdec92d1c836e9870d87e7900530ec81c
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/plugins/geometryloaders/default/objgeometryloader.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/geometryloaders/default/objgeometryloader.cpp b/src/plugins/geometryloaders/default/objgeometryloader.cpp index b1fb1f931..7184e2f69 100644 --- a/src/plugins/geometryloaders/default/objgeometryloader.cpp +++ b/src/plugins/geometryloaders/default/objgeometryloader.cpp @@ -144,7 +144,7 @@ bool ObjGeometryLoader::doLoad(QIODevice *ioDev, const QString &subMesh) ++normalsOffset; } } - } else if (!skipping && qstrncmp(tokens.charPtrAt(0), "f ", 2) == 0) { + } else if (!skipping && tokens.size() >= 4 && qstrncmp(tokens.charPtrAt(0), "f ", 2) == 0) { // Process face ++faceCount; |