Merge lp://staging/~jamesh/thumbnailer/use-aa-query-label into lp://staging/thumbnailer/devel
Proposed by
James Henstridge
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Approved by: | Michi Henning | ||||||||
Approved revision: | 226 | ||||||||
Merged at revision: | 218 | ||||||||
Proposed branch: | lp://staging/~jamesh/thumbnailer/use-aa-query-label | ||||||||
Merge into: | lp://staging/thumbnailer/devel | ||||||||
Diff against target: |
1060 lines (+321/-200) 22 files modified
CMakeLists.txt (+1/-0) include/internal/check_access.h (+42/-0) include/internal/thumbnailer.h (+4/-1) plugins/Ubuntu/Thumbnailer.0.1/CMakeLists.txt (+0/-1) plugins/Ubuntu/Thumbnailer.0.1/thumbnailgenerator.cpp (+1/-15) src/CMakeLists.txt (+2/-0) src/check_access.cpp (+83/-0) src/service/CMakeLists.txt (+1/-3) src/service/dbusinterface.cpp (+1/-2) src/service/dbusinterface.h (+0/-1) src/service/dbusinterface.xml (+1/-2) src/service/handler.cpp (+1/-1) src/thumbnailer-admin/get_local_thumbnail.cpp (+1/-13) src/thumbnailer.cpp (+41/-26) tests/CMakeLists.txt (+1/-0) tests/check_access/CMakeLists.txt (+3/-0) tests/check_access/check_access_test.cpp (+64/-0) tests/dbus/dbus_test.cpp (+5/-42) tests/qml/tst_albumart.qml (+4/-12) tests/testsetup.h.in (+1/-1) tests/thumbnailer-admin/thumbnailer-admin_test.cpp (+2/-2) tests/thumbnailer/thumbnailer_test.cpp (+62/-78) |
||||||||
To merge this branch: | bzr merge lp://staging/~jamesh/thumbnailer/use-aa-query-label | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Michi Henning (community) | Approve | ||
Review via email:
|
Commit message
Remove the filename_fd argument to the GetThumbnail() D-Bus method, instead using aa_query_label() to check whether the caller has access to the relevant file.
Description of the change
Update the GetThumbnail dbus method to remove the filename_fd argument. Instead, use aa_query_label() to check whether the client has access to the target file.
We were already canonicalising the path, so shouldn't have trouble getting tricked by symlinks.
I'll provide some more concrete ways to test this on the phone once I've got ARM binaries.
To post a comment you must log in.
FAILED: Continuous integration, rev:222 /code.launchpad .net/~jamesh/ thumbnailer/ use-aa- query-label/ +merge/ 262060/ +edit-commit- message
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https:/
http:// jenkins. qa.ubuntu. com/job/ thumbnailer- devel-ci/ 285/ jenkins. qa.ubuntu. com/job/ thumbnailer- devel-wily- amd64-ci/ 94/console jenkins. qa.ubuntu. com/job/ thumbnailer- devel-wily- armhf-ci/ 96/console jenkins. qa.ubuntu. com/job/ thumbnailer- devel-wily- i386-ci/ 94/console
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/thumbnailer -devel- ci/285/ rebuild
http://