We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
clixon/lib/src/clixon_json_parse.y
Line 301 in 3cd3f79
JSON parsing became quite slow, when request body contains a large size string. Since 'strlen' invokes every char for each.
int len = strlen($1); $$ = realloc($1, len+strlen($2) + 1); sprintf($$+len, "%s", $2);
int len = strlen($1);
$$ = realloc($1, len+strlen($2) + 1);
sprintf($$+len, "%s", $2);
'$1' orginal string may be damn large. eg. When post a large json param size exceed 10M by restconf protocol, then the process was stuck there.
The text was updated successfully, but these errors were encountered:
* JSON errors are now labelled with JSON and not XML
392e667
* Fixed: [Performance issue when parsing large JSON param](#266) * Moved strlen() from for end condition * Fixed debugging of xpath parser
Thanks for detecting, please verify
Sorry, something went wrong.
No branches or pull requests
clixon/lib/src/clixon_json_parse.y
Line 301 in 3cd3f79
JSON parsing became quite slow, when request body contains a large size string. Since 'strlen' invokes every char for each.
int len = strlen($1);
$$ = realloc($1, len+strlen($2) + 1);
sprintf($$+len, "%s", $2);
'$1' orginal string may be damn large. eg. When post a large json param size exceed 10M by restconf protocol, then the process was stuck there.
The text was updated successfully, but these errors were encountered: