[asterisk-bugs] [Asterisk 0014554]: [patch] # for fastforward goes beyond end of message

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14554 
====================================================================== 
Reported By:                lacoursj
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   14554
Category:                   Applications/app_voicemail
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     confirmed
Asterisk Version:           1.4.23 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-02-25 10:26 CST
Last Modified:              2009-03-09 17:15 CDT
====================================================================== 
Summary:                    [patch] # for fastforward goes beyond end of message
Description: 
When listening to message playback # and * will fastforward or rewind the
playback by 3000ms by default.  When rewinding checks are made to make sure
you cannot seek before the beginning of the stream, but no checks are made
to make sure you do not seek past the end of the stream when using '#'. 
The result is that users will seek past the end of the stream which lands
them in the next IVR menu, where '#' will exit the voicemail application. 
If a user is trying to fastforward to the end of a message he may hit '#'
too many times and find himself disconnected.
====================================================================== 

---------------------------------------------------------------------- 
 (0101430) lacoursj (reporter) - 2009-03-09 17:15
 http://bugs.digium.com/view.php?id=14554#c101430 
---------------------------------------------------------------------- 
Added.  I had not read that doc!  Excellent guidelines for this or any
project for that matter.  I noted that the snippet of code following mine
has "... else if..." - is "if" exempted from the guidelines for
readability? 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-03-09 17:15 lacoursj       Note Added: 0101430                          
======================================================================