Instead of crashing, allocate our header ast_str before we try to use it.
authorSean Bright <sean@malleable.com>
Sun, 24 Jan 2010 21:49:17 +0000 (21:49 +0000)
committerSean Bright <sean@malleable.com>
Sun, 24 Jan 2010 21:49:17 +0000 (21:49 +0000)
(closes issue #16680)
Reported by: lmadsen
Patches:
      issue16680_20100122.patch uploaded by seanbright (license 71)
Tested by: lmadsen

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

res/res_phoneprov.c

index ff3046b..91268a5 100644 (file)
@@ -514,6 +514,7 @@ static int phoneprov_callback(struct ast_tcptls_session_instance *ser, const str
                        ast_free(file);
                }
 
+               http_header = ast_str_create(80);
                ast_str_set(&http_header, 0, "Content-type: %s\r\n",
                        route->file->mime_type);
 
@@ -522,6 +523,7 @@ static int phoneprov_callback(struct ast_tcptls_session_instance *ser, const str
                        if (tmp) {
                                ast_free(tmp);
                        }
+                       ast_free(http_header);
                        goto out500;
                }
                ast_str_append(&result, 0, "%s", ast_str_buffer(tmp));