Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add previous-owner app for CorpseFinder #663

Closed
Gitoffthelawn opened this issue Dec 25, 2016 · 8 comments
Closed

Add previous-owner app for CorpseFinder #663

Gitoffthelawn opened this issue Dec 25, 2016 · 8 comments

Comments

@Gitoffthelawn
Copy link

CorpseFinder will find items that the user might want to delete, but it can be very challenging to find the previous owner of those files.

Perhaps, in CorpseFinder, list the app that created each file so the user can decide if they are really corpses.

For example, on one device, CorpseFinder is recommending the deletion of .bnr_fifo_rly and .bnr_fifo_req, but neither filename is found when searching this repo. An internet search finds some possible owners for those files, but only SDMaid knows for sure why it is recommending the removal of those files.

@d4rken
Copy link
Member

d4rken commented Dec 25, 2016

SD Maid doesn't have that information for every corpse. For locations such as the root of public storage (sdcard) SD Maid will only list files as corpses if there is a known owner that is not installed, meaning for this location items have to be whitelisted and may only then be checked and suggested for deletion.

In locations such as private data (e.g. /data/data) everything that is unknown is a corpse by default. So if you create a file named "santa-claus" in there, SD Maid will mark it as corpse, because the naming pattern doesn't fit in there, but we don't know who the real owner is.

So for some locations we could only say "Unknown" while for others we could say "Known owners are .... of which none are installed".

On the sdcard clutter database entries are needed to whitelist apps, to make them available for corpse checking and in areas such as private data storage, the clutter database entries are needed to fix such outliers.

Some ROMs just place random files where they don't belong and due to not fitting the naming pattern SD Maid assumes they are corpses, this is probably the case with those .bnr_fifo_rly files. If you can find an app that these belong to though, we add a database entry to fix this.

@Gitoffthelawn
Copy link
Author

Gitoffthelawn commented Dec 25, 2016

Okay, so you are recommending whitelisting them?

BTW, from the research I performed, it looks like they may be created by a backup app that came with the device. Not sure if they are trash or needed.

They are located in /data/user/0. What is triggering CorpseFinder to classify them as corpses?

@d4rken
Copy link
Member

d4rken commented Dec 26, 2016

SD Maid thinks it's a corpse because it doesn't belong there, it doesn't follow naming conventions, .bnr_fifo_rly is not a packagename.

The best solution would to create a database entry for this if we know which app creates them.

@Gitoffthelawn
Copy link
Author

Gitoffthelawn commented Dec 26, 2016

The data in this repo implies it is a common backup tool installed by a device manufacturer.

@d4rken
Copy link
Member

d4rken commented Dec 26, 2016

Is there an LG backup app? Can you search in AppControl just for "backup" and see what comes up?

@Gitoffthelawn
Copy link
Author

On the device with those files, searching within AppControl for "backup" returns 5 results. The relevant one is most likely LG Backup which has the package name of com.lge.bnr.

@d4rken
Copy link
Member

d4rken commented Dec 27, 2016

Oh that was a fruitful code dive 🎆 .

  • .bnr_fifo_rly and .bnr_fifo_req to com.lge.bnr, sounds like a perfect match, added.
  • Corpse details now show ownership information.
  • When checking the displayed ownership information i saw that test entries for some apps showed the same owner multiple times. Due to how the clutter database information is initiated in SD Maid, a later step that stores the data sorted by location (i.e. SDCARD) caused the data to be add multiple times. By changing the datastructure to disallow duplicates, we cleaned up the ownership details and slightly improved performance in some cases 👍.

@d4rken d4rken closed this as completed Dec 27, 2016
@Gitoffthelawn
Copy link
Author

Great job! :)

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

No branches or pull requests

2 participants