Summary:	ASTERISK-05758: L Option hangs up call with more time that specified.
Reporter:	Sebastian Nocetti (snocetti)	Labels:
Date Opened:	2005-12-02 06:49:23.000-0600	Date Closed:	2008-01-15 16:12:42.000-0600
Priority:	Minor	Regression?	No
Status:	Closed/Complete	Components:	Applications/app_dial
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:
Description:	Using L option, call is hunger UP after DURATION LIMIT is passed... L(120000:31000) that hungs up call, but when I get ANSWEREDTIME it says 121 SECS instead 120 secs-, warning file is played OK. L(720000:31000) that hungs up call, but when I get ANSWEREDTIME it says 730 SECS instead 720 SECS. I never heard WARNING MESSAGE :(. Actually I did not check S option, but I need L because the warning... it is for a callingcard, so because this error some cards have NEGATIVE credit.
Comments:	By: Sebastian Nocetti (snocetti) 2005-12-02 07:16:16.000-0600 I has been tried same with this: S(120) INSTEAD OF L(120000) when call is hunged up, then I got 120 secs for ANSWEREDTIME. So, problem seems to be only with L option. By: Matt O'Gorman (mogorman) 2006-01-05 17:03:28.000-0600 Eliminated rounding errors that caused call time limits to become inaccurate over time.  Also changed the time left option to round up so the time left sounds more accurate. Commit 7825 fixed in both 1.2 and trunk By: Digium Subversion (svnbot) 2008-01-15 16:11:56.000-0600 Repository: asterisk Revision: 7826 _U  trunk/ U   trunk/channel.c ------------------------------------------------------------------------ r7826 | kpfleming | 2008-01-15 16:11:56 -0600 (Tue, 15 Jan 2008) | 11 lines Merged revisions 7825 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.2 ........ r7825 | kpfleming | 2006-01-05 17:49:50 -0600 (Thu, 05 Jan 2006) | 3 lines eliminate rounding errors that caused call time limits to be inaccurate (issue ASTERISK-5758) round 'time left' reported during call limit warnings up to sound more accurate ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=7826 By: Digium Subversion (svnbot) 2008-01-15 16:11:56.000-0600 Repository: asterisk Revision: 7825 U   branches/1.2/channel.c ------------------------------------------------------------------------ r7825 | kpfleming | 2008-01-15 16:11:55 -0600 (Tue, 15 Jan 2008) | 3 lines eliminate rounding errors that caused call time limits to be inaccurate (issue ASTERISK-5758) round 'time left' reported during call limit warnings up to sound more accurate ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=7825 By: Digium Subversion (svnbot) 2008-01-15 16:12:42.000-0600 Repository: asterisk Revision: 7880 _U  team/oej/moduletest/ U   team/oej/moduletest/apps/app_chanspy.c U   team/oej/moduletest/apps/app_mixmonitor.c U   team/oej/moduletest/apps/app_privacy.c U   team/oej/moduletest/apps/app_stack.c U   team/oej/moduletest/apps/app_voicemail.c D   team/oej/moduletest/ast_expr.y U   team/oej/moduletest/ast_expr2.fl U   team/oej/moduletest/ast_expr2f.c U   team/oej/moduletest/channel.c U   team/oej/moduletest/channels/chan_agent.c U   team/oej/moduletest/channels/chan_iax2.c U   team/oej/moduletest/channels/chan_sip.c U   team/oej/moduletest/formats/format_pcm.c U   team/oej/moduletest/formats/format_pcm_alaw.c U   team/oej/moduletest/funcs/func_moh.c U   team/oej/moduletest/include/asterisk/chanspy.h U   team/oej/moduletest/include/asterisk/strings.h U   team/oej/moduletest/manager.c U   team/oej/moduletest/pbx/pbx_config.c U   team/oej/moduletest/pbx/pbx_spool.c U   team/oej/moduletest/res/res_features.c ------------------------------------------------------------------------ r7880 | oej | 2008-01-15 16:12:41 -0600 (Tue, 15 Jan 2008) | 150 lines Merged revisions 7677,7706,7709,7736,7738,7740,7743,7746,7771,7773,7792,7795,7799-7800,7803,7805,7807,7809,7812,7819,7823,7825,7827,7829,7831,7848,7870-7871 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.2 ........ r7677 | tilghman | 2005-12-30 15:54:19 +0100 (Fri, 30 Dec 2005) | 2 lines Bug 6091 - Fix race condition around uniqueid ........ r7706 | bweschke | 2006-01-02 03:04:14 +0100 (Mon, 02 Jan 2006) | 3 lines Fix compiler warnings. ........ r7709 | tilghman | 2006-01-02 08:31:54 +0100 (Mon, 02 Jan 2006) | 2 lines Bug 6084 - MixMonitor after a 'cli stop monitor' deadlocks ........ r7736 | kpfleming | 2006-01-03 17:34:12 +0100 (Tue, 03 Jan 2006) | 2 lines don't leak memory for (most) expression evaluations ........ r7738 | kpfleming | 2006-01-03 18:00:01 +0100 (Tue, 03 Jan 2006) | 2 lines backport rport scanning fix from trunk (bug ASTERISK-5913) ........ r7740 | kpfleming | 2006-01-03 18:24:56 +0100 (Tue, 03 Jan 2006) | 4 lines revert incorrect fix for bug ASTERISK-5890 from revision 7709 put in correct (simpler) fix add doxygen docs for channel spy 'state' values ........ r7743 | tilghman | 2006-01-03 19:15:12 +0100 (Tue, 03 Jan 2006) | 2 lines Bug 6121 - typo in application description ........ r7746 | kpfleming | 2006-01-03 21:22:18 +0100 (Tue, 03 Jan 2006) | 2 lines remove unused 'old' expression parser ........ r7771 | bweschke | 2006-01-04 06:27:38 +0100 (Wed, 04 Jan 2006) | 3 lines Fix the 'if' clause to be true under the right conditions. Bug ASTERISK-5969 ........ r7773 | russell | 2006-01-04 08:06:50 +0100 (Wed, 04 Jan 2006) | 2 lines use a more correct way of determining the size of the destination buffer ........ r7792 | oej | 2006-01-04 22:43:14 +0100 (Wed, 04 Jan 2006) | 2 lines Fixing typo in XML for video updates. ........ r7795 | oej | 2006-01-04 22:46:40 +0100 (Wed, 04 Jan 2006) | 2 lines Issue ASTERISK-5824: Removing extra CR+LF in manager events - needs port to trunk ........ r7799 | kpfleming | 2006-01-05 00:02:38 +0100 (Thu, 05 Jan 2006) | 2 lines make monitoring more tolerant of peers that deliver frames in bursts ........ r7800 | kpfleming | 2006-01-05 00:27:57 +0100 (Thu, 05 Jan 2006) | 2 lines ensure that ulaw/alaw sound files are filled with silence when extended (not zeroes) ........ r7803 | kpfleming | 2006-01-05 00:45:34 +0100 (Thu, 05 Jan 2006) | 2 lines return properly after extending file ........ r7805 | kpfleming | 2006-01-05 00:51:03 +0100 (Thu, 05 Jan 2006) | 2 lines use proper fwrite() parameters and return value ........ r7807 | kpfleming | 2006-01-05 01:18:46 +0100 (Thu, 05 Jan 2006) | 2 lines doh... fseek() has no useful return value ........ r7809 | kpfleming | 2006-01-05 01:58:29 +0100 (Thu, 05 Jan 2006) | 2 lines send device state updates for auto-logoff of agents as well ........ r7812 | oej | 2006-01-05 10:13:21 +0100 (Thu, 05 Jan 2006) | 2 lines Fix copyright of changed file ........ r7819 | kpfleming | 2006-01-05 21:52:38 +0100 (Thu, 05 Jan 2006) | 2 lines ensure that variable is initialized ........ r7823 | tilghman | 2006-01-06 00:07:08 +0100 (Fri, 06 Jan 2006) | 2 lines Bug 6081 - fix for memory leak, formatting fixes ........ r7825 | kpfleming | 2006-01-06 00:49:50 +0100 (Fri, 06 Jan 2006) | 3 lines eliminate rounding errors that caused call time limits to be inaccurate (issue ASTERISK-5758) round 'time left' reported during call limit warnings up to sound more accurate ........ r7827 | tilghman | 2006-01-06 00:53:07 +0100 (Fri, 06 Jan 2006) | 2 lines Bug 6076 - Fix documentation of ast_trim_blank return value ........ r7829 | kpfleming | 2006-01-06 01:21:00 +0100 (Fri, 06 Jan 2006) | 2 lines update agent persistence when an agent gets logged off by autologoff ........ r7831 | tilghman | 2006-01-06 01:34:40 +0100 (Fri, 06 Jan 2006) | 2 lines Dumb error messages - "Context 'context' already included in 'in' context" ........ r7848 | tilghman | 2006-01-07 08:27:18 +0100 (Sat, 07 Jan 2006) | 2 lines Bug 6156 - catch all threading errors, not just simple failure ........ r7870 | russell | 2006-01-09 05:52:16 +0100 (Mon, 09 Jan 2006) | 2 lines backport fix for unnecessary unlock (issue ASTERISK-6014) ........ r7871 | russell | 2006-01-09 06:11:44 +0100 (Mon, 09 Jan 2006) | 2 lines fix seg fault when using greek syntax in VoicemMailMain (issue ASTERISK-5878) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=7880