Video thumbnails always fail in gallery-app

[Jean-Baptiste Lallement]

com.ubuntu.gallery 2.9.1.1049

Video thumbnails always fail to generate in the gallery app and the thumbnail crash with bug 1364475 or bug 1356331. The thumbnail is correctly generated in the video scope for the same video.
It also affects video recorded by the camera

TEST CASE:
1. Download http://samplemedia.linaro.org/H264/tears_of_steel_1080p_H264_24fps.mov
2. Copy it to /home/phablet/Videos with MTP (nautilus or gvfs-copy)
3. Open the gallery-app
4. Locate the video in the gallery and wait until the thumbnail is generated

ACTUAL RESULT
The thumbnail shows the generic video icon and the thumbnailer crashes

[Bill Filler (bfiller)]

I am able to reproduce this easily on krillin using image 16. However thumbnails are generated correctly in the gallery on mako, so I don't believe the issue is related to the gallery app itself but rather something on the system. Also, I don't think it's a gstreamer related issue, as the thumbail is correctly displayed in the video scope so if it was a gstreamer issue it would fail there as well.

It could be a security/apparmor related issue. I see these DENIALS in /var/log/syslog when launching the gallery when there are video thumbnails:

Sep 4 23:28:54 ubuntu-phablet kernel: [ 8210.366923] (1)[2347:gallery-app]type=1400 audit(1409887734.744:487): apparmor="DENIED" operation="chmod" profile="com.ubuntu.gallery_gallery_2.9.1.1049" name="/run/user/32011/" pid=2347 comm="gallery-app" requested_mask="w" denied_mask="w" fsuid=32011 ouid=32011
Sep 4 23:28:56 ubuntu-phablet kernel: [ 8212.259456] (2)[2612:CodecLooper]type=1400 audit(1409887736.644:488): apparmor="DENIED" operation="open" profile="com.ubuntu.gallery_gallery_2.9.1.1049" name="/android/system/etc/mtk_omx_core.cfg" pid=2612 comm="CodecLooper" requested_mask="r" denied_mask="r" fsuid=32011 ouid=0

[Bill Filler (bfiller)]

Confirmed that without any videos, I only get this denial:

Sep 4 23:33:33 ubuntu-phablet kernel: [ 8488.607995] (0)[4407:gallery-app]type=1400 audit(1409888012.994:493): apparmor="DENIED" operation="chmod" profile="com.ubuntu.gallery_gallery_2.9.1.1049" name="/run/user/32011/" pid=4407 comm="gallery-app" requested_mask="w" denied_mask="w" fsuid=32011 ouid=32011

And with each video, I get one of these denials for each:

Sep 4 23:36:40 ubuntu-phablet kernel: [ 8676.502056] (0)[5866:CodecLooper]type=1400 audit(1409888200.884:494): apparmor="DENIED" operation="open" profile="com.ubuntu.gallery_gallery_2.9.1.1049" name="/android/system/etc/mtk_omx_core.cfg" pid=5866 comm="CodecLooper" requested_mask="r" denied_mask="r" fsuid=32011 ouid=0

[Jamie Strandboge (jdstrand)]

The first denial is bug #1359831 and won't be fixed in apparmor-easyprof-ubuntu. The second is krillin specific and discussed in bug #1365509 (also not fixed in apparmor-easyprof-ubuntu).

[Jamie Strandboge (jdstrand)]

I'm going to mark this a duplicate of bug #1365509 since that seems to be the root of the problem-- krillin needs some hardware specific rules that should be shipped via another means.

[Mahara Bot (dev-mahara)]

Patch for "master" branch: https://reviews.mahara.org/9820

[Mahara Bot (dev-mahara)]

Patch for "master" branch: https://reviews.mahara.org/9847

[Mahara Bot (dev-mahara)]

Patch for "master" branch: https://reviews.mahara.org/9851

[Mahara Bot (dev-mahara)]

Patch for "master" branch: https://reviews.mahara.org/9874

[Mahara Bot (dev-mahara)]

Patch for "master" branch: https://reviews.mahara.org/9877

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/9847
Committed: https://git.mahara.org/mahara/mahara/commit/06e88aafbd9a3374e05fd9d52fa35a18450e3b0f
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master

commit 06e88aafbd9a3374e05fd9d52fa35a18450e3b0f
Author: Cecilia Vela Gurovic <email address hidden>
Date: Wed Apr 24 16:00:13 2019 +1200

Bug 1365503: SE editor- save statuses only when creating a new framework

behatnotneeded

Change-Id: I8a8870d85cc44db5ed0c5382527e3d29acbff31c

[Mahara Bot (dev-mahara)]

Patch for "19.04_STABLE" branch: https://reviews.mahara.org/9885

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/9885
Committed: https://git.mahara.org/mahara/mahara/commit/a834842b04dd6e432a5f8a7e037bb4377a37a744
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: 19.04_STABLE

commit a834842b04dd6e432a5f8a7e037bb4377a37a744
Author: Cecilia Vela Gurovic <email address hidden>
Date: Wed Apr 24 16:00:13 2019 +1200

Bug 1365503: SE editor- save statuses only when creating a new framework

behatnotneeded

Change-Id: I8a8870d85cc44db5ed0c5382527e3d29acbff31c
(cherry picked from commit 06e88aafbd9a3374e05fd9d52fa35a18450e3b0f)

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/9820
Committed: https://git.mahara.org/mahara/mahara/commit/78fd860a5bd64f62d536d6c28e724a24b58f60ce
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master

commit 78fd860a5bd64f62d536d6c28e724a24b58f60ce
Author: Robert Lyon <email address hidden>
Date: Wed Apr 24 07:58:50 2019 +1200

Bug 1365503: SmartEvidence editor - add std dropdown

+ adding standardid dropdown in standard element section
+ change elementid when changing standard id
+ save standard id from the dropdown when saving form WIP
+ update parent element id when changing standardid to show only
elementids under that standard
+ hide own elementid when populating parent element id
dropdown

behatnotneeded

Change-Id: I0dbfe8a49d6b967d6c03ca0568c8532be54e5d9d

[Mahara Bot (dev-mahara)]

Patch for "19.04_STABLE" branch: https://reviews.mahara.org/9897

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/9897
Committed: https://git.mahara.org/mahara/mahara/commit/ee8759c18c32bddf835d58dc0a03e51aa81c8ff1
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: 19.04_STABLE

commit ee8759c18c32bddf835d58dc0a03e51aa81c8ff1
Author: Robert Lyon <email address hidden>
Date: Wed Apr 24 07:58:50 2019 +1200

Bug 1365503: SmartEvidence editor - add std dropdown

+ adding standardid dropdown in standard element section
+ change elementid when changing standard id
+ save standard id from the dropdown when saving form WIP
+ update parent element id when changing standardid to show only
elementids under that standard
+ hide own elementid when populating parent element id
dropdown

behatnotneeded

Change-Id: I0dbfe8a49d6b967d6c03ca0568c8532be54e5d9d
(cherry picked from commit 78fd860a5bd64f62d536d6c28e724a24b58f60ce)

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/9851
Committed: https://git.mahara.org/mahara/mahara/commit/d71c7bdf32961f2b0eab017b547a8c156126f7df
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master

commit d71c7bdf32961f2b0eab017b547a8c156126f7df
Author: Cecilia Vela Gurovic <email address hidden>
Date: Wed Apr 24 17:29:01 2019 +1200

Bug 1365503: SE editor- change standard dropdown format

to be 'id' - 'shotname'

behatnotneeded

Change-Id: I1332d99d09f6e7264bcb6bb1c32563dc7c5c1c3d

[Mahara Bot (dev-mahara)]

Patch for "19.04_STABLE" branch: https://reviews.mahara.org/9898

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/9898
Committed: https://git.mahara.org/mahara/mahara/commit/124516af90e89c2ba8b18cf207c20ab34e1c1a33
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: 19.04_STABLE

commit 124516af90e89c2ba8b18cf207c20ab34e1c1a33
Author: Cecilia Vela Gurovic <email address hidden>
Date: Wed Apr 24 17:29:01 2019 +1200

Bug 1365503: SE editor- change standard dropdown format

to be 'id' - 'shotname'

behatnotneeded

Change-Id: I1332d99d09f6e7264bcb6bb1c32563dc7c5c1c3d
(cherry picked from commit d71c7bdf32961f2b0eab017b547a8c156126f7df)

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/9874
Committed: https://git.mahara.org/mahara/mahara/commit/0104923900561b8933ede157f0dca6e82c09bac2
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master

commit 0104923900561b8933ede157f0dca6e82c09bac2
Author: Cecilia Vela Gurovic <email address hidden>
Date: Fri Apr 26 15:24:07 2019 +1200

Bug 1365503: SE editor - hide standardid field in standard section

behatnotneeded

Change-Id: I0275795ceff749e91b1043cbd07b047d62f6843a

[Mahara Bot (dev-mahara)]

Patch for "19.04_STABLE" branch: https://reviews.mahara.org/9899

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/9899
Committed: https://git.mahara.org/mahara/mahara/commit/e3991a38f905d9e74996899fe6ecb3964d1f05df
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: 19.04_STABLE

commit e3991a38f905d9e74996899fe6ecb3964d1f05df
Author: Cecilia Vela Gurovic <email address hidden>
Date: Fri Apr 26 15:24:07 2019 +1200

Bug 1365503: SE editor - hide standardid field in standard section

behatnotneeded

Change-Id: I0275795ceff749e91b1043cbd07b047d62f6843a
(cherry picked from commit 0104923900561b8933ede157f0dca6e82c09bac2)

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/9877
Committed: https://git.mahara.org/mahara/mahara/commit/77a571a49cf269a9eec8db734a3c9b577431cfe9
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master

commit 77a571a49cf269a9eec8db734a3c9b577431cfe9
Author: Cecilia Vela Gurovic <email address hidden>
Date: Fri Apr 26 17:38:17 2019 +1200

Bug 1365503: SE editor - adding saved framework in Edit dropdown only once

behatnotneeded

Change-Id: I79945991e4a502fd82165c5005cbbb6d8299c37e

[Mahara Bot (dev-mahara)]

Patch for "19.04_STABLE" branch: https://reviews.mahara.org/9900

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/9900
Committed: https://git.mahara.org/mahara/mahara/commit/7b50b3851bc44079edc02a4e0cb9c3a261f7a74c
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: 19.04_STABLE

commit 7b50b3851bc44079edc02a4e0cb9c3a261f7a74c
Author: Cecilia Vela Gurovic <email address hidden>
Date: Fri Apr 26 17:38:17 2019 +1200

Bug 1365503: SE editor - adding saved framework in Edit dropdown only once

behatnotneeded

Change-Id: I79945991e4a502fd82165c5005cbbb6d8299c37e
(cherry picked from commit 77a571a49cf269a9eec8db734a3c9b577431cfe9)