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

#257. Correct parsing of a splitted request #273

Merged
merged 4 commits into from
Sep 22, 2015
Merged

#257. Correct parsing of a splitted request #273

merged 4 commits into from
Sep 22, 2015

Conversation

avbelov23
Copy link
Contributor

No description provided.

*
* @len could be 0 if the header was fully read, but we realized this only
* now by facinng CRLF at begin of current data chunk.
*/
void
tfw_http_msg_hdr_chunk_fixup(TfwHttpMsg *hm, char *data, int len)
tfw_http_msg_field_chunk_fixup(TfwHttpMsg *hm, TfwStr *field, char *data, int len)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Too long line

@@ -177,39 +177,49 @@ tfw_http_msg_hdr_open(TfwHttpMsg *hm, unsigned char *hdr_start)
void
tfw_http_msg_hdr_chunk_fixup(TfwHttpMsg *hm, char *data, int len)
{
TfwStr *hdr = &hm->parser.hdr;
tfw_http_msg_field_chunk_fixup(hm, &hm->parser.hdr, data, len);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Usually in C we place calling function under called function.

@krizhanovsky
Copy link
Contributor

Good to merge

@avbelov23 avbelov23 merged commit 59d245d into master Sep 22, 2015
avbelov23 added a commit that referenced this pull request Sep 22, 2015
@avbelov23 avbelov23 deleted the avb-257 branch September 22, 2015 17:32
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.

2 participants