summaryrefslogtreecommitdiffstats
path: root/include/media/JetPlayer.h
diff options
context:
space:
mode:
authorGlenn Kasten <gkasten@google.com>2012-01-10 08:41:33 -0800
committerGlenn Kasten <gkasten@google.com>2012-01-10 15:43:24 -0800
commit0049acfc8b1c33eb90afc925dacd08a487618e17 (patch)
tree7e4d4b45c01eddf9fe71248edc7ebd24f70d2fc5 /include/media/JetPlayer.h
parentc1dc1cb1d1eaf84e88669f1a5f22579a0d9237c2 (diff)
downloadframeworks_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.h2
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: