[Asterisk-code-review] AMI: Escape string values. (asterisk[13])

[Kevin Harwell]

Kevin Harwell has posted comments on this change.

Change subject: AMI: Escape string values.
......................................................................


Patch Set 1:

I'll cherry pick this change to master once I feel like this patch is reaching the finished point.

I did some basic manual testing for this, added a check in the strings unit test, added a new testsuite test (https://gerrit.asterisk.org/#/c/561/), and also ran the currently existing manager/ami test and they all passed.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib55a5b84fe0481b0f2caaaab68c566f392c0aac0
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-HasComments: No