> On Jan. 16, 2014, 11 a.m., Mark Michelson wrote: > > /branches/12/main/http.c, line 813 > > <https://reviewboard.asterisk.org/r/3125/diff/2/?file=52892#file52892line813> > > > > chunked_atoh shouldn't be necessary. You should be able to just use: > > > > sscanf(chunk_header, %x, &chunk_length); > > > > You would need to change chunk_length to be an unsigned int instead of > > an int, but that's fine since the length will never be negative.
I had considered that, but I wanted to be overly cautious and throw extra checking into the algorithm, such that any invalid character or unexpected sequence would result in an error, rather than partial input and confused results. - Scott ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3125/#review10603 ----------------------------------------------------------- On Jan. 13, 2014, 5:26 p.m., Scott Griepentrog wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3125/ > ----------------------------------------------------------- > > (Updated Jan. 13, 2014, 5:26 p.m.) > > > Review request for Asterisk Developers and Matt Jordan. > > > Bugs: ASTERISK-23068 > https://issues.asterisk.org/jira/browse/ASTERISK-23068 > > > Repository: Asterisk > > > Description > ------- > > Implements support for HTTP chunked mode Transfer-Encoding in JSON and Form > (post vars) body content. Relocated code from ast_http_get_json() and > ast_http_get_post_vars() that reads content from stream into new function > ast_http_get_contents(), and added support for reading and decoding chunked > mode transfers. > > > Diffs > ----- > > /branches/12/main/http.c 405282 > > Diff: https://reviewboard.asterisk.org/r/3125/diff/ > > > Testing > ------- > > Testsuite test rest_api/chunked_transfer added (see > https://reviewboard.asterisk.org/r/3126/) to insure correct operation of > chunked and regular mode. > > > Thanks, > > Scott Griepentrog > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
