mirror of
https://github.com/asterisk/asterisk.git
synced 2025-10-04 12:12:48 +00:00
framehooks: Add callback for determining if a hook is consuming frames of a specific type.
In the past framehooks have had no capability to determine what frame types a hook is actually interested in consuming. This has meant that code has had to assume they want all frames, thus preventing native bridging. This change adds a callback which allows a framehook to be queried for whether it is consuming a frame of a specific type. The native RTP bridging module has also been updated to take advantange of this, allowing native bridging to occur when previously it would not. ASTERISK-23497 #comment Reported by: Etienne Lessard ASTERISK-23497 #close Review: https://reviewboard.asterisk.org/r/3522/ git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/12@413681 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
@@ -160,6 +160,10 @@ int ast_framehook_attach(struct ast_channel *chan, struct ast_framehook_interfac
|
||||
ast_frfree(frame);
|
||||
}
|
||||
|
||||
if (ast_channel_is_bridged(chan)) {
|
||||
ast_softhangup_nolock(chan, AST_SOFTHANGUP_UNBRIDGE);
|
||||
}
|
||||
|
||||
return framehook->id;
|
||||
}
|
||||
|
||||
@@ -214,6 +218,12 @@ int ast_framehook_list_is_empty(struct ast_framehook_list *framehooks)
|
||||
}
|
||||
|
||||
int ast_framehook_list_contains_no_active(struct ast_framehook_list *framehooks)
|
||||
{
|
||||
return ast_framehook_list_contains_no_active_of_type(framehooks, 0);
|
||||
}
|
||||
|
||||
int ast_framehook_list_contains_no_active_of_type(struct ast_framehook_list *framehooks,
|
||||
enum ast_frame_type type)
|
||||
{
|
||||
struct ast_framehook *cur;
|
||||
|
||||
@@ -229,6 +239,9 @@ int ast_framehook_list_contains_no_active(struct ast_framehook_list *framehooks)
|
||||
if (cur->detach_and_destroy_me) {
|
||||
continue;
|
||||
}
|
||||
if (type && cur->i.consume_cb && !cur->i.consume_cb(cur->i.data, type)) {
|
||||
continue;
|
||||
}
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user