[asterisk-bugs] [Asterisk 0012193]: PARKINGEXTEN misbehaves when it is not a canonical form number and can double park or destroy the 700 entry

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=12193 
====================================================================== 
Reported By:                davidw
Assigned To:                jpeeler
====================================================================== 
Project:                    Asterisk
Issue ID:                   12193
Category:                   Resources/res_features
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 101988 
Disclaimer on File?:        N/A 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             03-12-2008 06:53 CDT
Last Modified:              06-12-2008 18:02 CDT
====================================================================== 
Summary:                    PARKINGEXTEN misbehaves when it is not a canonical
form number and can double park or destroy the 700 entry
Description: 
If you set PARKINGEXTEN to "XXXX" it gets treated as 0.  (I think it would
also not be identiried as duplicate, if re-used, but I only have the
console at the moment.)

If you set it to "-1XXXX", it is handled correctly, with the exception
that it gets announced as minus one.

If you set it to "0700", it overwrites the system generated Park() line,
and removes it completely when the parking completes.

If you set it to "0701", it is treated as 701, however, I believe that it
would fail to detect the duplication and double park the call (I'd need two
phones, on the system I'm learning on, to demonstrate this other than by
code reading).

At some time, before the formal test run, I managed to get one with an
empty name created, which wouldn't go away, but I couldn't work out how to
reproduce that.

I will attach console traces, interleaved with the relevant extension.conf
lines.  Note that these traces also demonstrate a problem with returning
calls on timeout, which I intend to report separately.
====================================================================== 

---------------------------------------------------------------------- 
 svnbot - 06-12-08 18:02  
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 122434

_U  trunk/

------------------------------------------------------------------------
r122434 | jpeeler | 2008-06-12 18:02:47 -0500 (Thu, 12 Jun 2008) | 12
lines

Blocked revisions 122208 via svnmerge

........
r122208 | jpeeler | 2008-06-12 10:46:08 -0500 (Thu, 12 Jun 2008) | 5 lines

(closes issue http://bugs.digium.com/view.php?id=12193)
Reported by: davidw
Patch by: Corydon76, modified by me to work properly with ParkAndAnnounce
app


........

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

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

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
06-12-08 18:02  svnbot         Note Added: 0088653                          
======================================================================