Sprinter/mame
2
0
Fork 0
mirror of https://github.com/holub/mame synced 2025-04-22 08:22:15 +03:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #1296 from npwoods/legacy_cassette_check_for_error

Browse Source 
Performs argument checking on the return value of CassetteLegacyWaveFiller.chunk_sample_calc()
This commit is contained in:
Vas Crabb 2016-08-23 22:32:53 +10:00 committed by GitHub
commit 76ff0172ab
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/lib/formats/cassimg.cpp
View File

@ -903,7 +903,7 @@ cassette_image::error cassette_legacy_construct(cassette_image *cassette,
chunk.resize(args.chunk_size);
/* determine number of samples */
if (args.chunk_sample_calc)
if (args.chunk_sample_calc != nullptr)
{
if (size > 0x7FFFFFFF)
{
@ -914,6 +914,13 @@ cassette_image::error cassette_legacy_construct(cassette_image *cassette,
bytes.resize(size);
cassette_image_read(cassette, &bytes[0], 0, size);
sample_count = args.chunk_sample_calc(&bytes[0], (int)size);
// chunk_sample_calc functions report errors by returning negative numbers
if (sample_count < 0)
{
err = cassette_image::error::INVALID_IMAGE;
goto done;
}
if (args.header_samples < 0)
args.header_samples = sample_count;