Skip to content
This repository has been archived by the owner on Sep 6, 2019. It is now read-only.

[BUG] Upgrading from 1.11.11 to 1.99.11 includes restricted permissions #1165

Closed
jesselai opened this issue Jan 26, 2014 · 4 comments
Closed
Labels

Comments

@jesselai
Copy link

In the upgrade process from 1.11.11 to 1.99.11 there were some migration issues. On apps that have Location or System restricted, there are some of the red potentially dangerous items that were automatically checked after the upgrade that I don't think should be. I listed those below.

Location:

  • getProviders
  • isProviderEnabled

System:

  • getPackagesHoldingPermissions
  • queryIntentContentProviders

I reported in the forum, but mostly am creating this issue here so it doesn't get lost and also get a notification on update.

@M66B M66B closed this as completed in a1a2447 Jan 26, 2014
@jesselai
Copy link
Author

Hi Marcel, I tried upgrading again from 1.11.11 to 1.99.14 and 1.99.16, and it still had the problem described in the original report.

Edit: Also, just checked the commit and noticed that it attempted to add a version check for the Location settings. Was there a different commit that was for the System ones?

M66B pushed a commit that referenced this issue Jan 28, 2014
@M66B
Copy link
Owner

M66B commented Jan 28, 2014

Indeed, defining the version number from for the functions is not enough, the upgrade process should be run after the migrate to set it right.

This should be fixed in this version: http://d-h.st/nGX

I would be grateful if you test it and let me know if it is really fixed now.

@jesselai
Copy link
Author

I just tested the version that you linked in an upgrade from 1.11.11. It worked flawlessly. I noticed that there is now an upgrading dialog which is a nice touch. Thanks for looking into it again. I checked several apps to ensure that it was correct. I also double check some apps that had Internet restricted before the upgrade, and those settings looked correct as well in terms of the new restrictions.

@M66B
Copy link
Owner

M66B commented Jan 28, 2014

Thank your very much for testing and reporting back!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants