对话框创建和销毁循环会增加内存使用量

Question

我编写了一个简单的程序模块来询问用户的个人资料名称。为此，我创建了带有条目小部件的 windoww，并在网格中组织了两个按钮（确定和取消）。当用户输入一个已经存在的配置文件名称时，它会通过创建带有“确定”按钮的对话框来通知他这一事实，并且在他按下它之后，它会返回选择配置文件名称（同时窗口没有隐藏也没有破坏） .问题是，当我创建一个配置文件，然后在配置文件名称选择器和对话框（通过创建和销毁对话框创建一个简单的循环) 程序的内存使用量增加。

---

TL;DR 简单地创建和销毁 gtk 窗口（和对话框）似乎会导致内存泄漏。让应用程序处于循环状态使其内存使用量增加了约 1900%（从 10mb 到 200mb）。

不，我没有使用专为它设计的应用程序测试内存泄漏。 是的，我设置了 G_SLICE=always-malloc。 是的，在程序的后台运行了另一个线程（但我确信它不会导致任何泄漏） 如果您想了解有关内存中发生的情况的更多信息，我可以从 Windows 性能监视器发布屏幕。

问题是 - 是我造成的内存泄漏，还是 GTK 的错（我听说它有一个惰性的内存管理策略，但一段时间后内存使用量从 200mb 下降到 140mb 并保持在那里）？

代码如下：

// This callback racts to the ok and cancel buttons. If input was correcs
// or the user pressed cancel it destroys the window. Else it show error
// prompt. The showing error prompt seems to be the problem here.
void pickNameButtCB(GtkWidget *button, gpointer *call)
{
    GtkWidget *window = gtk_widget_get_toplevel(button);
    if( *((char*)call) == 'k')
    {
        GList *entryBase = gtk_container_get_children(GTK_CONTAINER(gtk_bin_get_child(GTK_BIN(window)))), *entry = entryBase;
        for(size_t i=g_list_length(entry); i>0 && !GTK_IS_ENTRY(entry->data); --i)
            entry = g_list_next(entry);
        if(gtk_entry_get_text_length(GTK_ENTRY(entry->data)) > 0)
        {
            const char *temp = gtk_entry_get_text(GTK_ENTRY(entry->data));
            char path[266];
            strcpy(path, settingsDir);
            strcat(path, temp);
            strcat(path, ".prof");

            if(settProfExists(path))
            {
                g_list_free(entryBase);
                showError(GTK_WINDOW(window), GTK_MESSAGE_ERROR, "Profile with that name already exists!");
                return;
            }
            // nothing here executes as well
        }
        else
        {
            /** doesn't execute when the memory leak happens */
        }
        g_list_free(entryBase);
    }
    gtk_widget_destroy(window);
    gtk_main_quit();
}

void showError(GtkWindow *parent, GtkMessageType type, const char *str)
{
    GtkWidget *window = gtk_message_dialog_new(parent, GTK_DIALOG_DESTROY_WITH_PARENT, type, GTK_BUTTONS_OK, str);
    g_signal_connect_swapped(window, "response", G_CALLBACK(gtk_widget_destroy), window);
    gtk_dialog_run(GTK_DIALOG(window));
}

bool settProfExists(const char *path)
{
    if(fileExists(path))
        return true;
    return false;
}

bool fileExists(const char *path)
{
    struct stat info;
    errno = 0;
    if((stat(path, &info)) != 0)
        if(errno == ENOENT)
            return false;
    return true;
}

Answer 1

您的代码太不完整，无法找到根本原因。

这一行尤其没有意义。我们缺少 entry 的定义，而您正试图在同一行 (*entry = entryBase;) 上执行多个操作。

GList *entryBase = gtk_container_get_children(GTK_CONTAINER(gtk_bin_get_child(GTK_BIN(window)))), *entry = entryBase;

但是，泄漏可能出现在您调用但未提供代码的函数中（settProfExists、newProfile）。所以请提供一个重现问题的最小可编译示例。

至于样式，有几个错误：您遍历列表的方式与处理数组的方式相同。看看 GList 是如何实现的，你会发现这很愚蠢（提示：看看 entry->prev 和 entry->next）：

   for(size_t i=g_list_length(entry); i>0 && !GTK_IS_ENTRY(entry->data); --i)
        entry = g_list_next(entry);

然后是架构问题：不要试图检查你的 UI 来猜测小部件在哪里，只需创建一个结构，然后将 call 参数和你感兴趣的 GtkEntry 传递给带有 gpointer 的结构回调的数据参数。

您也不应该手动创建路径：您使用的固定长度数组可能不够长。使用 GLib 提供的可移植的 g_build_path（并在使用后使用 g_free 释放路径），它将为您处理 Windows/Linux 目录分隔符。

对于您的对话框，当您运行 gtk_run_dialog 时，您可以在之后直接调用 gtk_destroy_widget 而不是连接信号：

GtkWidget *window = gtk_message_dialog_new(parent, GTK_DIALOG_DESTROY_WITH_PARENT, type, GTK_BUTTONS_OK, str);
gtk_dialog_run(GTK_DIALOG(window));
gtk_widget_destroy(window);