FusedLocationApi leaks when requestLocationUpdates is called [37126862]

Fixed

Bug

Status Update

No update yet.

Description

fe...@gmail.com

created issue #1

Nov 16, 2016 08:42PM

- Steps to reproduce the problem:
Call LocationServices.FusedLocationApi.requestLocationUpdates(mGoogleApiClient, mLocationRequest, this); will result in a leak. Even though I'm calling removeLocationUpdates, disconnect, unregisterConnectionCallbacks and unregisterConnectionFailedListener inside onPause/onStop.
See attached project file.

- What happened:
LeakCanary reports a leak in an internal class from the package com.google.android.gms.location.
See attached log.

- What you think the correct behavior should be:
There shouldn't be any leaks.

Android version 6.0
Google Play Services 9.8.0
Related issue:

https://github.com/googlesamples/android-play-location/issues/26

deleted

Restricted

0 B

deleted

Restricted

0 B

Comments

fe...@gmail.com <fe...@gmail.com> #2Nov 17, 2016 07:41PM

Information redacted by Android Beta Feedback.

in...@gmail.com <in...@gmail.com> #3Dec 11, 2016 12:25AM

Thank you for reporting this issue. For us to further investigate this issue, please provide the following additional information:

Please upgrade to the latest release from the link V-QPR.If the issue still persists, please capture the new bug report along with the screen record or screen shot and share it here.

[Deleted User] <[Deleted User]> #4Jul 12, 2017 03:30PM

Upgrade was never offered. I was offered directly Android 16. I can't
install Android 16 as I want to leave beta.

<buganizer-system@google.com> schrieb am Do., 27. Feb. 2025, 08:55:

- Show quoted text -

ub...@gmail.com <ub...@gmail.com> #5Mar 12, 2019 01:36PM

Thank you for reporting this issue. We have shared this with our product and engineering team and will update this issue with more information as it becomes available.

fi...@gmail.com <fi...@gmail.com> #6Aug 4, 2019 01:19AM

Thanks for reporting this issue.

We are handling "No more paying via Wallet because the system is not up to date" wallet issue here.
Please file a separate ticket for "Upgrade was never offered to install android 16" with details.

de...@gmail.com <de...@gmail.com> #7Dec 11, 2019 03:49PM

Just want to add, same situation as yours, only it's been happening since December.
After upgrading to stable the issue persists.

sa...@google.com <sa...@google.com> #8Aug 18, 2020 03:24PM

Status: Won't Fix (Obsolete)

Thank you for your feedback. We assure you that we are doing our best to address all issues reported. For now, we will be closing the issue as won't fix obsolete. If this issue currently still exists, we request that you log a new issue along with the bug report here

https://goo.gl/TbMiIO and reference this bug for context.

ub...@gmail.com <ub...@gmail.com> #9Aug 21, 2020 08:04PM

Comment #8

, the one thing that would matter to address is how to *actually* get this issue fixed. It appears pointless to file a new issue, and unclear where to do so, as this issue has not received attention from Google's side throughout the years despite quite a few people starring it and bug reports in different places. The continued existence of the bug was confirmed not long ago in prior comments by different people. I've just confirmed it again myself on R Beta 3.

di...@gmail.com <di...@gmail.com> #10Mar 12, 2021 04:02PM

According to some Google DevRel it might soon be fixed, see the answer (from 8 days ago) here:

https://github.com/googlecodelabs/while-in-use-location/issues/7

je...@google.com <je...@google.com> #11Feb 8, 2022 05:37PM

Status: New (reopened)

This issue still exists, how can this be prioritized?

ub...@gmail.com <ub...@gmail.com> #12Feb 8, 2022 05:57PM

There is an issuetracker Component available these days for Play Services Location: https://issuetracker.google.com/components/1065160

Maybe move this issue to that Component to track it better?

je...@google.com <je...@google.com> #13Feb 9, 2022 12:18AM

I can't move it right now, but I think it's ok to stay here.

As I wrote in the code lab issue, I misunderstood when the rewrite went out (which should fix the issue).

It goes live in version 20 (current version location 19.0.1). The engineer who wrote the rewrite checked and wasn't seeing the same issue, i.e., it should be resolved.

However, that does mean we have to wait until 20 is released. I can check in again here when it's live for you to double check.

br...@gmail.com <br...@gmail.com> #14Jun 13, 2022 11:55AM

8 ram

je...@google.com <je...@google.com> #15Jul 18, 2022 05:32PM

I haven't been on location in awhile, but I wanted to follow up on this.

The issue should be resolved with the play-services-location:20.0.0 release from my understanding.

so...@google.com <so...@google.com> Aug 8, 2022 09:05PM

Marked as fixed, reassigned to so...@google.com.