> On Dec. 10, 2013, 4:10 a.m., Matt Jordan wrote: > > branches/12/main/http.c, lines 630-632 > > <https://reviewboard.asterisk.org/r/3052/diff/1/?file=49131#file49131line630> > > > > Check for memory allocation failure of type here. > > > > What happens here if v->value is NULL?
Matt: I don't think v->value could ever be NULL here. - rmudgett ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3052/#review10346 ----------------------------------------------------------- On Dec. 10, 2013, 4:51 p.m., Kevin Harwell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3052/ > ----------------------------------------------------------- > > (Updated Dec. 10, 2013, 4:51 p.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-22872 > https://issues.asterisk.org/jira/browse/ASTERISK-22872 > > > Repository: Asterisk > > > Description > ------- > > Added the ability to specify channel variables when creating/originating a > channel in ARI. The variables are sent in the body of the request and should > be formatted as a single level JSON object. For example: {"variable1": > "foo", "variable2": "bar"}. No nested objects allowed. > > > Diffs > ----- > > branches/12/rest-api/api-docs/channels.json 403413 > branches/12/res/res_ari_channels.c 403413 > branches/12/res/ari/resource_channels.c 403413 > branches/12/res/ari/resource_channels.h 403413 > branches/12/main/http.c 403413 > > Diff: https://reviewboard.asterisk.org/r/3052/diff/ > > > Testing > ------- > > Some manual testing to make sure variables were being set. Also wrote a test > for the testsuite. > > > Thanks, > > Kevin Harwell > >
-- _____________________________________________________________________ -- 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