Review: Approve

Looks good to me! We've done similar changes elsewhere already so the proposed 
solution seems field-tested to me. What do others think?

By the way, it looks like abstractions/user-write has the same problem, so 
perhaps you could fix it as well in your branch, for the sake of consistency? 
Now, if I had commit rights I would merge your branch as-is anyway, as it seems 
to be an obvious incremental changes not worth blocking on further possible 
improvements.
-- 
https://code.launchpad.net/~talkless/apparmor/fix_user_download_nonlatin/+merge/326259
Your team AppArmor Developers is requested to review the proposed merge of 
lp:~talkless/apparmor/fix_user_download_nonlatin into lp:apparmor.

-- 
AppArmor mailing list
AppArmor@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to