10

我试图找出如何解决我在使用 Valgrind 运行这个程序时遇到的这些内存泄漏。泄漏发生在nShell_client_main. 但我不确定如何正确释放它们。

我尝试在 nShell_Connect 上释放它们，但它导致 libUV 中止程序。我尝试在 结束时释放它们nShell_client_main，但是在关闭循环时出现读/写错误。有谁知道我应该如何关闭这些句柄？我读过这个，这让我开始了。但是，它看起来已经过时了，因为uv_ip4_addr在最新版本中有不同的原型。

（nShell_main是“入口”点）

#include "nPort.h"
#include "nShell-main.h"

void nShell_Close(
    uv_handle_t * term_handle
){
}

void nShell_Connect(uv_connect_t * term_handle, int status){
    uv_close((uv_handle_t *) term_handle, 0);
}

nError * nShell_client_main(nShell * n_shell, uv_loop_t * n_shell_loop){

    int uv_error = 0;

    nError * n_error = 0;

    uv_tcp_t * n_shell_socket = 0;
    uv_connect_t * n_shell_connect = 0;

    struct sockaddr_in dest_addr;

    n_shell_socket = malloc(sizeof(uv_tcp_t));

    if (!n_shell_socket){
        // handle error
    }

    uv_error = uv_tcp_init(n_shell_loop, n_shell_socket);

    if (uv_error){
        // handle error
    }

    uv_error = uv_ip4_addr("127.0.0.1", NPORT, &dest_addr);

    if (uv_error){
        // handle error
    }

    n_shell_connect = malloc(sizeof(uv_connect_t));

    if (!n_shell_connect){
        // handle error
    }

    uv_error = uv_tcp_connect(n_shell_connect, n_shell_socket, (struct sockaddr *) &dest_addr, nShell_Connect);

    if (uv_error){
        // handle error
    }

    uv_error = uv_run(n_shell_loop, UV_RUN_DEFAULT);

    if (uv_error){
        // handle error
    }

    return 0;
}

nError * nShell_loop_main(nShell * n_shell){

    int uv_error = 0;

    nError * n_error = 0;

    uv_loop_t * n_shell_loop = 0;

    n_shell_loop = malloc(sizeof(uv_loop_t));

    if (!n_shell_loop){
        // handle error
    }

    uv_error = uv_loop_init(n_shell_loop);

    if (uv_error){
        // handle error
    }

    n_error = nShell_client_main(n_shell, n_shell_loop);

    if (n_error){
        // handle error
    }

    uv_loop_close(n_shell_loop);
    free(n_shell_loop);

    return 0;
}

断言发生在这段代码摘录中 switch 语句的末尾（取自 Joyent 在 Github 上的 libUV 页面）：

void uv_close(uv_handle_t* handle, uv_close_cb close_cb) {
  assert(!(handle->flags & (UV_CLOSING | UV_CLOSED)));

  handle->flags |= UV_CLOSING;
  handle->close_cb = close_cb;

  switch (handle->type) {
  case UV_NAMED_PIPE:
    uv__pipe_close((uv_pipe_t*)handle);
    break;

  case UV_TTY:
    uv__stream_close((uv_stream_t*)handle);
    break;

  case UV_TCP:
    uv__tcp_close((uv_tcp_t*)handle);
    break;

  case UV_UDP:
    uv__udp_close((uv_udp_t*)handle);
    break;

  case UV_PREPARE:
    uv__prepare_close((uv_prepare_t*)handle);
    break;

  case UV_CHECK:
    uv__check_close((uv_check_t*)handle);
    break;

  case UV_IDLE:
    uv__idle_close((uv_idle_t*)handle);
    break;

  case UV_ASYNC:
    uv__async_close((uv_async_t*)handle);
    break;

  case UV_TIMER:
    uv__timer_close((uv_timer_t*)handle);
    break;

  case UV_PROCESS:
    uv__process_close((uv_process_t*)handle);
    break;

  case UV_FS_EVENT:
    uv__fs_event_close((uv_fs_event_t*)handle);
    break;

  case UV_POLL:
    uv__poll_close((uv_poll_t*)handle);
    break;

  case UV_FS_POLL:
    uv__fs_poll_close((uv_fs_poll_t*)handle);
    break;

  case UV_SIGNAL:
    uv__signal_close((uv_signal_t*) handle);
    /* Signal handles may not be closed immediately. The signal code will */
    /* itself close uv__make_close_pending whenever appropriate. */
    return;

  default:
    assert(0); // assertion is happening here
  }

  uv__make_close_pending(handle);
}

我可以uv__tcp_close手动调用，但它不在公共标题中（而且可能不是正确的解决方案）。

4

2 回答 2

29

在调用关闭回调之前，libuv 不会使用句柄完成。那是您可以释放手柄的确切时刻。

我看到你打电话uv_loop_close，但你不检查返回值。如果还有待处理的句柄，它将返回 UV_EBUSY，所以你应该检查一下。

如果要关闭循环并关闭所有句柄，则需要执行以下操作：

  • 用于uv_stop停止循环
  • 使用uv_walk并调用uv_close所有未关闭的句柄
  • 再次运行循环，uv_run以便调用所有关闭回调，您可以在回调中释放内存
  • 调用uv_loop_close，它现在应该返回 0
于 2014-09-14T09:00:23.407 回答
12

我终于想出了如何停止循环并清理所有句柄。我创建了一堆句柄和SIGINT信号句柄：

uv_signal_t *sigint = new uv_signal_t;
uv_signal_init(uv_default_loop(), sigint);
uv_signal_start(sigint, on_sigint_received, SIGINT);

当收到SIGINT时（按下控制台中的 Ctrl+C）on_sigint_received回调被调用。on_sigint_received看起来像：

void on_sigint_received(uv_signal_t *handle, int signum)
{
    int result = uv_loop_close(handle->loop);
    if (result == UV_EBUSY)
    {
        uv_walk(handle->loop, on_uv_walk, NULL);
    }
}

它触发回调函数on_uv_walk：

void on_uv_walk(uv_handle_t* handle, void* arg)
{
    uv_close(handle, on_uv_close);
}

它试图关闭每个打开的 libuv 句柄。 注意：uv_stop我之前没有打电话给uv_walksaghul 。在函数被调用后，libuv 循环连续执行，并在下一次迭代中调用每个打开的句柄。如果您调用该函数，则不会调用回调。on_sigint_receivedon_uv_closeuv_stopon_uv_close

void on_uv_close(uv_handle_t* handle)
{
    if (handle != NULL)
    {
        delete handle;
    }
}

之后 libuv 没有打开句柄并完成循环（从 退出uv_run）：

uv_run(uv_default_loop(), UV_RUN_DEFAULT);
int result = uv_loop_close(uv_default_loop());
if (result)
{
    cerr << "failed to close libuv loop: " << uv_err_name(result) << endl;
}
else
{
    cout << "libuv loop is closed successfully!\n";
}
于 2017-11-13T17:43:41.587 回答