Merge pull request #453 from SUPERCILEX/fix-NPE

samtstern · web-flow · commit d1d678ec10b1 · 2016-12-12T14:39:15.000-08:00
Fix NPE in FacebookProvider
diff --git a/auth/src/main/java/com/firebase/ui/auth/provider/FacebookProvider.java b/auth/src/main/java/com/firebase/ui/auth/provider/FacebookProvider.java
@@ -113,7 +113,9 @@ public void setAuthenticationCallback(IdpCallback callback) {
 
     @Override
     public void onActivityResult(int requestCode, int resultCode, Intent data) {
-        sCallbackManager.onActivityResult(requestCode, resultCode, data);
+        if (sCallbackManager != null) {
+            sCallbackManager.onActivityResult(requestCode, resultCode, data);
+        }
     }
 
     @Override

Original file line number	Diff line number	Diff line change
`@@ -113,7 +113,9 @@ public void setAuthenticationCallback(IdpCallback callback) {`
`113`	`113`
`114`	`114`	`@Override`
`115`	`115`	`public void onActivityResult(int requestCode, int resultCode, Intent data) {`
`116`		`- sCallbackManager.onActivityResult(requestCode, resultCode, data);`
	`116`	`+ if (sCallbackManager != null) {`
	`117`	`+ sCallbackManager.onActivityResult(requestCode, resultCode, data);`
	`118`	`+ }`
`117`	`119`	`}`
`118`	`120`
`119`	`121`	`@Override`