[asterisk-dev] [Code Review] IAX2 prune realtime fix

David Vossel dvossel at digium.com
Wed Feb 25 17:57:30 CST 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/176/
-----------------------------------------------------------

Review request for Asterisk Developers and Russell Bryant.


Summary
-------

Iax2 prune realtime had issues.  If "iax2 prune realtime all" was called, it would appear like the command was successful, but in reality nothing happened.  This is because the reload that was supposed to take place checks the config files, sees no changes, and does nothing.  If there had been a change in the the config file, the realtime users would have been marked for deletion and everything would have been fine.  Now prune_users() and prune_peers() are called instead of reload_config() to prune all users/peers that are realtime.  These functions remove all users/peers with the rtfriend and delme flags set. 

iax2_prune_realtime() also lacked the code to properly delete a single friend.  For example. if iax2 prune realtime <friend> was called, only the peer instance would be removed. The user would still remain.  


This addresses bug 0014479.
    http://bugs.digium.com/view.php?id=0014479


Diffs
-----

  /trunk/channels/chan_iax2.c 178639 

Diff: http://reviewboard.digium.com/r/176/diff


Testing
-------


Thanks,

David




More information about the asterisk-dev mailing list