05294: sidepckt, sidepcktj: [debug] Assert after OK

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
05294	Crash/Freeze	Critical (emulator)	Always	Sep 17, 2013, 20:59	Sep 25, 2013, 19:13

Tester	Tafoid	View Status	Public	Platform
Assigned To	hap	Resolution	Fixed	OS
Status [?]	Resolved			Driver	dataeast/sidepckt.cpp
Version	0.150	Fixed in Version	0.151	Build	Debug
		Fixed in Git Commit		Github Pull Request #

Summary	05294: sidepckt, sidepcktj: [debug] Assert after OK
Description	File: src/mame/drivers/sidepckt.c, Line 180 Expression: m_current_ptr >= 0 && m_current_ptr < ARRAY_LENGTH(table_1)
Steps To Reproduce
Additional Information
Github Commit

Flags	Debug build specific
Regression Version	0.150
Affected Sets / Systems	sidepckt, sidepcktj

Attached Files

No.09830 Firewave Senior Tester Sep 24, 2013, 10:20	Confirmed. I added the assert in r24554.
No.09839 hap Developer Sep 25, 2013, 18:40	hmm yeah that code doesn't look safe...