Merged revisions 260434 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r260434 | jpeeler | 2010-04-30 17:22:46 -0500 (Fri, 30 Apr 2010) | 11 lines Ensure channel state is not incorrectly set in the case of a very early answer. The needringing bit was being read in dahdi_read after answering thereby setting the state to ringing from up. This clears needringing upon answering so that is no longer possible. (closes issue #17067) Reported by: tzafrir Patches: needringing.diff uploaded by tzafrir (license 46) ........ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@260437 f38db490-d61c-443f-a65b-d21fe96a405b
This commit is contained in:
parent
57427ca327
commit
762557326c
|
@ -2625,6 +2625,12 @@ static int my_off_hook(void *pvt)
|
|||
return dahdi_set_hook(p->subs[SUB_REAL].dfd, DAHDI_OFFHOOK);
|
||||
}
|
||||
|
||||
static void my_set_needringing(void *pvt, int value)
|
||||
{
|
||||
struct dahdi_pvt *p = pvt;
|
||||
p->subs[SUB_REAL].needringing = value;
|
||||
}
|
||||
|
||||
static int my_start(void *pvt)
|
||||
{
|
||||
struct dahdi_pvt *p = pvt;
|
||||
|
@ -3129,6 +3135,7 @@ static struct analog_callback dahdi_analog_callbacks =
|
|||
.confmute = my_confmute,
|
||||
.set_pulsedial = my_set_pulsedial,
|
||||
.get_orig_dialstring = my_get_orig_dialstring,
|
||||
.set_needringing = my_set_needringing,
|
||||
};
|
||||
|
||||
static struct dahdi_pvt *round_robin[32];
|
||||
|
@ -7599,6 +7606,7 @@ static struct ast_frame *dahdi_handle_event(struct ast_channel *ast)
|
|||
p->subs[idx].f.frametype = AST_FRAME_CONTROL;
|
||||
p->subs[idx].f.subclass.integer = AST_CONTROL_ANSWER;
|
||||
/* Make sure it stops ringing */
|
||||
p->subs[SUB_REAL].needringing = 0;
|
||||
dahdi_set_hook(p->subs[idx].dfd, DAHDI_OFFHOOK);
|
||||
ast_debug(1, "channel %d answered\n", p->channel);
|
||||
if (p->cidspill) {
|
||||
|
|
|
@ -519,6 +519,13 @@ static int analog_off_hook(struct analog_pvt *p)
|
|||
return -1;
|
||||
}
|
||||
|
||||
static void analog_set_needringing(struct analog_pvt *p, int value)
|
||||
{
|
||||
if (p->calls->set_needringing) {
|
||||
return p->calls->set_needringing(p->chan_pvt, value);
|
||||
}
|
||||
}
|
||||
|
||||
static int analog_dsp_set_digitmode(struct analog_pvt *p, enum analog_dsp_digitmode mode)
|
||||
{
|
||||
if (p->calls->dsp_set_digitmode) {
|
||||
|
@ -2662,6 +2669,7 @@ static struct ast_frame *__analog_handle_event(struct analog_pvt *p, struct ast_
|
|||
p->subs[index].f.frametype = AST_FRAME_CONTROL;
|
||||
p->subs[index].f.subclass.integer = AST_CONTROL_ANSWER;
|
||||
/* Make sure it stops ringing */
|
||||
analog_set_needringing(p, 0);
|
||||
analog_off_hook(p);
|
||||
ast_debug(1, "channel %d answered\n", p->channel);
|
||||
analog_cancel_cidspill(p);
|
||||
|
|
|
@ -144,6 +144,7 @@ struct analog_callback {
|
|||
int (* const on_hook)(void *pvt);
|
||||
/*! \brief Set channel off hook */
|
||||
int (* const off_hook)(void *pvt);
|
||||
void (* const set_needringing)(void *pvt, int value);
|
||||
/* We're assuming that we're going to only wink on ANALOG_SUB_REAL - even though in the code there's an argument to the index
|
||||
* function */
|
||||
int (* const wink)(void *pvt, enum analog_sub sub);
|
||||
|
|
Reference in New Issue