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

crash in canRemoveVertex #508

Open
wenxiu2199 opened this issue Jan 11, 2022 · 2 comments
Open

crash in canRemoveVertex #508

wenxiu2199 opened this issue Jan 11, 2022 · 2 comments

Comments

@wenxiu2199
Copy link

see DetourTileCacheBuilder.cpp
in canRemoveVertex

// Find edges which share the removed vertex.
unsigned short edges[MAX_REM_EDGES];

I compare the func in RecastMesh.cpp
it seem to be edges[MAX_REM_EDGES * 3]

@SpangMcgraw
Copy link

Yup have hit this also, it seems edges should be declared as edges[MAX_REM_EDGES * 3]

@grahamboree
Copy link
Member

@SpangMcgraw Do you have a repro case I can use to test this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants