Merge lp:~vicamo/media-hub/add-lib64-for-64bit-android into lp:media-hub

Proposed by You-Sheng Yang
Status: Merged
Approved by: Alfonso Sanchez-Beato
Approved revision: 206
Merged at revision: 208
Proposed branch: lp:~vicamo/media-hub/add-lib64-for-64bit-android
Merge into: lp:media-hub
Diff against target: 34 lines (+12/-12)
1 file modified
debian/usr.bin.media-hub-server (+12/-12)
To merge this branch: bzr merge lp:~vicamo/media-hub/add-lib64-for-64bit-android
Reviewer Review Type Date Requested Status
Alfonso Sanchez-Beato Approve
Review via email: mp+303652@code.launchpad.net

Commit message

Add /system/lib64 to apparmor permission rules

Description of the change

Currently we have only /system/lib etched in apparmor permission rules. On arm64, /system/lib64 is required. Example apparmor error:

kernel: [ 189.457372].(0)[7350:media-hub-serve]type=1400 audit(1471933369.640:152): apparmor="DENIED" operation="open" profile="/usr/bin/media-hub-server" name="/android/system/lib64/libmedia_compat_layer.so" pid=7350 comm="media-hub-serve" requested_mask="r" denied_mask="r" fsuid=32011 ouid=0

To post a comment you must log in.
Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/usr.bin.media-hub-server'
--- debian/usr.bin.media-hub-server 2016-01-26 20:13:48 +0000
+++ debian/usr.bin.media-hub-server 2016-08-23 07:03:56 +0000
@@ -53,18 +53,18 @@
53 /usr/lib/@{multiarch}/libhybris/*.so mr,53 /usr/lib/@{multiarch}/libhybris/*.so mr,
54 /{,android/}system/build.prop r,54 /{,android/}system/build.prop r,
55 # These libraries can be in any of:55 # These libraries can be in any of:
56 # /vendor/lib56 # /vendor/lib{,64}
57 # /system/lib57 # /system/lib{,64}
58 # /system/vendor/lib58 # /system/vendor/lib{,64}
59 # /android/vendor/lib59 # /android/vendor/lib{,64}
60 # /android/system/lib60 # /android/system/lib{,64}
61 # /android/system/vendor/lib61 # /android/system/vendor/lib{,64}
62 /{,android/}vendor/lib/** r,62 /{,android/}vendor/lib{,64}/** r,
63 /{,android/}vendor/lib/**.so m,63 /{,android/}vendor/lib{,64}/**.so m,
64 /{,android/}system/lib/** r,64 /{,android/}system/lib{,64}/** r,
65 /{,android/}system/lib/**.so m,65 /{,android/}system/lib{,64}/**.so m,
66 /{,android/}system/vendor/lib/** r,66 /{,android/}system/vendor/lib{,64}/** r,
67 /{,android/}system/vendor/lib/**.so m,67 /{,android/}system/vendor/lib{,64}/**.so m,
6868
69 # attach_disconnected path69 # attach_disconnected path
70 /dev/socket/property_service rw,70 /dev/socket/property_service rw,

Subscribers

People subscribed via source and target branches