----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3591/#review12185 -----------------------------------------------------------
/trunk/res/res_config_pgsql.c <https://reviewboard.asterisk.org/r/3591/#comment22240> Since ast_str_create can fail, you need to check if the allocation succeeded before using it: if (!conn_info) { /* bail */ } - Matt Jordan On June 10, 2014, 2:24 a.m., doome wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3591/ > ----------------------------------------------------------- > > (Updated June 10, 2014, 2:24 a.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-23737 > https://issues.asterisk.org/jira/browse/ASTERISK-23737 > > > Repository: Asterisk > > > Description > ------- > > See https://issues.asterisk.org/jira/browse/ASTERISK-23737 > > > Diffs > ----- > > /trunk/res/res_config_pgsql.c 415300 > /trunk/configs/res_pgsql.conf.sample 415300 > /trunk/configs/cel_pgsql.conf.sample 415300 > /trunk/configs/cdr_pgsql.conf.sample 415300 > /trunk/cel/cel_pgsql.c 415300 > /trunk/cdr/cdr_pgsql.c 415300 > > Diff: https://reviewboard.asterisk.org/r/3591/diff/ > > > Testing > ------- > > Used asterisk with the config paramterer pgdbappname and dbappname in > cdr_pgsql.conf and res_config_pgsql.conf respectively, enabled and disabled. > Worked as expected. > > > Thanks, > > doome > >
-- _____________________________________________________________________ -- 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
