[Merge] lp:~thomas-voss/trust-store/fix-1504022 into lp:trust-store/15.04

Thomas Voß thomas.voss at canonical.com
Sat Nov 21 11:42:03 UTC 2015


Hey Tyler,

thanks for the detailed review and your comments. Let me first try to clarify on the use of the localized name in trust prompts. I actually raised the same issue as you, and it turns out that we are "vulnerable" to applications faking their identity in a couple of places throughout the system. First and foremost: Unity itself relies on the .desktop file entry for displaying the name. We might want to fix the default approach for handling localized application names meant for human consumption in snappy, though.

One possible way to resolve the issue is by altering the prompt to show the localized name alongside the actual application id. Ideally, we would have an expendable field "Details" as part of the prompt containing in-depth information about the request, at the very least the actual app ID together with the UID (and the human-readable user name).

(@1:) The function always runs under the actual user. I will however add a check to make sure that the current user id matches the user id of the request. If not, we will return early, and deny the request.

(@2:) Sure, happy to. I mainly added the external code here to guarantee a swift landing. I will do the packaging and carry out the requesting asap, but would like to make sure that a fix for this bug is not blocked by package/MIR review processes.
-- 
https://code.launchpad.net/~thomas-voss/trust-store/fix-1504022/+merge/277266
Your team Ubuntu Phablet Team is subscribed to branch lp:trust-store/15.04.



More information about the Ubuntu-reviews mailing list