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 688789 - Crash on startup
Crash on startup
Status: RESOLVED FIXED
Product: gnome-boxes
Classification: Applications
Component: general
3.6.x
Other Linux
: Normal normal
: --
Assigned To: GNOME Boxes maintainer(s)
GNOME Boxes maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-11-21 09:57 UTC by Bastien Nocera
Modified: 2016-03-31 14:00 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Don't crash when running Boxes while it's already running (1.15 KB, patch)
2012-11-21 11:21 UTC, Christophe Fergeau
committed Details | Review

Description Bastien Nocera 2012-11-21 09:57:13 UTC
gnome-boxes-3.6.2-1.fc18.x86_64

With an existing instance running, run gnome-boxes again.

Program received signal SIGSEGV, Segmentation fault.
0x0000000000425600 in boxes_app_suspend_machines (self=0xb3b060) at app.c:3262
3262		_tmp3_ = _tmp2_->items;

(gdb) bt
  • #0 boxes_app_suspend_machines
    at app.c line 3262
  • #1 _vala_main
    at main.c line 953
  • #2 __libc_start_main
    from /lib64/libc.so.6
  • #3 _start

(gdb) list
3257		_tmp0_ = g_main_context_new ();
3258		context = _tmp0_;
3259		_tmp1_ = context;
3260		g_main_context_push_thread_default (_tmp1_);
3261		_tmp2_ = self->collection;
3262		_tmp3_ = _tmp2_->items;
3263		_tmp4_ = _tmp3_->pdata;
3264		_tmp4__length1 = (gint) _tmp3_->len;
3265		{
3266			BoxesCollectionItem** item_collection = NULL;
Comment 1 Christophe Fergeau 2012-11-21 11:21:55 UTC
Created attachment 229564 [details] [review]
Don't crash when running Boxes while it's already running

When running Boxes while an instance is already running, the
new instance crashes in App::suspend_machines. This happens because
App::collection is null and App::suspend_machines tries to dereference
it. This commit just silently exits from App::suspend_machines in such
situations as there will be no machines to save anyway.
Comment 2 Marc-Andre Lureau 2012-11-21 11:46:32 UTC
Review of attachment 229564 [details] [review]:

ack
Comment 3 Zeeshan Ali 2013-01-07 01:21:23 UTC
Comment on attachment 229564 [details] [review]
Don't crash when running Boxes while it's already running

This has been committed a long time ago. :)