mirror of
https://github.com/asterisk/asterisk.git
synced 2025-11-11 04:18:21 +00:00
res_http_websocket.c: Fix some off nominal path cleanup.
* Remove extraneous unlock on off-nominal path. * Add missing HTTP error reply. Change-Id: I1f402bfe448fba8696b507477cab5f060ccd9b2b
This commit is contained in:
@@ -207,7 +207,6 @@ int AST_OPTIONAL_API_NAME(ast_websocket_server_add_protocol)(struct ast_websocke
|
||||
|
||||
protocol = ast_websocket_sub_protocol_alloc(name);
|
||||
if (!protocol) {
|
||||
ao2_unlock(server->protocols);
|
||||
return -1;
|
||||
}
|
||||
protocol->session_established = callback;
|
||||
@@ -824,6 +823,7 @@ int AST_OPTIONAL_API_NAME(ast_websocket_uri_cb)(struct ast_tcptls_session_instan
|
||||
&& protocol_handler->session_attempted(ser, get_vars, headers)) {
|
||||
ast_debug(3, "WebSocket connection from '%s' rejected by protocol handler '%s'\n",
|
||||
ast_sockaddr_stringify(&ser->remote_address), protocol_handler->name);
|
||||
websocket_bad_request(ser);
|
||||
ao2_ref(protocol_handler, -1);
|
||||
return 0;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user