From: Tinet-mucw <mucw@ti-net.com.cn>
Date: Fri, 14 Nov 2025 02:01:28 +0000 (-0800)
Subject: core_unreal.c: Use ast instead of p->chan to get the DIALSTATUS variable
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=fe194b401658dc6a96d0e4ed872d051b207fbaf7;p=thirdparty%2Fasterisk.git

core_unreal.c: Use ast instead of p->chan to get the DIALSTATUS variable

After p->chan = NULL, ast still points to the valid channel object,
using ast safely accesses the channel's DIALSTATUS variable before it's fully destroyed

Resolves: #1590
---

diff --git a/main/core_unreal.c b/main/core_unreal.c
index bcae2405b7..fd0f05b4e3 100644
--- a/main/core_unreal.c
+++ b/main/core_unreal.c
@@ -1042,7 +1042,7 @@ int ast_unreal_hangup(struct ast_unreal_pvt *p, struct ast_channel *ast)
 		ast_clear_flag(p, AST_UNREAL_CARETAKER_THREAD);
 		p->chan = NULL;
 		if (p->owner) {
-			const char *status = pbx_builtin_getvar_helper(p->chan, "DIALSTATUS");
+			const char *status = pbx_builtin_getvar_helper(chan, "DIALSTATUS");
 
 			if (status) {
 				ast_channel_hangupcause_set(p->owner, cause);