IntenseFinance/gnucash
3
0
Fork 0
mirror of https://github.com/Gnucash/gnucash.git synced 2025-02-25 18:55:30 -06:00
Code Issues Packages Projects Releases Wiki Activity

Prevent running off the end of the GncOptionUI widget's parent tree.

Browse Source
This commit is contained in:
John Ralls 2021-08-18 15:51:19 -07:00
parent b361582cf2
commit 7392ac6fcf
1 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
gnucash/gnome-utils/dialog-options.cpp
View File

@ -228,9 +228,16 @@ gnc_option_get_gtk_widget (const GncOption* option)
static void
dialog_changed_internal (GtkWidget *widget, bool sensitive)
{
while (widget && !GTK_IS_WINDOW(widget))
widget = gtk_widget_get_parent(widget);
if (!widget) return;
g_return_if_fail(widget);
while (TRUE)
{
auto new_widget = gtk_widget_get_parent(widget);
if (new_widget && GTK_IS_WIDGET(new_widget) &&
GTK_IS_WINDOW(new_widget))
widget = new_widget;
else
break;
}
/* find the ok and cancel buttons, we know where they will be so do it
this way as opposed to using gtk_container_foreach, much less iteration */