diff options
author | mvanbaak <mvanbaak@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-09-14 22:22:48 +0000 |
---|---|---|
committer | mvanbaak <mvanbaak@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-09-14 22:22:48 +0000 |
commit | 58bad7893f0d7e1e470272c2dfe236fccaf8763a (patch) | |
tree | d5d5056ea0f4fcb0cfe0fbb825aad511b5a40323 /channels/chan_skinny.c | |
parent | 90fd266baff465e63c4912b2489de1e603557022 (diff) |
Merged revisions 143082 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk
........
r143082 | mvanbaak | 2008-09-15 00:16:34 +0200 (Mon, 15 Sep 2008) | 11 lines
plug a couple of memleaks in chan_skinny.
(closes issue #13452)
Reported by: pj
Patches:
memleak5.diff uploaded by wedhorn (license 30)
Tested by: wedhorn, pj, mvanbaak
(closes issue #13294)
Reported by: pj
........
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.0@143083 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'channels/chan_skinny.c')
-rw-r--r-- | channels/chan_skinny.c | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/channels/chan_skinny.c b/channels/chan_skinny.c index cf0838eec..f415ad6ca 100644 --- a/channels/chan_skinny.c +++ b/channels/chan_skinny.c @@ -1845,7 +1845,7 @@ static int transmit_response(struct skinnysession *s, struct skinny_req *req) memcpy(s->outbuf+skinny_header_size, &req->data, letohl(req->len)); res = write(s->fd, s->outbuf, letohl(req->len)+8); - + if (res != letohl(req->len)+8) { ast_log(LOG_WARNING, "Transmit: write only sent %d out of %d bytes: %s\n", res, letohl(req->len)+8, strerror(errno)); if (res == -1) { @@ -1856,6 +1856,7 @@ static int transmit_response(struct skinnysession *s, struct skinny_req *req) } + ast_free(req); ast_mutex_unlock(&s->lock); return 1; } @@ -1953,9 +1954,11 @@ static void transmit_tone(struct skinnysession *s, int tone, int instance, int r req->data.stoptone.reference = htolel(reference); } - if (tone > 0) { - req->data.starttone.tone = htolel(tone); - } + //Bad, tone is already set so this is redundant and a change to the if above + //may lead to issues where we try to set a tone to a stop_tone_message + //if (tone > 0) { + // req->data.starttone.tone = htolel(tone); + //} transmit_response(s, req); } @@ -2019,8 +2022,10 @@ static void transmit_displaymessage(struct skinnysession *s, const char *text, i if (!(req = req_alloc(0, CLEAR_DISPLAY_MESSAGE))) return; - req->data.clearpromptstatus.lineInstance = instance; - req->data.clearpromptstatus.callReference = reference; + //what do we want hear CLEAR_DISPLAY_MESSAGE or CLEAR_PROMPT_STATUS??? + //if we are clearing the display, it appears there is no instance and refernece info (size 0) + //req->data.clearpromptstatus.lineInstance = instance; + //req->data.clearpromptstatus.callReference = reference; if (skinnydebug) ast_debug(1, "Clearing Display\n"); @@ -3444,6 +3449,7 @@ static int skinny_hangup(struct ast_channel *ast) sub->rtp = NULL; } ast_mutex_unlock(&sub->lock); + ast_free(sub); return 0; } |