diff options
author | Glenn Kasten <gkasten@google.com> | 2012-01-10 08:41:33 -0800 |
---|---|---|
committer | Glenn Kasten <gkasten@google.com> | 2012-01-10 15:43:24 -0800 |
commit | 0049acfc8b1c33eb90afc925dacd08a487618e17 (patch) | |
tree | 7e4d4b45c01eddf9fe71248edc7ebd24f70d2fc5 /include/media/JetPlayer.h | |
parent | c1dc1cb1d1eaf84e88669f1a5f22579a0d9237c2 (diff) | |
download | frameworks_av-0049acfc8b1c33eb90afc925dacd08a487618e17.zip frameworks_av-0049acfc8b1c33eb90afc925dacd08a487618e17.tar.gz frameworks_av-0049acfc8b1c33eb90afc925dacd08a487618e17.tar.bz2 |
Fix bug in JetPlayer::loadFromFile
A long pathname would corrupt the heap due to incorrect use of strncpy.
Also was using hard-coded constant 256 instead of PATH_MAX.
Change-Id: Iba382bbb38624fbc41fec91449f9814f77d4ac0d
Diffstat (limited to 'include/media/JetPlayer.h')
-rw-r--r-- | include/media/JetPlayer.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/media/JetPlayer.h b/include/media/JetPlayer.h index 6d53989..491a950 100644 --- a/include/media/JetPlayer.h +++ b/include/media/JetPlayer.h @@ -94,7 +94,7 @@ private: S_JET_STATUS mJetStatus; S_JET_STATUS mPreviousJetStatus; - char mJetFilePath[256]; + char mJetFilePath[PATH_MAX]; class JetPlayerThread : public Thread { public: |