Browse Source

ui:activity: Check if OCFileListFragment is null before calling onItemClicked().

A reference to null should never be dereferenced/accessed
ardevd 6 years ago
parent
commit
774b06cc03

+ 4 - 1
src/main/java/com/owncloud/android/ui/activity/FileDisplayActivity.java

@@ -2108,7 +2108,10 @@ public class FileDisplayActivity extends HookActivity
     private void onCreateFolderOperationFinish(CreateFolderOperation operation,
     private void onCreateFolderOperationFinish(CreateFolderOperation operation,
                                                RemoteOperationResult result) {
                                                RemoteOperationResult result) {
         if (result.isSuccess()) {
         if (result.isSuccess()) {
-            getListOfFilesFragment().onItemClicked(getStorageManager().getFileByPath(operation.getRemotePath()));
+            OCFileListFragment fileListFragment = getListOfFilesFragment();
+            if (fileListFragment != null) {
+                getListOfFilesFragment().onItemClicked(getStorageManager().getFileByPath(operation.getRemotePath()));
+            }
         } else {
         } else {
             try {
             try {
                 if (ResultCode.FOLDER_ALREADY_EXISTS == result.getCode()) {
                 if (ResultCode.FOLDER_ALREADY_EXISTS == result.getCode()) {