Bug 867149

Summary: [abrt] gnome-boxes-3.6.1-1.fc18: strlen: Process /usr/libexec/gnome-boxes-search-provider was killed by signal 11 (SIGSEGV)
Product: [Fedora] Fedora Reporter: Andreas Lorensen <andlo>
Component: gnome-boxesAssignee: Christophe Fergeau <cfergeau>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 18CC: alexl, berrange, cfergeau, kerstgens, marcandre.lureau, mfojtik, virt-maint, zeenix
Target Milestone: ---   
Target Release: ---   
Hardware: i686   
OS: Unspecified   
Whiteboard: abrt_hash:45d658a7bc1f049e562dba724115cb7caa32e0fa
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-11-23 07:47:54 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
File: core_backtrace
none
File: environ
none
File: limits
none
File: backtrace
none
File: cgroup
none
File: maps
none
File: dso_list
none
File: build_ids
none
File: var_log_messages
none
File: open_fds
none
File: proc_pid_status
none
Fix none

Description Andreas Lorensen 2012-10-16 21:41:05 UTC
Version-Release number of selected component:
gnome-boxes-3.6.1-1.fc18

Additional info:
libreport version: 2.0.16
abrt_version:   2.0.15
backtrace_rating: 4
cmdline:        /usr/libexec/gnome-boxes-search-provider
crash_function: strlen
kernel:         3.6.1-1.fc18.i686

truncated backtrace:
:Thread no. 1 (8 frames)
: #0 strlen at ../sysdeps/i386/i686/multiarch/strlen-sse2-bsf.S:50
: #1 g_strdup at gstrfuncs.c:363
: #2 boxes_search_provider_GetResultMetas_co at /extra-data/checkout/gnome/gnome-boxes/src/gnome-boxes-search-provider.vala:116
: #3 g_simple_async_result_complete at gsimpleasyncresult.c:775
: #4 complete_in_idle_cb_for_thread at gsimpleasyncresult.c:843
: #9 g_main_context_iteration at gmain.c:3351
: #10 g_application_run at gapplication.c:1620
: #11 _vala_main at /extra-data/checkout/gnome/gnome-boxes/src/gnome-boxes-search-provider.vala:197

Comment 1 Andreas Lorensen 2012-10-16 21:41:08 UTC
Created attachment 628419 [details]
File: core_backtrace

Comment 2 Andreas Lorensen 2012-10-16 21:41:11 UTC
Created attachment 628420 [details]
File: environ

Comment 3 Andreas Lorensen 2012-10-16 21:41:13 UTC
Created attachment 628421 [details]
File: limits

Comment 4 Andreas Lorensen 2012-10-16 21:41:15 UTC
Created attachment 628422 [details]
File: backtrace

Comment 5 Andreas Lorensen 2012-10-16 21:41:17 UTC
Created attachment 628423 [details]
File: cgroup

Comment 6 Andreas Lorensen 2012-10-16 21:41:19 UTC
Created attachment 628424 [details]
File: maps

Comment 7 Andreas Lorensen 2012-10-16 21:41:21 UTC
Created attachment 628425 [details]
File: dso_list

Comment 8 Andreas Lorensen 2012-10-16 21:41:23 UTC
Created attachment 628426 [details]
File: build_ids

Comment 9 Andreas Lorensen 2012-10-16 21:41:24 UTC
Created attachment 628427 [details]
File: var_log_messages

Comment 10 Andreas Lorensen 2012-10-16 21:41:26 UTC
Created attachment 628428 [details]
File: open_fds

Comment 11 Andreas Lorensen 2012-10-16 21:41:28 UTC
Created attachment 628429 [details]
File: proc_pid_status

Comment 12 Christophe Fergeau 2012-10-17 10:45:28 UTC
/extra-data/checkout/gnome/gnome-boxes/src/gnome-boxes-search-provider.vala:116 is foreach (var id in ids), and vala indeed generates a g_strdup when doing the iteration. As g_strdup handles NULL string, this probably means we ended up with an invalid string in this list.
I couldn't spot something wrong in there though :( Any chance you can catch this crash in valgrind?

Comment 13 Andreas Lorensen 2012-10-17 12:04:28 UTC
Any chance you can catch this crash in valgrind?

I dont know what that means :( I am new to fedora and new on repporting bugs.

Comment 14 Christophe Fergeau 2012-10-17 12:23:41 UTC
Ok, never mind then, thanks for the bug report and your answer ;) Is it an issue you can reproduce whenever you want, or did it occur only once?

Comment 15 Andreas Lorensen 2012-10-17 12:44:59 UTC
I get it when I do a reboot. It started after latest update where gnome-boxes were updated.

I dosent seem to have any impact on my system, as it is working as usaual.

Comment 16 Andreas Lorensen 2012-10-17 15:52:42 UTC
Just updated gnome-boxes to 3.6.1.1-1.fc18 and no ABRT when rebooting.

Comment 17 Christophe Fergeau 2012-10-19 09:19:59 UTC
*** Bug 868186 has been marked as a duplicate of this bug. ***

Comment 18 Alexander Larsson 2012-10-23 18:16:22 UTC
Created attachment 632271 [details]
Fix

Comment 19 Christophe Fergeau 2012-10-24 12:25:20 UTC
+    /* We have to put this in a separate file because vala does not seem to honor "owned"

should be 'separate method'
Did I understand correctly that the issue is that we are calling async methods from the dbus callback, and that vala does not copy the callback arguments in this case, which means they don't stay alive for the whole duration of the async method calls, hence the use of 'owned' to force the copy? If yes, ACK, otherwise I'll have to go back reviewing ;)

Comment 20 Alexander Larsson 2012-10-25 08:07:14 UTC
> Did I understand correctly that the issue is that we are calling async methods 
> from the dbus callback, and that vala does not copy the callback arguments in 
> this case, which means they don't stay alive for the whole duration of the
> async method calls, hence the use of 'owned' to force the copy?

Yes, this is exactly the problem, which then gets extra complicated by a valac bug that makes calling methods with "owned" arguments not work in the generated dbus wrapper method.

Comment 21 Alexander Larsson 2012-10-25 08:11:49 UTC
Pushed to git, will be fixed in next minor release.

Comment 22 Christophe Fergeau 2012-11-12 14:23:30 UTC
*** Bug 872970 has been marked as a duplicate of this bug. ***

Comment 23 Fedora Update System 2012-11-13 11:28:21 UTC
gnome-boxes-3.6.2-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/gnome-boxes-3.6.2-1.fc18

Comment 24 Fedora Update System 2012-11-13 19:04:46 UTC
Package gnome-boxes-3.6.2-1.fc18:
* should fix your issue,
* was pushed to the Fedora 18 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing gnome-boxes-3.6.2-1.fc18'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-18091/gnome-boxes-3.6.2-1.fc18
then log in and leave karma (feedback).

Comment 25 Fedora Update System 2012-11-23 07:47:56 UTC
gnome-boxes-3.6.2-1.fc18 has been pushed to the Fedora 18 stable repository.  If problems still persist, please make note of it in this bug report.