diff options
author | mmichelson <mmichelson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-09-15 15:12:35 +0000 |
---|---|---|
committer | mmichelson <mmichelson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-09-15 15:12:35 +0000 |
commit | 7801c53d3f3d83621f7b2d9b138384e4aed656b1 (patch) | |
tree | e14141af131fdf0edb5d3bacbe8ab030831f9bed /channels | |
parent | 2e3a704ab034289bf4c26263a2edd7c7e4f0e384 (diff) |
Merged revisions 218499,218504 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk
........
r218499 | mmichelson | 2009-09-15 09:59:50 -0500 (Tue, 15 Sep 2009) | 3 lines
Fix off-by-one error when reading SDP sent over TCP.
........
r218504 | mmichelson | 2009-09-15 10:05:53 -0500 (Tue, 15 Sep 2009) | 3 lines
Ensure that SDP read from TCP socket is null-terminated.
........
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.1@218506 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'channels')
-rw-r--r-- | channels/chan_sip.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index d967bde59..5a07d782e 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -2449,10 +2449,11 @@ static void *_sip_tcp_helper_thread(struct sip_pvt *pvt, struct ast_tcptls_sessi if (sscanf(get_header(&reqcpy, "Content-Length"), "%30d", &cl)) { while (cl > 0) { ast_mutex_lock(&tcptls_session->lock); - if (!fread(buf, (cl < sizeof(buf)) ? cl : sizeof(buf), 1, tcptls_session->f)) { + if (!fread(buf, MIN(sizeof(buf) - 1, cl), 1, tcptls_session->f)) { ast_mutex_unlock(&tcptls_session->lock); goto cleanup; } + buf[sizeof(buf)-1] = '\0'; ast_mutex_unlock(&tcptls_session->lock); if (me->stop) goto cleanup; |