bundled_pjproject: sip_parser: Fix return code in pjsip_find_msg
[asterisk/asterisk.git] / third-party / pjproject / patches / 0021-sip_parser-Fix-return-code-in-pjsip_find_msg-and-add.patch
1 From 186f82627c40d0c3a56a6a94ce55c055ad1f7620 Mon Sep 17 00:00:00 2001
2 From: George Joseph <gjoseph@digium.com>
3 Date: Fri, 10 Nov 2017 09:26:29 -0700
4 Subject: [PATCH] sip_parser: Fix return code in pjsip_find_msg and add
5  "volatile"
6
7 The default return code for pjsip_find_msg was PJ_SUCCESS so if
8 a Content-Length header wasn't found at all, pjsip_find_msg was
9 returning PJ_SUCCESS instead of PJSIP_EMISSINGHDR.
10
11 Also added the volatile keyword to a few variables what are used
12 both insude and outsude the PJ_TRY/PJ_CATCH block.
13 ---
14  pjsip/src/pjsip/sip_parser.c | 8 ++++----
15  1 file changed, 4 insertions(+), 4 deletions(-)
16
17 diff --git a/pjsip/src/pjsip/sip_parser.c b/pjsip/src/pjsip/sip_parser.c
18 index f9a0e65b5..f65ffca5a 100644
19 --- a/pjsip/src/pjsip/sip_parser.c
20 +++ b/pjsip/src/pjsip/sip_parser.c
21 @@ -834,13 +834,13 @@ PJ_DEF(pj_status_t) pjsip_find_msg( const char *buf, pj_size_t size,
22                                   pj_bool_t is_datagram, pj_size_t *msg_size)
23  {
24  #if PJ_HAS_TCP
25 -    const char *hdr_end;
26 -    const char *body_start;
27 +    const char *volatile hdr_end;
28 +    const char *volatile body_start;
29      const char *pos;
30 -    const char *line;
31 +    const char *volatile line;
32      int content_length = -1;
33      pj_str_t cur_msg;
34 -    pj_status_t status = PJ_SUCCESS;
35 +    volatile pj_status_t status = PJSIP_EMISSINGHDR;
36      const pj_str_t end_hdr = { "\n\r\n", 3};
37  
38      *msg_size = size;
39 -- 
40 2.13.6
41