am d1f37448: Check mCurrentThumbRequest.mPath to prevent crash.
* commit 'd1f37448350890725736ababcc23c7deb0b2153f':
Check mCurrentThumbRequest.mPath to prevent crash.
diff --git a/src/com/android/providers/media/MediaProvider.java b/src/com/android/providers/media/MediaProvider.java
index ee4aad3..c96028e 100755
--- a/src/com/android/providers/media/MediaProvider.java
+++ b/src/com/android/providers/media/MediaProvider.java
@@ -619,24 +619,26 @@
Log.w(TAG, "Have message but no request?");
} else {
try {
- File origFile = new File(mCurrentThumbRequest.mPath);
- if (origFile.exists() && origFile.length() > 0) {
- mCurrentThumbRequest.execute();
- // Check if more requests for the same image are queued.
- synchronized (mMediaThumbQueue) {
- for (MediaThumbRequest mtq : mMediaThumbQueue) {
- if ((mtq.mOrigId == mCurrentThumbRequest.mOrigId) &&
- (mtq.mIsVideo == mCurrentThumbRequest.mIsVideo) &&
- (mtq.mMagic == 0) &&
- (mtq.mState == MediaThumbRequest.State.WAIT)) {
- mtq.mMagic = mCurrentThumbRequest.mMagic;
+ if (mCurrentThumbRequest.mPath != null) {
+ File origFile = new File(mCurrentThumbRequest.mPath);
+ if (origFile.exists() && origFile.length() > 0) {
+ mCurrentThumbRequest.execute();
+ // Check if more requests for the same image are queued.
+ synchronized (mMediaThumbQueue) {
+ for (MediaThumbRequest mtq : mMediaThumbQueue) {
+ if ((mtq.mOrigId == mCurrentThumbRequest.mOrigId) &&
+ (mtq.mIsVideo == mCurrentThumbRequest.mIsVideo) &&
+ (mtq.mMagic == 0) &&
+ (mtq.mState == MediaThumbRequest.State.WAIT)) {
+ mtq.mMagic = mCurrentThumbRequest.mMagic;
+ }
}
}
- }
- } else {
- // original file hasn't been stored yet
- synchronized (mMediaThumbQueue) {
- Log.w(TAG, "original file hasn't been stored yet: " + mCurrentThumbRequest.mPath);
+ } else {
+ // original file hasn't been stored yet
+ synchronized (mMediaThumbQueue) {
+ Log.w(TAG, "original file hasn't been stored yet: " + mCurrentThumbRequest.mPath);
+ }
}
}
} catch (IOException ex) {