Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0c1d27ce authored by Steve McKay's avatar Steve McKay
Browse files

Explicitly close cursor.

BUG=21330700

Change-Id: I6cdfdaa1ac162075157a97a831d7349e34522a4a
parent 87c0620b
Loading
Loading
Loading
Loading
+5 −1
Original line number Diff line number Diff line
@@ -93,6 +93,8 @@ import com.android.documentsui.model.RootInfo;

import com.google.android.collect.Lists;

import libcore.io.IoUtils;

import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
@@ -1264,15 +1266,17 @@ public class DirectoryFragment extends Fragment {
            Uri itemUri = item.getUri();
            if (itemUri != null && DocumentsContract.isDocumentUri(context, itemUri)) {
                ContentProviderClient client = null;
                Cursor cursor = null;
                try {
                    client = DocumentsApplication.acquireUnstableProviderOrThrow(
                            resolver, itemUri.getAuthority());
                    Cursor cursor = client.query(itemUri, null, null, null, null);
                    cursor = client.query(itemUri, null, null, null, null);
                    cursor.moveToPosition(0);
                    srcDocs.add(DocumentInfo.fromCursor(cursor, itemUri.getAuthority()));
                } catch (Exception e) {
                    Log.e(TAG, e.getMessage());
                } finally {
                    IoUtils.closeQuietly(cursor);
                    ContentProviderClient.releaseQuietly(client);
                }
            }