[Asterisk-code-review] cel/cel radius: Fix wrong pointer. (asterisk[master])
Aaron An
asteriskteam at digium.com
Mon Jan 4 05:37:31 CST 2016
Aaron An has uploaded a new change for review.
https://gerrit.asterisk.org/1901
Change subject: cel/cel_radius: Fix wrong pointer.
......................................................................
cel/cel_radius: Fix wrong pointer.
The macro ADD_VENDOR_CODE defined in the cel_radius.c should use the parameter
y not the address of y.
The radius udp packet av pairs it not corret, then review the source
code and compare it with cdr/cdr_radius.c i find the bug, after fix it work
ASTERISK-25647 #close
Reported by: Aaron An
Tested by: Aaron An
Change-Id: I72889bccd8fde120d47aa659edc0e7e6d4d019f0
---
M cel/cel_radius.c
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/01/1901/1
diff --git a/cel/cel_radius.c b/cel/cel_radius.c
index 617ca66..41e3340 100644
--- a/cel/cel_radius.c
+++ b/cel/cel_radius.c
@@ -95,7 +95,7 @@
#define RADIUS_BACKEND_NAME "CEL Radius Logging"
-#define ADD_VENDOR_CODE(x,y) (rc_avpair_add(rh, send, x, &y, strlen(y), VENDOR_CODE))
+#define ADD_VENDOR_CODE(x,y) (rc_avpair_add(rh, send, x, y, strlen(y), VENDOR_CODE))
static int build_radius_record(VALUE_PAIR **send, struct ast_cel_event_record *record)
{
--
To view, visit https://gerrit.asterisk.org/1901
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I72889bccd8fde120d47aa659edc0e7e6d4d019f0
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Aaron An <anjb at ti-net.com.cn>
More information about the asterisk-code-review
mailing list