please move click sound out of qtubuntu-camera

Bug #1369512 reported by Omer Akram on 2014-09-15
32
This bug affects 6 people
Affects Status Importance Assigned to Milestone
Media Hub
Invalid
Undecided
Unassigned
apparmor-easyprof-ubuntu (Ubuntu)
Critical
Jamie Strandboge
qtubuntu-camera (Ubuntu)
High
Jim Hodapp

Bug Description

RTM image 39 on mako
media-hub 1.0.0+14.10.20140908~rtm-0ubuntu1

The Ubuntu Store have three torch apps, I installed all but none of them starts. The upstart log shows something bad re: apparmor/media-hub

Omer Akram (om26er) wrote :
Dave Morley (davmor2) wrote :

also affects krillin

Changed in media-hub:
status: New → Confirmed
Jamie Strandboge (jdstrand) wrote :

Here is the relevant output:
"Failed to start a new media-hub player session: org.freedesktop.DBus.Error.AccessDenied: An AppArmor policy prevents this sender from sending this message to this recipient, 0 matched rules; type="method_call", sender=":1.309" (uid=32011 pid=12412 comm="/usr/lib/arm-linux-gnueabihf/qt5/bin/qmlscene $@ u") interface="core.ubuntu.media.Service" member="CreateSession" error name="(unset)" requested_reply="0" destination="core.ubuntu.media.Service" (uid=32011 pid=2865 comm="media-hub-server ")
Failed to create a new media player backend. Video playback will not function."

The app is trying to play media of some sort and the media-hub is used for playback. As such, these apps need to use the appropriate policy group in their security json. Eg, audio or video.

Jamie Strandboge (jdstrand) wrote :

Note: this looks to be a bug in the affected apps, not media-hub or apparmor policy.

Jamie Strandboge (jdstrand) wrote :

After discussion on IRC, this is because qtubuntu-camera implements the 'click' camera sound. The proper fix is to move this outside of qtubuntu-camera and apps can optionally make the click sound using Audio, which only then would bring in media-hub, and only then require the audio policy group. Since recording with the camera is moving to media-hub, I will be adding to the camera policy group a dbus rule to allow talking to it. This will workaround this bug for now.

summary: - None of the torch app starts
+ please move click sound out of qtubuntu-camera
Changed in qtubuntu-camera (Ubuntu):
importance: Undecided → Critical
importance: Critical → High
Changed in apparmor-easyprof-ubuntu (Ubuntu):
importance: Undecided → Critical
Changed in qtubuntu-camera (Ubuntu):
importance: High → Undecided
Changed in apparmor-easyprof-ubuntu (Ubuntu):
status: New → In Progress
Changed in qtubuntu-camera (Ubuntu):
status: New → Triaged
Changed in apparmor-easyprof-ubuntu (Ubuntu):
assignee: nobody → Jamie Strandboge (jdstrand)
Changed in qtubuntu-camera (Ubuntu):
assignee: nobody → Jim Hodapp (jhodapp)
tags: added: touch-2014-09-25
tags: added: rtm14
Jamie Strandboge (jdstrand) wrote :

In testing uTorch, I noticed it also doesn't specify the camera policy group. It will need to do so after this bug is marked fixed in apparmor-easyprof-ubuntu.

Jamie Strandboge (jdstrand) wrote :

In testing tagger, it does specify the camera policy group and works correctly with apparmor-easyprof-ubuntu 1.2.24.

Jim Hodapp (jhodapp) on 2014-09-17
Changed in media-hub:
status: Confirmed → Invalid
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package apparmor-easyprof-ubuntu - 1.2.24

---------------
apparmor-easyprof-ubuntu (1.2.24) utopic; urgency=medium

  * ubuntu/camera: allow DBus communications with media-hub (LP: #1369512)
  * ubuntu/*: drop redundnat 'ptrace (read) peer=@{profile_name}' since we
    include it in the base abstraction now
 -- Jamie Strandboge <email address hidden> Tue, 16 Sep 2014 08:48:37 -0500

Changed in apparmor-easyprof-ubuntu (Ubuntu):
status: In Progress → Fix Released
Pat McGowan (pat-mcgowan) wrote :

Critical fix landed, the rest is cleanup, marked for 10-9

Changed in qtubuntu-camera (Ubuntu):
importance: Undecided → High
tags: added: touch-2014-10-09
removed: touch-2014-09-25
Michael Frey (mfrey) on 2014-09-24
Changed in qtubuntu-camera (Ubuntu):
status: Triaged → Fix Released
Justin McPherson (justinmcp) wrote :

There are jurisdictions where it is mandatory for any "camera app" to make an audible sound on picture taking/begin video record. We should be mindful of this requirement if applications become responsible for making the click sound.

Agreed, although I would add that if any app really needs such sound, the
app author should be very aware of adding this to their app right?

Jim

On Mon, Oct 6, 2014 at 9:13 PM, Justin McPherson <
<email address hidden>> wrote:

> There are jurisdictions where it is mandatory for any "camera app" to
> make an audible sound on picture taking/begin video record. We should be
> mindful of this requirement if applications become responsible for
> making the click sound.
>
> --
> You received this bug notification because you are a bug assignee.
> https://bugs.launchpad.net/bugs/1369512
>
> Title:
> please move click sound out of qtubuntu-camera
>
> Status in Media Hub:
> Invalid
> Status in “apparmor-easyprof-ubuntu” package in Ubuntu:
> Fix Released
> Status in “qtubuntu-camera” package in Ubuntu:
> Fix Released
>
> Bug description:
> RTM image 39 on mako
> media-hub 1.0.0+14.10.20140908~rtm-0ubuntu1
>
> The Ubuntu Store have three torch apps, I installed all but none of
> them starts. The upstart log shows something bad re: apparmor/media-
> hub
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/media-hub/+bug/1369512/+subscriptions
>

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Bug attachments