diff options
author | Andreas Huber <andih@google.com> | 2010-10-27 10:33:04 -0700 |
---|---|---|
committer | Andreas Huber <andih@google.com> | 2010-10-27 10:33:04 -0700 |
commit | 4e6572967da698d568a44d18797b751ec8ae5401 (patch) | |
tree | d234ee861f583d46650e92e9718087aa08ecbb9a /media/libstagefright/FileSource.cpp | |
parent | 7b54d6652692ce078df9c606d2d72ebc8ee1f4a2 (diff) | |
download | frameworks_av-4e6572967da698d568a44d18797b751ec8ae5401.zip frameworks_av-4e6572967da698d568a44d18797b751ec8ae5401.tar.gz frameworks_av-4e6572967da698d568a44d18797b751ec8ae5401.tar.bz2 |
FileSource may fail to open a file in which case subsequent code dereferences a NULL pointer. This fixes this recently introduced problem.
Change-Id: I60dd9532b694a1b488743b71f43e107c778d6683
Diffstat (limited to 'media/libstagefright/FileSource.cpp')
-rw-r--r-- | media/libstagefright/FileSource.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/media/libstagefright/FileSource.cpp b/media/libstagefright/FileSource.cpp index b46d8d0..e4f9a47 100644 --- a/media/libstagefright/FileSource.cpp +++ b/media/libstagefright/FileSource.cpp @@ -21,7 +21,7 @@ namespace android { FileSource::FileSource(const char *filename) : mFile(fopen(filename, "rb")), - mFd(fileno(mFile)), + mFd(mFile == NULL ? -1 : fileno(mFile)), mOffset(0), mLength(-1), mDecryptHandle(NULL), |