Bug 1147079

Summary: [abrt] shotwell: __strcat_sse2_unaligned(): shotwell-video-thumbnailer killed by SIGSEGV
Product: [Fedora] Fedora Reporter: Glenn Johnson <glennjohnson57>
Component: shotwellAssignee: Matthias Clasen <mclasen>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 21CC: alst2821, bigant, bruno, code, dhansen, dyocum, jan.public, jon.ingason, laufor, leucir, mariolinux, martin, maurygls, mclasen, metherid, michel, mnovotny, nanopqr, pbrobinson, rds, redhat-bugzilla, richard_maxime, RIKINET71, ruben.fastenau, r.weyland, sly, spoyarek, surkum, svc-bugzilla.redhat.com, tadp, thomas.jethan, thomas.moschny, twaugh, vincent.aaron
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Unspecified   
URL: https://retrace.fedoraproject.org/faf/reports/bthash/e329a1b51b92a6a5c3d63eaa4a05c4bab1dddc56
Whiteboard: abrt_hash:0358b98cb69172c354d972c1ca8cf718814dfbf3
Fixed In Version: shotwell-0.20.2-3.fc21 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-06-23 09:09:04 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: backtrace
none
File: cgroup
none
File: core_backtrace
none
File: dso_list
none
File: environ
none
File: exploitable
none
File: limits
none
File: maps
none
File: open_fds
none
File: proc_pid_status
none
File: var_log_messages
none
patches to upgrade from shotwell 0.20 to 0.22
none
upstream commit 22902f6
none
upstream commit 6a35d6
none
upstream commit 4d088d7b none

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.