Fix: release alpn string before set for tls_ctx_options (#588)

xiazhvera · TingDaoK · web-flow · commit a3555c86bd10 · 2023-08-03T00:04:22.000-07:00
* release alpn string before set

* Update source/tls_channel_handler.c

Co-authored-by: Dengke Tang &lt;dengket@amazon.com&gt;

* clang-format

---------

Co-authored-by: Dengke Tang &lt;dengket@amazon.com&gt;
diff --git a/source/tls_channel_handler.c b/source/tls_channel_handler.c
@@ -441,6 +441,8 @@ int aws_tls_ctx_options_init_default_server(
 }
 
 int aws_tls_ctx_options_set_alpn_list(struct aws_tls_ctx_options *options, const char *alpn_list) {
+    aws_string_destroy(options->alpn_list);
+
     options->alpn_list = aws_string_new_from_c_str(options->allocator, alpn_list);
     if (!options->alpn_list) {
         return AWS_OP_ERR;

Original file line number	Diff line number	Diff line change
`@@ -441,6 +441,8 @@ int aws_tls_ctx_options_init_default_server(`
`441`	`441`	`}`
`442`	`442`
`443`	`443`	`int aws_tls_ctx_options_set_alpn_list(struct aws_tls_ctx_options options, const char alpn_list) {`
	`444`	`+ aws_string_destroy(options->alpn_list);`
	`445`	`+`
`444`	`446`	`options->alpn_list = aws_string_new_from_c_str(options->allocator, alpn_list);`
`445`	`447`	`if (!options->alpn_list) {`
`446`	`448`	`return AWS_OP_ERR;`