[Asterisk-code-review] jansson: Backport fixes to bundled, use json vsprintf if ava... (asterisk[13])

Corey Farrell asteriskteam at digium.com
Mon Sep 24 17:59:48 CDT 2018


Hello Richard Mudgett, Kevin Harwell, Jenkins2, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/10252

to look at the new patch set (#2).

Change subject: jansson: Backport fixes to bundled, use json_vsprintf if available.
......................................................................

jansson: Backport fixes to bundled, use json_vsprintf if available.

Use json_vsprintf from versions which contain fix for va_copy leak.

Apply fixes from jansson master:
* va_copy leak fix.
* Avoid potential invalid memory read in json_pack.
* Rename variable that shadowed another.

Change-Id: I7522e462d2a52f53010ffa1e7d705c666ec35539
---
M configure
M include/asterisk/autoconfig.h.in
M main/json.c
M menuselect/configure
M third-party/jansson/configure.m4
A third-party/jansson/patches/0022-Avoid-invalid-memory-read-in-json_pack.patch
A third-party/jansson/patches/0025-Call-va_end-after-va_copy-in-json_vsprintf.patch
A third-party/jansson/patches/0027-Rename-a-varialble-that-shadows-another-one.patch
8 files changed, 182 insertions(+), 34 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/52/10252/2
-- 
To view, visit https://gerrit.asterisk.org/10252
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I7522e462d2a52f53010ffa1e7d705c666ec35539
Gerrit-Change-Number: 10252
Gerrit-PatchSet: 2
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2 (1000185)
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180924/c21191ba/attachment-0001.html>


More information about the asterisk-code-review mailing list