[Asterisk-code-review] cdr raidus,cel radius: Fix old memleak in unload (asterisk[master])
Joshua Colp
asteriskteam at digium.com
Tue Oct 25 06:45:33 CDT 2016
Hello Richard Mudgett, Anonymous Coward #1000019, Badalian Vyacheslav,
I'd like you to reexamine a change. Please visit
https://gerrit.asterisk.org/4059
to look at the new patch set (#4).
Change subject: cdr_raidus,cel_radius: Fix old memleak in unload
......................................................................
cdr_raidus,cel_radius: Fix old memleak in unload
- Call "rc_openlog" optional. If you do not call,
you will simply NULL instead of a name.
- On the one PID can be only one syslog channel.
And it can already be run in logger.c
- Calling rc_openlog we assigns a new name for
the channel syslog. This unexpected behavior for logger.c.
Most lesser evil, is to agree on a NULL name syslog
if the channel was not launched in logger.c.
It also solves the problem of memory leaks.
ASTERISK-26455 #close
Change-Id: Ic17c38de67583e971d78fe18807d1a9faf8f0afd
---
M CHANGES
M cdr/cdr_radius.c
M cel/cel_radius.c
3 files changed, 10 insertions(+), 24 deletions(-)
git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/59/4059/4
--
To view, visit https://gerrit.asterisk.org/4059
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic17c38de67583e971d78fe18807d1a9faf8f0afd
Gerrit-PatchSet: 4
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Badalian Vyacheslav <v.badalyan at open-bs.ru>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Badalian Vyacheslav <v.badalyan at open-bs.ru>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
More information about the asterisk-code-review
mailing list