On 14:55, Tue 22 Jan 08, Dan Austin wrote: > I have been trying to add call parking to chan_skinny, > and chasing odd segfaults related to playing the > parked extension number back to the caller. > > There have been a number of commits to res_features > that seemed to be related. Each commit worked with > the test for a masqueraded channel, which was the issue > I had with chan_skinny. > > The test for a masqueraded channel involve the variable > orig_chan_name, but that variable was explicitly set to > NULL in ast_park_call: > > 1. Channel/app calls ast_masq_park_call > 2. ast_masq_park_call sets up the masqueraged and > calls ast_park_call > 3. ast_park_call calls park_call_full with orig_chan_name > set to NULL, and does nothing else. > > I added patch to bug 11342 that has ast_masq_park_call call > park_call_full with orig_chan_name properly set. Now I > do not get a segfault when parking from chan_skinny. > > So, I am on drugs or was this a real bug?
Probably both ;) Seriously: I've been following that patch from the start, and I think you hit a real bug. I dont know why orig_chan_name is set to null, it looks wrong to me -- Michiel van Baak [EMAIL PROTECTED] http://michiel.vanbaak.eu GnuPG key: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x71C946BD "Why is it drug addicts and computer afficionados are both called users?" _______________________________________________ --Bandwidth and Colocation Provided by http://www.api-digital.com-- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
