Author: coreyfarrell Date: Tue Nov 4 13:33:21 2014 New Revision: 427202 URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=427202 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/ ........ Merged revisions 427200 from http://svn.asterisk.org/svn/asterisk/branches/12 ........ Merged revisions 427201 from http://svn.asterisk.org/svn/asterisk/branches/13 Modified: trunk/ (props changed) trunk/res/res_http_websocket.c Propchange: trunk/ ------------------------------------------------------------------------------ Binary property 'branch-13-merged' - no diff available. Modified: trunk/res/res_http_websocket.c URL: http://svnview.digium.com/svn/asterisk/trunk/res/res_http_websocket.c?view=diff&rev=427202&r1=427201&r2=427202 ============================================================================== --- trunk/res/res_http_websocket.c (original) +++ trunk/res/res_http_websocket.c Tue Nov 4 13:33:21 2014 @@ -1292,7 +1292,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
