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

Crash during account deletion #1544

Closed
stalker314314 opened this issue Sep 13, 2017 · 8 comments
Closed

Crash during account deletion #1544

stalker314314 opened this issue Sep 13, 2017 · 8 comments

Comments

@stalker314314
Copy link
Contributor

Actual behaviour

Deleting account results in crash of app

Expected behaviour

No crash

Steps to reproduce

  1. Create account
  2. Auto upload some files (not sure about this part though)
  3. Delete account

Environment data

Android version:

Device model:

Stock or customized system:
LineageOS

Nextcloud app version:
master

Nextcloud server version:
12.0.2RC

Logs

`09-13 21:14:58.895 4562-4686/com.nextcloud.android.beta D/FileContentProvider: Removing FILE 00048297ocgghykd5h22

09-13 21:14:58.898 4562-4686/com.nextcloud.android.beta D/FileContentProvider: DB-Error removing file! java.lang.IndexOutOfBoundsException
at android.net.Uri$PathSegments.get(Uri.java:989)
at android.net.Uri$PathSegments.get(Uri.java:987)
at

com.owncloud.android.providers.FileContentProvider.delete(FileContentProvider.java:122)
at

com.owncloud.android.providers.FileContentProvider.delete(FileContentProvider.java:97)
at

android.content.ContentProvider$Transport.delete(ContentProvider.java:339)
at

android.content.ContentResolver.delete(ContentResolver.java:1332)
at

com.owncloud.android.datamodel.FileDataStorageManager.deleteAllFiles(FileDataStorageManager.java:2185)
at

com.owncloud.android.jobs.AccountRemovalJob.onRunJob(AccountRemovalJob.java:75)
at

com.evernote.android.job.Job.runJob(Job.java:123)
at

com.evernote.android.job.JobExecutor$JobCallable.runJob(JobExecutor.java:165)
at

com.evernote.android.job.JobExecutor$JobCallable.call(JobExecutor.java:150)
at

com.evernote.android.job.JobExecutor$JobCallable.call(JobExecutor.java:133)
at

java.util.concurrent.FutureTask.run(FutureTask.java:237)
at

java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1113)
at

java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:588)
at java.lang.Thread.run(Thread.java:818)
09-13 21:14:58.909 4562-4686/com.nextcloud.android.beta I/JobExecutor: Finished job{id=39, finished=true, result=SUCCESS, canceled=false, periodic=false, class=AccountRemovalJob, tag=AccountRemovalJob}`

@AndyScherzinger
Copy link
Member

@mario any chance you can have a look at this? While looking at the logs /com.nextcloud.android.beta D/FileContentProvider tells me it must be the dev branch not the master 😕

@mario
Copy link
Contributor

mario commented Sep 13, 2017

@AndyScherzinger I'd assume the same, but @tobiasKaminsky will have to as this is his work + absolutely busy on other things :(

@stalker314314
Copy link
Contributor Author

@AndyScherzinger Ops, you are right, I am switching between multiple versions and got them confused. Sorry!

@AndyScherzinger
Copy link
Member

No worries @stalker314314 :) Does this also happen on the master branch?

@tobiasKaminsky tobiasKaminsky added the needs info Waiting for info from user(s). Issues with this label will auto-stale. label Dec 20, 2017
@tobiasKaminsky
Copy link
Member

Is this still happening for you?

@stalker314314
Copy link
Contributor Author

TBH, I didn't recently tried to delete account while there was upload in progress:) As far as I am concerned, you can tentatively close issue, but I still think there is some concurrency issue lurking underneath uploading logic (based by stack trace)

@tobiasKaminsky tobiasKaminsky added approved and removed needs info Waiting for info from user(s). Issues with this label will auto-stale. labels Dec 21, 2017
@tobiasKaminsky
Copy link
Member

tobiasKaminsky commented Dec 21, 2017

Thank you, I overread the part that you were removing the account during an upload.

Steps to reproduce:

  • upload a big file
  • meanwhile delete the same account
  • crash

On account removal job: remove account at very last position!

@tobiasKaminsky
Copy link
Member

Retested it now, this seems to be fixed.
If not, please re-open.

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

4 participants