[asterisk-commits] branch 1.2 r14234 - in /branches/1.2: cdr.c include/asterisk/cdr.h

[asterisk-commits at lists.digium.com]

Author: file
Date: Wed Mar 22 15:38:32 2006
New Revision: 14234

URL: http://svn.digium.com/view/asterisk?rev=14234&view=rev
Log:
Issue #5918 - Disposition showing FAILED even though call is answered successfully (Reported by tracinet)

Modified:
    branches/1.2/cdr.c
    branches/1.2/include/asterisk/cdr.h

Modified: branches/1.2/cdr.c
URL: http://svn.digium.com/view/asterisk/branches/1.2/cdr.c?rev=14234&r1=14233&r2=14234&view=diff
==============================================================================
--- branches/1.2/cdr.c (original)
+++ branches/1.2/cdr.c Wed Mar 22 15:38:32 2006
@@ -520,8 +520,10 @@
 		chan = !ast_strlen_zero(cdr->channel) ? cdr->channel : "<unknown>";
 		if (ast_test_flag(cdr, AST_CDR_FLAG_POSTED))
 			ast_log(LOG_WARNING, "CDR on channel '%s' already posted\n", chan);
-		if (!ast_test_flag(cdr, AST_CDR_FLAG_LOCKED))
-			cdr->disposition = AST_CDR_FAILED;
+		if (!ast_test_flag(cdr, AST_CDR_FLAG_LOCKED)) {
+			if (cdr->disposition < AST_CDR_FAILED)
+				cdr->disposition = AST_CDR_FAILED;
+		}
 		cdr = cdr->next;
 	}
 }

Modified: branches/1.2/include/asterisk/cdr.h
URL: http://svn.digium.com/view/asterisk/branches/1.2/include/asterisk/cdr.h?rev=14234&r1=14233&r2=14234&view=diff
==============================================================================
--- branches/1.2/include/asterisk/cdr.h (original)
+++ branches/1.2/include/asterisk/cdr.h Wed Mar 22 15:38:32 2006
@@ -32,8 +32,8 @@
 
 #define AST_CDR_NOANSWER			(1 << 0)
 #define AST_CDR_BUSY				(1 << 1)
-#define AST_CDR_ANSWERED			(1 << 2)
-#define AST_CDR_FAILED				(1 << 3)
+#define AST_CDR_FAILED				(1 << 2)
+#define AST_CDR_ANSWERED			(1 << 3)
 
 /*! AMA Flags */
 #define AST_CDR_OMIT				(1)