Merged revisions 310240 via svnmerge from
authorTerry Wilson <twilson@digium.com>
Thu, 10 Mar 2011 16:09:09 +0000 (16:09 +0000)
committerTerry Wilson <twilson@digium.com>
Thu, 10 Mar 2011 16:09:09 +0000 (16:09 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r310240 | twilson | 2011-03-10 10:05:45 -0600 (Thu, 10 Mar 2011) | 13 lines

  Add \r\n to remaining http headers passed to ast_http_send

  r309204 changed the behavior of ast_http_send. It now requires headers
  to be passed with trailing \r\n. This change updates the remaining
  instances in the code that did not pass the \r\n.

  (closes issue #18186)
  Reported by: nivaldomjunior
  Patches:
        res_phoneprov.c.diff uploaded by lathama (license 1028)
        manager.diff.txt uploaded by twilson (license 396)
  Tested by: lathama
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@310241 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/manager.c
res/res_phoneprov.c

index 1b0ce88..f69f3fa 100644 (file)
@@ -5852,7 +5852,7 @@ static int auth_http_callback(struct ast_tcptls_session_instance *ser,
                goto auth_callback_out;
        }
 
-       ast_str_append(&http_header, 0, "Content-type: text/%s", contenttype[format]);
+       ast_str_append(&http_header, 0, "Content-type: text/%s\r\n", contenttype[format]);
 
        if (format == FORMAT_XML) {
                ast_str_append(&out, 0, "<ajax-response>\n");
index e969bc4..24d7b65 100644 (file)
@@ -440,7 +440,7 @@ static int phoneprov_callback(struct ast_tcptls_session_instance *ser, const str
                }
 
                http_header = ast_str_create(80);
-               ast_str_set(&http_header, 0, "Content-type: %s",
+               ast_str_set(&http_header, 0, "Content-type: %s\r\n",
                        route->file->mime_type);
 
                ast_http_send(ser, method, 200, NULL, http_header, NULL, fd, 0);
@@ -504,7 +504,7 @@ static int phoneprov_callback(struct ast_tcptls_session_instance *ser, const str
                }
 
                http_header = ast_str_create(80);
-               ast_str_set(&http_header, 0, "Content-type: %s",
+               ast_str_set(&http_header, 0, "Content-type: %s\r\n",
                        route->file->mime_type);
 
                if (!(result = ast_str_create(512))) {