Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[http] Memory leak with wrapper_http_deinit(...) call #162

Open
ccli8 opened this issue Dec 6, 2019 · 0 comments
Open

[http] Memory leak with wrapper_http_deinit(...) call #162

ccli8 opened this issue Dec 6, 2019 · 0 comments

Comments

@ccli8
Copy link

ccli8 commented Dec 6, 2019

iotkit-embedded
3e60c94 (HEAD, origin/v3.1.0) [dev_model] change data type of alink msgid to int

In both the functions below, the wrapper_http_deinit(...) call must fix to wrapper_http_deinit(&http_handle):

src/http/iotx_http_api.c:

int IOT_HTTP_SendMessage(void *handle, iotx_http_message_param_t *msg_param)
{
......
do_exit_pre:

    if (http_handle) {
        wrapper_http_deinit(http_handle);
        http_handle = NULL;
    }
......
int IOT_HTTP_DeviceNameAuth(void *handle)
{
......
do_exit_pre:

    if (http_handle) {
        wrapper_http_deinit(http_handle);
        http_handle = NULL;
    }
......
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant