[asterisk-bugs] [Asterisk 0017630]: Chanspy Keeps using G729 Encoder licenses even after the spying channel hangs up.

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17630 
====================================================================== 
Reported By:                manvirr
Assigned To:                dvossel
====================================================================== 
Project:                    Asterisk
Issue ID:                   17630
Category:                   Applications/app_chanspy
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Target Version:             1.6.2.12
Asterisk Version:           SVN 
JIRA:                       SWP-1845 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-07-12 09:02 CDT
Last Modified:              2010-07-27 15:54 CDT
====================================================================== 
Summary:                    Chanspy Keeps using G729 Encoder licenses even after
the spying channel hangs up.
Description: 
Hi,

We are using Asterisk 1.6.2.9 and running on Cent OS 5.4 x86_64 kernel
2.6.18-194.3.1.el5

We are using G729 for our sip trunks. Our extensions use G729 as well.
When I try to spy on a channel with chanspy, I observe that the G729
licenses keep getting used up but do not get freed up even when I hang up
the Spy / Spying Extension.

Is there any way we can free up the licenses when the spying of a channel
is over?

When I do g729 show licenses I get the following output.. And with every
channel that is spied on, the encoder count keeps increasing but with every
call that is hung up, the encoder count does not decrease.

2/0 encoders/decoders of 16 licensed channels are currently in use

Licenses Found:
File: G729-REMOVED.lic -- Key: G729-REMOVED -- Host-ID: REMOVED --
Channels: 11 (Expires: 2030-07-01) (OK)
File: G729-REMOVED.lic -- Key: G729-REMOVED -- Host-ID: REMOVED --
Channels: 5 (Expires: 2030-06-28) (OK)
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0013513 Recording transcoded call with Monitor(...
====================================================================== 

---------------------------------------------------------------------- 
 (0125146) svnbot (reporter) - 2010-07-27 15:54
 https://issues.asterisk.org/view.php?id=17630#c125146 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 279946

_U  branches/1.6.2/
U   branches/1.6.2/include/asterisk/audiohook.h
U   branches/1.6.2/main/audiohook.c
U   branches/1.6.2/main/channel.c

------------------------------------------------------------------------
r279946 | dvossel | 2010-07-27 15:54:30 -0500 (Tue, 27 Jul 2010) | 24
lines

Merged revisions 279945 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r279945 | dvossel | 2010-07-27 15:33:40 -0500 (Tue, 27 Jul 2010) | 19
lines
  
  remove empty audiohook write list on channel
  
  If a channel has an audiohook write list created on it, that
  list stays on the channel until the channel is destroyed.  There
  is no reason to keep that list on the channel if it becomes empty.
  If it is empty that just means we are doing needless translating
  for every ast_read and ast_write.  This patch removes the audiohook
  list from the channel once it is detected to be empty on either a
  read or write.  If a audiohook is added back to the channel after
  this list is destroyed, the list just gets recreated as if it never
  existed to begin with.
  
  (closes issue https://issues.asterisk.org/view.php?id=17630)
  Reported by: manvirr
  
  Review: https://reviewboard.asterisk.org/r/799/
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=279946 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-07-27 15:54 svnbot         Checkin                                      
2010-07-27 15:54 svnbot         Note Added: 0125146                          
======================================================================