diff options
author | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-11-23 15:35:19 +0000 |
---|---|---|
committer | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-11-23 15:35:19 +0000 |
commit | 3623baae6e72a55a9123b149db8b4be2c1e5ba60 (patch) | |
tree | 458a93bd44645002412e3faf65fe4bf26ba16ab1 | |
parent | f62331e8629f90506e629665a9941d3b8ac175c3 (diff) |
Merged revisions 230877 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk
................
r230877 | kpfleming | 2009-11-23 09:34:16 -0600 (Mon, 23 Nov 2009) | 9 lines
Merged revisions 230839 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r230839 | kpfleming | 2009-11-23 09:09:24 -0600 (Mon, 23 Nov 2009) | 1 line
Correct fix for issue #16268... the reporter's original patch was very close to correct.
........
................
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.0@230878 f38db490-d61c-443f-a65b-d21fe96a405b
-rw-r--r-- | channels/chan_sip.c | 25 |
1 files changed, 15 insertions, 10 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index 1495606ab..5790824f9 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -830,7 +830,7 @@ struct sip_request { int method; /*!< Method of this request */ int lines; /*!< Body Content */ unsigned int sdp_start; /*!< the line number where the SDP begins */ - unsigned int sdp_end; /*!< the line number where the SDP ends */ + unsigned int sdp_count; /*!< the number of lines of SDP */ char debug; /*!< print extra debugging if non zero */ char has_to_tag; /*!< non-zero if packet has To: tag */ char ignore; /*!< if non-zero This is a re-transmit, ignore it */ @@ -5883,26 +5883,31 @@ static const char *get_sdp_iterate(int *start, struct sip_request *req, const ch { int len = strlen(name); - while (*start <= req->sdp_end) { + while (*start < (req->sdp_start + req->sdp_count)) { const char *r = get_body_by_line(req->line[(*start)++], name, len); if (r[0] != '\0') return r; } + /* if the line was not found, ensure that *start points past the SDP */ + (*start)++; + return ""; } /*! \brief Fetches the next valid SDP line between the 'start' line - * and the 'stop' line (*excluding* the 'stop' line). Returns the type - * ('a', 'c', ...) and matching line in reference 'start' is updated with the - * next line number. + * (inclusive) and the 'stop' line (exclusive). Returns the type + * ('a', 'c', ...) and matching line in reference 'start' is updated + * with the next line number. */ static char get_sdp_line(int *start, int stop, struct sip_request *req, const char **value) { char type = '\0'; const char *line = NULL; - if (stop > req->sdp_end || stop < req->sdp_start) stop = req->sdp_end + 1; + if (stop > (req->sdp_start + req->sdp_count)) { + stop = req->sdp_start + req->sdp_count; + } while (*start < stop) { line = req->line[(*start)++]; @@ -6715,7 +6720,7 @@ static int parse_request(struct sip_request *req) \param req the SIP request to process \return 1 if SDP found, 0 if not found - Also updates req->sdp_start and req->sdp_end to indicate where the SDP + Also updates req->sdp_start and req->sdp_count to indicate where the SDP lives in the message body. */ static int find_sdp(struct sip_request *req) @@ -6748,7 +6753,7 @@ static int find_sdp(struct sip_request *req) /* if the body contains only SDP, this is easy */ if (!strncasecmp(content_type, "application/sdp", 15)) { req->sdp_start = 0; - req->sdp_end = req->lines; + req->sdp_count = req->lines; return req->lines ? 1 : 0; } @@ -6787,7 +6792,7 @@ static int find_sdp(struct sip_request *req) for (x = 0; x < (req->lines ); x++) { if(!strncasecmp(req->line[x], boundary, strlen(boundary))){ if(found_application_sdp && found_end_of_headers){ - req->sdp_end = x-1; + req->sdp_count = (x - 1) - req->sdp_start; return 1; } found_application_sdp = FALSE; @@ -6803,7 +6808,7 @@ static int find_sdp(struct sip_request *req) } } if(found_application_sdp && found_end_of_headers) { - req->sdp_end = x; + req->sdp_count = x - req->sdp_start; return TRUE; } return FALSE; |