Bug 1147079 - [abrt] shotwell: __strcat_sse2_unaligned(): shotwell-video-thumbnailer killed by SIGSEGV
Summary: [abrt] shotwell: __strcat_sse2_unaligned(): shotwell-video-thumbnailer killed...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: shotwell
Version: 21
Hardware: x86_64
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Matthias Clasen
QA Contact: Fedora Extras Quality Assurance
URL: https://retrace.fedoraproject.org/faf...
Whiteboard: abrt_hash:0358b98cb69172c354d972c1ca8...
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-09-26 20:23 UTC by Glenn Johnson
Modified: 2015-06-23 09:09 UTC (History)
34 users (show)

Fixed In Version: shotwell-0.20.2-3.fc21
Clone Of:
Environment:
Last Closed: 2015-06-23 09:09:04 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
File: backtrace (66.03 KB, text/plain)
2014-09-26 20:23 UTC, Glenn Johnson
no flags Details
File: cgroup (190 bytes, text/plain)
2014-09-26 20:23 UTC, Glenn Johnson
no flags Details
File: core_backtrace (19.16 KB, text/plain)
2014-09-26 20:23 UTC, Glenn Johnson
no flags Details
File: dso_list (8.33 KB, text/plain)
2014-09-26 20:23 UTC, Glenn Johnson
no flags Details
File: environ (2.11 KB, text/plain)
2014-09-26 20:23 UTC, Glenn Johnson
no flags Details
File: exploitable (82 bytes, text/plain)
2014-09-26 20:23 UTC, Glenn Johnson
no flags Details
File: limits (1.29 KB, text/plain)
2014-09-26 20:23 UTC, Glenn Johnson
no flags Details
File: maps (39.93 KB, text/plain)
2014-09-26 20:23 UTC, Glenn Johnson
no flags Details
File: open_fds (441 bytes, text/plain)
2014-09-26 20:23 UTC, Glenn Johnson
no flags Details
File: proc_pid_status (947 bytes, text/plain)
2014-09-26 20:23 UTC, Glenn Johnson
no flags Details
File: var_log_messages (221 bytes, text/plain)
2014-09-26 20:23 UTC, Glenn Johnson
no flags Details
patches to upgrade from shotwell 0.20 to 0.22 (3.68 MB, patch)
2015-05-30 16:31 UTC, R Mercado
no flags Details | Diff
upstream commit 22902f6 (2.33 KB, patch)
2015-05-31 23:23 UTC, R Mercado
no flags Details | Diff
upstream commit 6a35d6 (1.01 KB, patch)
2015-05-31 23:23 UTC, R Mercado
no flags Details | Diff
upstream commit 4d088d7b (4.90 KB, patch)
2015-05-31 23:24 UTC, R Mercado
no flags Details | Diff

Description Glenn Johnson 2014-09-26 20:23:27 UTC
Version-Release number of selected component:
shotwell-0.19.0-2.fc21

Additional info:
reporter:       libreport-2.2.3
backtrace_rating: 4
cmdline:        /usr/libexec/shotwell/shotwell-video-thumbnailer /mnt/terra/freeagent_backup/photos/2009/september/Nascar/mvi8140.avi
crash_function: __strcat_sse2_unaligned
executable:     /usr/libexec/shotwell/shotwell-video-thumbnailer
kernel:         3.16.3-300.fc21.x86_64
runlevel:       N 5
type:           CCpp
uid:            1000

Truncated backtrace:
Thread no. 1 (10 frames)
 #0 __strcat_sse2_unaligned at ../sysdeps/x86_64/multiarch/strcpy-sse2-unaligned.S:169
 #1 memcpy at /usr/include/bits/string3.h:51
 #2 png_write_row at pngwrite.c:820
 #3 png_write_rows at pngwrite.c:572
 #4 real_save_png at io-png.c:1040
 #5 gdk_pixbuf__png_image_save_to_callback at io-png.c:1080
 #6 gdk_pixbuf_real_save_to_callback at gdk-pixbuf-io.c:2300
 #7 gdk_pixbuf_save_to_callbackv at gdk-pixbuf-io.c:2661
 #8 gdk_pixbuf_save_to_bufferv at gdk-pixbuf-io.c:2807
 #9 gdk_pixbuf_save_to_buffer at gdk-pixbuf-io.c:2719

Comment 1 Glenn Johnson 2014-09-26 20:23:29 UTC
Created attachment 941704 [details]
File: backtrace

Comment 2 Glenn Johnson 2014-09-26 20:23:30 UTC
Created attachment 941705 [details]
File: cgroup

Comment 3 Glenn Johnson 2014-09-26 20:23:31 UTC
Created attachment 941706 [details]
File: core_backtrace

Comment 4 Glenn Johnson 2014-09-26 20:23:32 UTC
Created attachment 941707 [details]
File: dso_list

Comment 5 Glenn Johnson 2014-09-26 20:23:33 UTC
Created attachment 941708 [details]
File: environ

Comment 6 Glenn Johnson 2014-09-26 20:23:34 UTC
Created attachment 941709 [details]
File: exploitable

Comment 7 Glenn Johnson 2014-09-26 20:23:34 UTC
Created attachment 941710 [details]
File: limits

Comment 8 Glenn Johnson 2014-09-26 20:23:35 UTC
Created attachment 941711 [details]
File: maps

Comment 9 Glenn Johnson 2014-09-26 20:23:36 UTC
Created attachment 941712 [details]
File: open_fds

Comment 10 Glenn Johnson 2014-09-26 20:23:37 UTC
Created attachment 941713 [details]
File: proc_pid_status

Comment 11 Glenn Johnson 2014-09-26 20:23:38 UTC
Created attachment 941714 [details]
File: var_log_messages

Comment 12 Ben Beasley 2015-01-10 20:16:55 UTC
This happened right after I started an import from a camera SD card.

Comment 13 Marek Novotny 2015-03-08 11:13:55 UTC
I experience this failure too, after importing from SD card in Shotwell.

Comment 14 Bruno Martins 2015-04-08 22:51:01 UTC
I've experienced this bug while importing from my home folder.

Comment 15 R Mercado 2015-05-14 07:33:26 UTC
Upstream have fixed this problem.


I tested the rpm with upstream fixes as prepared for Fedora 22 (0.22.0-4.fc22)

https://apps.fedoraproject.org/packages/shotwell/builds/


The source rpm is copied at this location

https://www.dropbox.com/s/8zvzipm731jkb6u/shotwell-0.22.0-4.fc21.src.rpm

Comment 16 Peter Robinson 2015-05-28 13:07:07 UTC
(In reply to r_mercado from comment #15)
> Upstream have fixed this problem.

Got a link to the patch(es) needed so we can push this fix?

Comment 17 R Mercado 2015-05-28 17:52:13 UTC
(In reply to Peter Robinson from comment #16)
> Got a link to the patch(es) needed so we can push this fix?

No. I don't have a link to the patches. Upstream have it in a git repository.

This bug is moot, since an upgrade to Fedora 22 will upgrade from shotwell-0.19 to shotwell-0.22 pulling in the fixes.

Comment 18 Peter Robinson 2015-05-29 11:32:35 UTC
(In reply to r_mercado from comment #17)
> (In reply to Peter Robinson from comment #16)
> > Got a link to the patch(es) needed so we can push this fix?
> 
> No. I don't have a link to the patches. Upstream have it in a git repository.

Do you have details of the upstream commit then?

> This bug is moot, since an upgrade to Fedora 22 will upgrade from
> shotwell-0.19 to shotwell-0.22 pulling in the fixes.

No it's not moot, this is a bug against F-21 which has 7 months more support and there are people that are clearly hitting this that might not be ready or able to move straight to F-22.

Comment 19 R Mercado 2015-05-30 16:31:52 UTC
Created attachment 1032521 [details]
patches to upgrade from shotwell 0.20 to 0.22

The location of the repository is

git://git.gnome.org/shotwell

I downloaded the whole repo and compared the output of versions 0.22
(5e53cc99066516f6d30669866d0516e937a28578) and 0.20.2
(6a5194e2f4bd99ce18d169f2212399c8ec041437) The result is attached in
file shotwell-upgrade-0.20-to-0.22.patch

I also verified that the git repos match the sources in the .src.rpm
files for shotwell at
https://dl.fedoraproject.org/pub/fedora/linux/updates/22/SRPMS and
https://dl.fedoraproject.org/pub/fedora/linux/updates/21/SRPMS

The notes in the NEWS file explain the changes
Shotwell 0.22.0 - 23 March 2015
-------------------------------

  * Improved metadata support for Facebook upload (#742163)
  * Saved search now properly supports text is-set, not-set (#743924, #743923)
  * Import title, exposure time when available
  * User interface improvements (#744291, #719284, more)
  * Sort photos by filename (#717878)
  * Updated translations


Shotwell 0.21.0 - 2 February 2015
---------------------------------

  * User interface refresh (#742326, #742443)
  * Fix Shotwell Viewer .desktop file (#743799)
  * Natural sorting of photo titles (#717960)
  * Improved video thumbnailer (#739396, #741821)
  * Updated translations

Comment 20 Thomas Moschny 2015-05-30 17:16:38 UTC
I am sorry, but we will not update Shotwell to 0.22 in Fedora 21.

That is simply against Fedora policies. We did this once, and got a lot of bad feedback.

What we can do though (and I think this is what Peter was asking for) is to go through the upstream commits, try to find commits fixing the issue at hand, and cherry-pick them onto the current package.

If I find time, I can try to do that, unless someone else beats me.

Meanwhile, you can try to use builds from my side-repo:
http://copr.fedoraproject.org/coprs/thm/shotwell/

It should have a Shotwell 0.22 build for Fedora 21.

Comment 21 R Mercado 2015-05-31 23:23:05 UTC
Created attachment 1033010 [details]
upstream commit 22902f6

Comment 22 R Mercado 2015-05-31 23:23:43 UTC
Created attachment 1033011 [details]
upstream commit 6a35d6

Comment 23 R Mercado 2015-05-31 23:24:13 UTC
Created attachment 1033012 [details]
upstream commit 4d088d7b

Comment 24 R Mercado 2015-05-31 23:24:41 UTC
(In reply to Thomas Moschny from comment #20)
> I am sorry, but we will not update Shotwell to 0.22 in Fedora 21.
> 
okay.
I tracked down the gnome bugzilla entries https://bugzilla.gnome.org/
* Improved video thumbnailer (#739396, #741821)

bugzilla 739396: Jim Nelson 2014-12-30 22:07:58 UTC
mentions commit hash: 22902f6c831a7bc2f4a228f04c394421137c4061
also refers to an old bug 718989
fixed with commit hash: 4d088d7bfd5dcf82d8f301667b3856764e7489ad
bugzilla 741821:  Jim Nelson 2014-12-22 21:24:47 UTC
commit hash: 6a35d6

The chrogit checkout shotwell-0.20.2nological order of the relevant commits is
22902f6, 6a35d6 and 4d088d7b

the steps below show my test of the generation of a video thumbnail that previously failed

git checkout shotwell-0.20.2
git checkout -b fedora21-fix
git cherry-pick 22902f6 6a35d6 4d088d7b
./configure
make
./thumbnailer/shotwell-video-thumbnailer /home/polka/Pictures/2010/07/18/DSCF2919.AVI > thumb.png

I attach the upstream commits corresponding to 22902f6, 6a35d6 and 4d088d7b.
The file names are 01-commit-49507e4 02-commit-64b6ab8 03-commit-ca533f9

Comment 25 R Mercado 2015-05-31 23:26:28 UTC
Apologies. In comment 24, the section
"The chrogit checkout shotwell-0.20.2nological order of the relevant commits is
22902f6, 6a35d6 and 4d088d7b"

should read

The chronological order of the relevant commits is
22902f6, 6a35d6 and 4d088d7b

Comment 26 Peter Robinson 2015-06-01 15:49:52 UTC
(In reply to Thomas Moschny from comment #20)
> I am sorry, but we will not update Shotwell to 0.22 in Fedora 21.

Not asking for that, asking for a particular patch set to be pulled into 0.20 to fix this specific crash.

> That is simply against Fedora policies. We did this once, and got a lot of
> bad feedback.
> 
> What we can do though (and I think this is what Peter was asking for) is to
> go through the upstream commits, try to find commits fixing the issue at
> hand, and cherry-pick them onto the current package.

Yes. That's exactly what I want :-)

Comment 27 R Mercado 2015-06-08 23:48:23 UTC
Hello

I am posting because I uploaded two things related to this bug:

An updated spec file including the patches I posted last month.
https://kiso49j.fedorapeople.org/shotwell.spec

A source rpm I built on my fedora 22 machine
https://kiso49j.fedorapeople.org/shotwell-0.20.2-2.fc22.src.rpm

I will attempt in the next few days to get copr to generate a repository as in comment #20

Thanks.

Comment 28 Fedora Update System 2015-06-11 18:50:35 UTC
shotwell-0.20.2-3.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/shotwell-0.20.2-3.fc21

Comment 29 Fedora Update System 2015-06-13 06:33:51 UTC
Package shotwell-0.20.2-3.fc21:
* should fix your issue,
* was pushed to the Fedora 21 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing shotwell-0.20.2-3.fc21'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2015-9836/shotwell-0.20.2-3.fc21
then log in and leave karma (feedback).

Comment 30 Fedora Update System 2015-06-23 09:09:04 UTC
shotwell-0.20.2-3.fc21 has been pushed to the Fedora 21 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.