[Asterisk-code-review] Fix 'core show channel' CDR variable printing. (asterisk[13])

[snuffy]

snuffy has uploaded a new change for review.

  https://gerrit.asterisk.org/478

Change subject: Fix 'core show channel' CDR variable printing.
......................................................................

Fix 'core show channel' CDR variable printing.

When the new Bridging API was implemented, workspace variable
changed to a malloc'd string, causing sizeof() to always be 8 (char).

Revert back to static string for workspace.

ASTERISK-25090 #close

Change-Id: I51e610ae87371df771ce7693a955510efb90f8f7
---
M main/cdr.c
1 file changed, 1 insertion(+), 5 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/78/478/1

diff --git a/main/cdr.c b/main/cdr.c
index 5e24dae..96d055c 100644
--- a/main/cdr.c
+++ b/main/cdr.c
@@ -3096,12 +3096,8 @@
 	struct cdr_object *it_cdr;
 	struct ast_var_t *variable;
 	const char *var;
-	RAII_VAR(char *, workspace, ast_malloc(256), ast_free);
+	char workspace[256];
 	int total = 0, x = 0, i;
-
-	if (!workspace) {
-		return 0;
-	}
 
 	if (!cdr) {
 		RAII_VAR(struct module_config *, mod_cfg,

-- 
To view, visit https://gerrit.asterisk.org/478
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I51e610ae87371df771ce7693a955510efb90f8f7
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: snuffy <snuffy22 at gmail.com>