After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 768981 - Failed assertion when adding slaves to a new bond connection
Failed assertion when adding slaves to a new bond connection
Status: RESOLVED FIXED
Product: NetworkManager
Classification: Platform
Component: nmtui
git master
Other Linux
: Normal normal
: ---
Assigned To: NetworkManager maintainer(s)
NetworkManager maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2016-07-20 06:51 UTC by Beniamino Galvani
Modified: 2016-08-01 15:30 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
[PATCH] tui: avoid failed assertion when adding bond slaves (3.48 KB, patch)
2016-07-20 06:54 UTC, Beniamino Galvani 		none Details | Review

Description Beniamino Galvani 2016-07-20 06:51:06 UTC 
When trying to add multiple slaves to a new bond connection through nmtui the following assertion fails:

nmtui-CRITICAL **: nmt_newt_form_quit: assertion 'priv->form != NULL' failed

+ Trace 236472

  • #0 g_logv
    at gmessages.c line 324
  • #1 g_logv
    at gmessages.c line 1081
  • #2 g_log
    at gmessages.c line 1119
  • #3 g_return_if_fail_warning
    at gmessages.c line 1128
  • #4 nmt_newt_form_quit
    at nmt-newt-form.c line 425
  • #5 create_connection
    at nmtui-edit.c line 188
  • #6 nmt_add_connection_show
    at nmtui-edit.c line 271
  • #7 nmt_newt_form_show
    at nmt-newt-form.c line 388
  • #8 nmt_add_connection_full
    at nmtui-edit.c line 434
  • #9 nmt_slave_list_add_connection
    at nmt-slave-list.c line 136
  • #10 _g_closure_invoke_va
  • #11 g_signal_emit_valist
    at gsignal.c line 3292
  • #12 g_signal_emit
    at gsignal.c line 3439
  • #13 add_clicked
    at nmt-edit-connection-list.c line 287
  • #14 _g_closure_invoke_va
  • #15 g_signal_emit_valist
    at gsignal.c line 3292
  • #16 g_signal_emit
    at gsignal.c line 3439
  • #17 nmt_newt_button_activated
    at nmt-newt-button.c line 173
  • #18 _g_closure_invoke_va
  • #19 g_signal_emit_valist
    at gsignal.c line 3292
  • #20 g_signal_emit
    at gsignal.c line 3439
  • #21 nmt_newt_widget_activated
    at nmt-newt-widget.c line 329
  • #22 nmt_newt_form_iterate
    at nmt-newt-form.c line 309
  • #23 nmt_newt_form_keypress_callback
    at nmt-newt-form.c line 335
  • #24 g_main_context_dispatch
    at gmain.c line 3154
  • #25 g_main_context_dispatch
    at gmain.c line 3769
  • #26 g_main_context_iterate
    at gmain.c line 3840
  • #27 g_main_loop_run
    at gmain.c line 4034
  • #28 main
    at nmtui.c line 298 






Comment 1


Beniamino Galvani





          2016-07-20 06:54:44 UTC
        



Created attachment 331800 [details] [review]
[PATCH] tui: avoid failed assertion when adding bond slaves






Comment 2


Dan Williams





          2016-07-30 14:33:34 UTC
        



LGTM






Comment 3


Francesco Giudici





          2016-08-01 14:40:26 UTC
        



LGTM