Author: coreyfarrell Date: Tue Nov 4 13:30:43 2014 New Revision: 427200 URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=427200 Log: res_http_websockets: Fix extra unref of module
In websocket_add_protocol_internal is used to add the "echo" protocol, but ast_websocket_remove_protocol is used to remove it. This causes an extra call to ast_module_unref. ASTERISK-24480 #close Reported by: Corey Farrell Review: https://reviewboard.asterisk.org/r/4140/ Modified: branches/12/res/res_http_websocket.c Modified: branches/12/res/res_http_websocket.c URL: http://svnview.digium.com/svn/asterisk/branches/12/res/res_http_websocket.c?view=diff&rev=427200&r1=427199&r2=427200 ============================================================================== --- branches/12/res/res_http_websocket.c (original) +++ branches/12/res/res_http_websocket.c Tue Nov 4 13:30:43 2014 @@ -886,7 +886,7 @@ static int unload_module(void) { - ast_websocket_remove_protocol("echo", websocket_echo_callback); + websocket_remove_protocol_internal("echo", websocket_echo_callback); ast_http_uri_unlink(&websocketuri); ao2_ref(websocketuri.data, -1); websocketuri.data = NULL; -- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- svn-commits mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/svn-commits
