[ADTSReader] Return false when id3tagparser fails due to eos

[CastagnaIT]

Description

Use case of HLS+AAC audio

CADTSSampleReader::ReadSample call ADTSReader::ReadPacket each time kodi request buffer data but when the stream buffer reach the end the m_id3TagParser.parse cannot read data from m_stream by returning ID3TAG::PARSE_FAIL
this was breaking the while loop and return true

since return true was causing that CADTSSampleReader::ReadSample to return AP4_SUCCESS with bad data
where instead was needed to check for waitingForSegment and mark the m_eos as true, this to allow stop the playback and so avoid flooding kodi demuxer of bad package data

seem weird to me that its needed return true also when the audio data is not parsed
it come from the initial ISA implementation so it could be a very old oversight or a workaround for other problems

Motivation and context

fix #1629

How has this been tested?

Sample stream provided in pvt by user and other 2 hls+aac

Screenshots (if appropriate):

Types of change

• Bug fix (non-breaking change which fixes an issue)
• Clean up (non-breaking change which removes non-working, unmaintained functionality)
• Improvement (non-breaking change which improves existing functionality)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that will cause existing functionality to change)
• Cosmetic change (non-breaking change that doesn't touch code)
• None of the above (please explain below)

Checklist:

• I have read the Contributing document
• My code follows the Code Guidelines of this project
• My change requires a change to the Wiki documentation
• I have updated the documentation accordingly