Skip to content

Comments

Fix bug in LineExtractor.cpp#9

Open
RayShark0605 wants to merge 1 commit intoHarborC:masterfrom
RayShark0605:patch-1
Open

Fix bug in LineExtractor.cpp#9
RayShark0605 wants to merge 1 commit intoHarborC:masterfrom
RayShark0605:patch-1

Conversation

@RayShark0605
Copy link

At line 55 of the original LineExtractor.cpp, if the IF STATEMENT expression here is FALSE, after skipping the IF STATEMENT, the INDEX in the following "_keylines.resize(index + 1);" statement is the length of _KEYLINES, At this time, if it is resized to INDEX+1 (that is, "length plus one"), the attribute in the added data will be a random value, which will lead to the following "lbd->compute(image, _keylines, descriptors); "statement (Calculate the descriptor of the feature line segment) Infinite loop or error!

At line 55 of the original LineExtractor.cpp, if the IF STATEMENT expression here is FALSE, after skipping the IF STATEMENT, the INDEX in the following "_keylines.resize(index + 1);" statement is the length of _KEYLINES, At this time, if it is resized to INDEX+1 (that is, "length plus one"), the attribute in the added data will be a random value, which will lead to the following "lbd->compute(image, _keylines, descriptors); "statement (Calculate the descriptor of the feature line segment) Infinite loop or error!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant