----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3986/#review13270 -----------------------------------------------------------
Ship it! Ship It! - Joshua Colp On Sept. 10, 2014, 1:34 a.m., George Joseph wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3986/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2014, 1:34 a.m.) > > > Review request for Asterisk Developers. > > > Repository: Asterisk > > > Description > ------- > > ast_config_text_file_save() currently truncates include files as they are > processed. If a subsequent include file or the main config file has a > permissions error that prevents writing, earlier include files are left > truncated resulting in a frantic search for backups. > > This patch causes ast_config_text_file_save to check for write access on all > files before it truncates any of them. > > Will be applied 1.8 > trunk. > > > Diffs > ----- > > branches/1.8/main/config.c 422882 > > Diff: https://reviewboard.asterisk.org/r/3986/diff/ > > > Testing > ------- > > Tested normal access and verified that no files are truncated if any of the > can't be written to. > > > Thanks, > > George Joseph > >
-- _____________________________________________________________________ -- 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