From 7fc24c6578a945840cf3a062b7c07900f4e9f116 Mon Sep 17 00:00:00 2001
From: Soonil Nagarkar <sooniln@google.com>
Date: Fri, 18 Oct 2019 17:32:38 -0700
Subject: [PATCH] Add 0,0 location check

Ignore 0,0 locations from any real provider.

Test: n/a
Bug: 142977619
Change-Id: Ia3fd9712b6a8b3881ebfe2b29279700fa4b32d0f
---
 .../java/com/android/server/LocationManagerService.java     | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/services/core/java/com/android/server/LocationManagerService.java b/services/core/java/com/android/server/LocationManagerService.java
index 1d22b824aff99..ad3f5020031de 100644
--- a/services/core/java/com/android/server/LocationManagerService.java
+++ b/services/core/java/com/android/server/LocationManagerService.java
@@ -1000,6 +1000,12 @@ public class LocationManagerService extends ILocationManager.Stub {
 
         @Override
         public void onReportLocation(Location location) {
+            // likelihood of a 0,0 bug is far greater than this being a valid location
+            if (!isMock() && location.getLatitude() == 0 && location.getLongitude() == 0) {
+                Slog.w(TAG, "blocking 0,0 location from " + mName + " provider");
+                return;
+            }
+
             synchronized (mLock) {
                 handleLocationChangedLocked(location, this);
             }