[Asterisk-code-review] ast json pack(): Use safer json ref mechanism. (asterisk[15])
Jenkins2
asteriskteam at digium.com
Tue Dec 19 10:16:37 CST 2017
Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/7636 )
Change subject: ast_json_pack(): Use safer json ref mechanism.
......................................................................
ast_json_pack(): Use safer json ref mechanism.
Change-Id: I49204db2e57ae96eee43909c18ed007c09ac817e
---
M main/cel.c
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Corey Farrell: Looks good to me, but someone else must approve
George Joseph: Looks good to me, approved
Jenkins2: Approved for Submit
diff --git a/main/cel.c b/main/cel.c
index cec0a85..24a35be 100644
--- a/main/cel.c
+++ b/main/cel.c
@@ -1835,9 +1835,9 @@
struct ast_json *cel_blob;
struct stasis_message *message;
- cel_blob = ast_json_pack("{s: i, s: O}",
+ cel_blob = ast_json_pack("{s: i, s: o}",
"event_type", event_type,
- "event_details", blob);
+ "event_details", ast_json_ref(blob));
message = ast_channel_blob_create_from_cache(ast_channel_uniqueid(chan), cel_generic_type(), cel_blob);
if (message) {
--
To view, visit https://gerrit.asterisk.org/7636
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-Project: asterisk
Gerrit-Branch: 15
Gerrit-MessageType: merged
Gerrit-Change-Id: I49204db2e57ae96eee43909c18ed007c09ac817e
Gerrit-Change-Number: 7636
Gerrit-PatchSet: 1
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20171219/45520fac/attachment.html>
More information about the asterisk-code-review
mailing list