From 73f1f6cda63e33f965bec16e6f4bc667eed6eced Mon Sep 17 00:00:00 2001 From: mnicholson Date: Thu, 16 Sep 2010 19:53:41 +0000 Subject: Merged revisions 287114 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r287114 | mnicholson | 2010-09-16 14:52:39 -0500 (Thu, 16 Sep 2010) | 8 lines Don't stop printing cdr variables if we encounter one with a blank name or value. (closes issue #17900) Reported by: under Patches: core-show-channel-cdr-fix1.diff uploaded by mnicholson (license 96) Tested by: mnicholson ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@287115 f38db490-d61c-443f-a65b-d21fe96a405b --- main/cdr.c | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/main/cdr.c b/main/cdr.c index a7a096a35..6e92f6414 100644 --- a/main/cdr.c +++ b/main/cdr.c @@ -370,7 +370,7 @@ int ast_cdr_copy_vars(struct ast_cdr *to_cdr, struct ast_cdr *from_cdr) int ast_cdr_serialize_variables(struct ast_cdr *cdr, struct ast_str **buf, char delim, char sep, int recur) { struct ast_var_t *variables; - const char *var, *val; + const char *var; char *tmp; char workspace[256]; int total = 0, x = 0, i; @@ -382,16 +382,16 @@ int ast_cdr_serialize_variables(struct ast_cdr *cdr, struct ast_str **buf, char ast_str_append(buf, 0, "\n"); AST_LIST_TRAVERSE(&cdr->varshead, variables, entries) { - if (variables && - (var = ast_var_name(variables)) && (val = ast_var_value(variables)) && - !ast_strlen_zero(var) && !ast_strlen_zero(val)) { - if (ast_str_append(buf, 0, "level %d: %s%c%s%c", x, var, delim, val, sep) < 0) { - ast_log(LOG_ERROR, "Data Buffer Size Exceeded!\n"); - break; - } else - total++; - } else + if (!(var = ast_var_name(variables))) { + continue; + } + + if (ast_str_append(buf, 0, "level %d: %s%c%s%c", x, var, delim, S_OR(ast_var_value(variables), ""), sep) < 0) { + ast_log(LOG_ERROR, "Data Buffer Size Exceeded!\n"); break; + } + + total++; } for (i = 0; cdr_readonly_vars[i]; i++) { -- cgit v1.2.1