[Home]

Summary:ASTERISK-07428: [patch] Erroneous rename event when blind transfering to parkext
Reporter:Havard Sorbo (hs)Labels:
Date Opened:2006-07-31 07:42:26Date Closed:2006-12-11 08:12:30.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/ManagerInterface
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) bug-7625.patch
( 1) log-combined-new.txt
Description:When using blind transfer to park a call I get a rename event from the manager api with the Oldname value set to a non-existing channel. (If it exist it hasn' been announced in a newchannel-event)

Event: Rename
Privilege: call,all
Oldname: SIPPeer/SIP/803-0817c610
Newname: SIP/803-0817c610
Uniqueid: 1154347088.26

****** ADDITIONAL INFORMATION ******

Attacing log containg dialup and parking.
Comments:By: Serge Vecher (serge-v) 2006-07-31 10:11:02

Can you please also attach a respective SIP debug. Please do the following:
1) Prepare test environment (reduce the ammount of unrelated traffic on the server);
2) Make sure your logger.conf has the following line:
  console => notice,warning,error,debug
3) restart Asterik.
4) Enable SIP transaction logging with the following CLI commands:
set debug 4
set verbose 4
sip debug
5) Save complete console log to file and _attach_ said file to the bug.
By: Havard Sorbo (hs) 2006-08-02 09:58:59

Uploaded new debug-log combined with asterisk manager log.
By: Serge Vecher (serge-v) 2006-08-02 10:01:33

as requested, please post the console capture -- debug file clips some important info out.
By: Havard Sorbo (hs) 2006-08-02 11:31:42

Logging from console posted.
Do you want me to file a bug on debug-to-file clipping important info? :P
By: Serge Vecher (serge-v) 2006-08-02 11:52:57

I guess the event in question is:
Event: Rename
Privilege: call,all
Oldname: SIPPeer/SIP/803-081a7998
Newname: SIP/803-081a7998
Uniqueid: 1154535826.8

Since it is preceded by the following event, Oldname: should really by SIPPeer/SIP/803-081a7998<MASQ>

Event: Rename
Privilege: call,all
Oldname: SIP/803-081a7998
Newname: SIP/803-081a7998<MASQ>
Uniqueid: 1154535822.5
By: Havard Sorbo (hs) 2006-08-02 12:24:42

the renaming is as follows:
SIPPeer/SIP/803-081a7998 -> SIP/803-081a7998
SIP/803-081a7998 -> SIP/803-081a7998<MASQ>

which would be perfectly valid if the channel/name "SIPPeer/SIP/803-081a7998" existed. If you parse the log you see that the first reference to "SIPPeer/SIP..." is in Oldname, which is a bit weird
By: Moises Silva (moy) 2006-09-04 11:12:49

serge-v: I see this situation very similar to the channel created by manager action "Redirect" with prefix name "AsyncGoto/" AND REPORTED with event Newchannel because the chan->_state is set with ast_setstate() before doing the masquerading thing with ast_channel_masquerade() and ast_do_masquerade(). I think the proper thing to do is report this new channel "SIPPeer/SIP/803-0817c610" with a Newchannel event, just as AsyncGoto does. I can make a patch quickly, but need to know if that solves your problem hs?

May be usefull bug references:
ASTERISK-7309
ASTERISK-7635

Regards
By: Moises Silva (moy) 2006-09-04 15:26:51

im uploading a fix for 1.2.11

disclaimer faxed several months ago.

regards
By: Serge Vecher (serge-v) 2006-09-27 13:39:26

hs: did you try moy's patch?
By: jmls (jmls) 2006-11-01 06:46:21.000-0600

ping: hs: did you try moy's patch?
By: Matt O'Gorman (mogorman) 2006-12-08 17:52:48.000-0600

hs if this is still an issue please reopen bug and comment about the patch.

mog