Index: cdr.c =================================================================== --- cdr.c (revision 64654) +++ cdr.c (working copy) @@ -528,6 +528,22 @@ } } +void ast_cdr_noanswer(struct ast_cdr *cdr) +{ + char *chan; + + while (cdr) { + chan = !ast_strlen_zero(cdr->channel) ? cdr->channel : ""; + 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)) { + if (cdr->disposition < AST_CDR_NOANSWER) + cdr->disposition = AST_CDR_NOANSWER; + } + cdr = cdr->next; + } +} + int ast_cdr_disposition(struct ast_cdr *cdr, int cause) { int res = 0; @@ -638,7 +654,7 @@ ast_copy_string(cdr->clid, tmp, sizeof(cdr->clid)); ast_copy_string(cdr->src, num ? num : "", sizeof(cdr->src)); - cdr->disposition = (c->_state == AST_STATE_UP) ? AST_CDR_ANSWERED : AST_CDR_NOANSWER; + cdr->disposition = (c->_state == AST_STATE_UP) ? AST_CDR_ANSWERED : AST_CDR_NULL; cdr->amaflags = c->amaflags ? c->amaflags : ast_default_amaflags; ast_copy_string(cdr->accountcode, c->accountcode, sizeof(cdr->accountcode)); /* Destination information */ @@ -671,6 +687,8 @@ char *ast_cdr_disp2str(int disposition) { switch (disposition) { + case AST_CDR_NULL: + return "NO ANSWER"; /* by default, for backward compatibility */ case AST_CDR_NOANSWER: return "NO ANSWER"; case AST_CDR_FAILED: @@ -862,7 +880,7 @@ cdr->billsec = 0; cdr->duration = 0; ast_cdr_start(cdr); - cdr->disposition = AST_CDR_NOANSWER; + cdr->disposition = AST_CDR_NULL; } cdr = cdr->next; Index: apps/app_dial.c =================================================================== --- apps/app_dial.c (revision 64654) +++ apps/app_dial.c (working copy) @@ -678,6 +678,7 @@ if (!f || ((f->frametype == AST_FRAME_CONTROL) && (f->subclass == AST_CONTROL_HANGUP))) { /* Got hung up */ *to=-1; + ast_cdr_noanswer(in->cdr); strcpy(status, "CANCEL"); if (f) ast_frfree(f); @@ -691,6 +692,7 @@ if (option_verbose > 3) ast_verbose(VERBOSE_PREFIX_3 "User hit %c to disconnect call.\n", f->subclass); *to=0; + ast_cdr_noanswer(in->cdr); *result = f->subclass; strcpy(status, "CANCEL"); ast_frfree(f); @@ -703,6 +705,7 @@ if (option_verbose > 3) ast_verbose(VERBOSE_PREFIX_3 "User hit %c to disconnect call.\n", f->subclass); *to=0; + ast_cdr_noanswer(in->cdr); strcpy(status, "CANCEL"); ast_frfree(f); return NULL; @@ -727,6 +730,10 @@ } if (!*to && (option_verbose > 2)) ast_verbose( VERBOSE_PREFIX_3 "Nobody picked up in %d ms\n", orig); + if (!*to || ast_check_hangup(in)) { + ast_cdr_noanswer(in->cdr); + } + } return peer; Index: include/asterisk/cdr.h =================================================================== --- include/asterisk/cdr.h (revision 64654) +++ include/asterisk/cdr.h (working copy) @@ -30,9 +30,10 @@ #define AST_CDR_FLAG_CHILD (1 << 3) #define AST_CDR_FLAG_POST_DISABLED (1 << 4) -#define AST_CDR_NOANSWER (1 << 0) +#define AST_CDR_NULL 0 +#define AST_CDR_FAILED (1 << 0) #define AST_CDR_BUSY (1 << 1) -#define AST_CDR_FAILED (1 << 2) +#define AST_CDR_NOANSWER (1 << 2) #define AST_CDR_ANSWERED (1 << 3) /*! AMA Flags */