diff options
author | Thomas Jarosch <tomj@simonv.com> | 2014-12-20 13:37:40 +0100 |
---|---|---|
committer | Thomas Jarosch <tomj@simonv.com> | 2014-12-20 13:37:40 +0100 |
commit | c54537a6639bc9ea495fac5e9ecb3ee7a699b484 (patch) | |
tree | a8955bdb176f46a043423e946964fabb0c9e4d7f /apps/plugins | |
parent | f8d9e9cb6c4cbec8a611f871de006608ad7a1dbd (diff) | |
download | rockbox-c54537a6639bc9ea495fac5e9ecb3ee7a699b484.tar.gz rockbox-c54537a6639bc9ea495fac5e9ecb3ee7a699b484.zip |
Prevent theoretical out-of-bounds access in STM_Load()
We should be safe since STM_Load() should
never be called if STM_Test() fails.
Still it's better safe than sorry.
cppcheck reported:
[rockbox/apps/plugins/mikmod/load_stm.c:302]: (error) Array 'STM_Version[3]' accessed at index 3, which is out of bounds.
Change-Id: I914935fd108c492d013de24d17dcb9c227af6cd8
Diffstat (limited to 'apps/plugins')
-rw-r--r-- | apps/plugins/mikmod/load_stm.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/apps/plugins/mikmod/load_stm.c b/apps/plugins/mikmod/load_stm.c index b2537ab68f..994b0e5355 100644 --- a/apps/plugins/mikmod/load_stm.c +++ b/apps/plugins/mikmod/load_stm.c | |||
@@ -299,6 +299,8 @@ static int STM_Load(int curious) | |||
299 | /* set module variables */ | 299 | /* set module variables */ |
300 | for(t=0;t<STM_NTRACKERS;t++) | 300 | for(t=0;t<STM_NTRACKERS;t++) |
301 | if(!memcmp(mh->trackername,STM_Signatures[t],8)) break; | 301 | if(!memcmp(mh->trackername,STM_Signatures[t],8)) break; |
302 | if(t == STM_NTRACKERS) | ||
303 | return 0; | ||
302 | of.modtype = StrDup(STM_Version[t]); | 304 | of.modtype = StrDup(STM_Version[t]); |
303 | of.songname = DupStr(mh->songname,20,1); /* make a cstr of songname */ | 305 | of.songname = DupStr(mh->songname,20,1); /* make a cstr of songname */ |
304 | of.numpat = mh->numpat; | 306 | of.numpat = mh->numpat; |