Duncan Webb wrote: > I discovered this trace back after checking something else out. I copied > over a Kill Bill.mkv to the system to check audio, may be I should > rename it to Kill kaa.mkv ;) > > Probably we need to do the same file that I added to mpeg parsing, inbuf > will be empty. > > Traceback (most recent call last): > File "/usr/lib/python2.4/site-packages/kaa/metadata/factory.py", line > 162, in create_from_file > return parser(file) > File "/usr/lib/python2.4/site-packages/kaa/metadata/video/mkv.py", > line 302, in __init__ > self.process_elem(elem) > File "/usr/lib/python2.4/site-packages/kaa/metadata/video/mkv.py", > line 337, in process_elem > self.process_seekhead(elem) > File "/usr/lib/python2.4/site-packages/kaa/metadata/video/mkv.py", > line 359, in process_seekhead > self.process_elem(elem) > File "/usr/lib/python2.4/site-packages/kaa/metadata/video/mkv.py", > line 328, in process_elem > self.process_tracks(elem) > File "/usr/lib/python2.4/site-packages/kaa/metadata/video/mkv.py", > line 368, in process_tracks > self.process_track(trackelem) > File "/usr/lib/python2.4/site-packages/kaa/metadata/video/mkv.py", > line 386, in process_track > elements = [ x for x in self.process_one_level(track) ] > File "/usr/lib/python2.4/site-packages/kaa/metadata/video/mkv.py", > line 378, in process_one_level > elem = EbmlEntity(buf[index:]) > File "/usr/lib/python2.4/site-packages/kaa/metadata/video/mkv.py", > line 139, in __init__ > self.build_entity(inbuf) > File "/usr/lib/python2.4/site-packages/kaa/metadata/video/mkv.py", > line 146, in build_entity > self.compute_id(inbuf) > File "/usr/lib/python2.4/site-packages/kaa/metadata/video/mkv.py", > line 183, in compute_id > self.entity_id = (ord(inbuf[0])<<16) | (ord(inbuf[1])<<8) | \ > IndexError: string index out of range
Adding checks on the buffer length has corrected this problem, now kill bill doesn't kill kaa. It would be nice to add the aid and sid values, the first audio and subtitle tracks start at 0. Very similar to mpeg files which start at 128 (except DTS) Duncan Duncan ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Freevo-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/freevo-devel
