Re: [asterisk-dev] [Code Review] 3144: Protect manager data structures during shutdown.

2014-01-24 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3144/ --- (Updated Jan. 24, 2014, 12:29 p.m.) Status -- This change has been ma

Re: [asterisk-dev] [Code Review] 3144: Protect manager data structures during shutdown.

2014-01-24 Thread Mark Michelson
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3144/#review10682 --- Ship it! Ship It! - Mark Michelson On Jan. 21, 2014, 5:34 p

Re: [asterisk-dev] [Code Review] 3144: Protect manager data structures during shutdown.

2014-01-23 Thread Guenther Kelleter
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3144/#review10662 --- Ship it! Ship It! - Guenther Kelleter On Jan. 21, 2014, 6:3

Re: [asterisk-dev] [Code Review] 3144: Protect manager data structures during shutdown.

2014-01-21 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3144/ --- (Updated Jan. 21, 2014, 11:34 a.m.) Review request for Asterisk Developers

Re: [asterisk-dev] [Code Review] 3144: Protect manager data structures during shutdown.

2014-01-21 Thread rmudgett
> On Jan. 21, 2014, 3:18 a.m., Guenther Kelleter wrote: > > /branches/11/main/manager.c, lines 5628-5633 > > > > > > This replaces session_destroy(...) with unref_mansession(...) so > > session is not removed from

Re: [asterisk-dev] [Code Review] 3144: Protect manager data structures during shutdown.

2014-01-21 Thread Guenther Kelleter
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3144/#review10646 --- /branches/11/main/manager.c

[asterisk-dev] [Code Review] 3144: Protect manager data structures during shutdown.

2014-01-20 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3144/ --- Review request for Asterisk Developers. Bugs: AST-1242 https://issues.