Segfault when linking Scale widget with Entry

[timholy]

If I move the scale slowly, this works as intended. However, quick movement---or any movement, if I delete the @schedule---causes a segfault for me:

using Gtk.ShortNames

e = @Entry()
sc = @Scale(false, 1:10)
hbox = @Box(:v)
win = @Window(hbox, "Segfault")
push!(hbox, e)
push!(hbox, sc)

signal_connect(sc, "value-changed") do widget
    println("value-changed")
    adj = @Adjustment(sc)
    val = getproperty(adj, :value, Int)
    @schedule setproperty!(e, :text, string(val))
end

showall(win)

I also get the segfault if I use G_.value(sc) rather than getproperty.

[timholy]

Oh, and here's the backtrace:

signal (11): Segmentation fault
unknown function (ip: 0x7f5bfe964178)
g_signal_emit_valist at /usr/lib/x86_64-linux-gnu/libgobject-2.0.so (unknown line)
g_signal_emit at /usr/lib/x86_64-linux-gnu/libgobject-2.0.so (unknown line)
unknown function (ip: 0x7f5bfe957725)
g_object_notify at /usr/lib/x86_64-linux-gnu/libgobject-2.0.so (unknown line)
g_closure_invoke at /usr/lib/x86_64-linux-gnu/libgobject-2.0.so (unknown line)
unknown function (ip: 0x7f5bfe964557)
g_signal_emit_valist at /usr/lib/x86_64-linux-gnu/libgobject-2.0.so (unknown line)
g_signal_emit at /usr/lib/x86_64-linux-gnu/libgobject-2.0.so (unknown line)
unknown function (ip: 0x7f5bfdc25c54)
unknown function (ip: 0x7f5bfdc1c1ae)
g_closure_invoke at /usr/lib/x86_64-linux-gnu/libgobject-2.0.so (unknown line)
unknown function (ip: 0x7f5bfe964afb)
g_signal_emit_valist at /usr/lib/x86_64-linux-gnu/libgobject-2.0.so (unknown line)
g_signal_emit_by_name at /usr/lib/x86_64-linux-gnu/libgobject-2.0.so (unknown line)
unknown function (ip: 0x7f5bfdc16d57)
gtk_entry_set_text at /usr/lib/x86_64-linux-gnu/libgtk-3.so.0 (unknown line)
g_object_set_property at /usr/lib/x86_64-linux-gnu/libgobject-2.0.so (unknown line)
setproperty! at /home/tim/.julia/v0.4/Gtk/src/GLib/gvalues.jl:163
jl_apply_generic at /home/tim/src/julia/usr/bin/../lib/libjulia.so (unknown line)
anonymous at task.jl:67
unknown function (ip: 0x7f5e119d0b58)
unknown function (ip: (nil))
Segmentation fault (core dumped)

[timholy]

I wanted to check that I wasn't missing something, so here's a comparison C program (no segfault):

#include <gtk/gtk.h>
#include <stdio.h>

char lbl[255] = "howdy";

void set_entry(GtkWidget *widget, gpointer data)
{
    GtkAdjustment *adj;
    GtkEntry *entry;
    double val;
    char *lbl;

    adj = gtk_range_get_adjustment((GtkRange*) widget);
    val = gtk_adjustment_get_value(adj);
    sprintf(lbl, "%g", val);
    entry = (GtkEntry*) data;
    gtk_entry_set_text(entry, lbl);
}

int main(int argc, char *argv[])
{
    GtkWidget *window;
    GtkWidget *entry;
    GtkWidget *scale;
    GtkBox *vbox;

    gtk_init(&argc, &argv);

    window = gtk_window_new(GTK_WINDOW_TOPLEVEL);
    vbox = (GtkBox*) gtk_box_new(GTK_ORIENTATION_VERTICAL, 10);
    entry = gtk_entry_new();
    scale = gtk_scale_new_with_range(GTK_ORIENTATION_HORIZONTAL, 0, 10, 1);
    gtk_entry_set_text((GtkEntry*)entry, lbl);

    gtk_container_add(GTK_CONTAINER(window), (GtkWidget*)vbox);
    gtk_box_pack_start(vbox, entry, FALSE, FALSE, 10);
    gtk_box_pack_start(vbox, scale, FALSE, FALSE, 10);

    g_signal_connect(window, "delete-event", G_CALLBACK (gtk_main_quit),
NULL);
    g_signal_connect(scale, "value-changed", G_CALLBACK (set_entry), entry);

    gtk_widget_show_all(window);
    gtk_main();
    return 0;
}

[timholy]

No segfault if I use the more low-level approach:

function value_changed_cb(ptr::Ptr, entry)
    sc = convert(Scale, ptr)
    val = G_.value(sc)
    setproperty!(entry, :text, string(val))
    nothing
end

signal_connect(value_changed_cb, sc, "value-changed", Void, (), false, e)

[vtjnash]

this is most likely an issue with the julia option COPY_STACKS corrupting the stack. it looks like another redesign of the gtk callback handling may be in my future to work around that.

[timholy]

Joy joy. Meanwhile, no rush since I figured out a workaround that I'm quite content with.

[vtjnash]

While I think the real solution to this will be to merge #13099, I think a temporary solution would be to remove the g_siginterruptible from https://github.com/JuliaLang/Gtk.jl/blob/40f7442dea20919c5a7b137594fb6d7636fa2329/src/GLib/signals.jl#L91 and just avoid calling yield / print / async from inside any Gtk callback.