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

Fix segfaults in ALL_IN_ONE mode #164

wants to merge 1 commit into
base: master


None yet
1 participant
Copy link

Azq2 commented Jan 9, 2019

Test data:

'<div>ewfewfweffefwe</div>' x 10000


myhtml = myhtml_create();
myhtml_init(myhtml, MyHTML_OPTIONS_PARSE_MODE_ALL_IN_ONE, 1, 0);

myhtml_tree_t *tree = myhtml_tree_create();
myhtml_tree_init(tree, myhtml);
myhtml_parse(tree, MyENCODING_DEFAULT, html_str, html_length);

This comment has been minimized.

Copy link

Azq2 commented Jan 9, 2019

from <= mythread->entries_length

Its strange. I faster check code and i think it like a bug.

I replace to
from < mythread->entries_length

May be it ok for multithread/single mode... but after this PR nothing breaks.

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