doStartTableScan() result not checked.
Add __attribute__((warn_unused_result)) to Cursor::startTableScan()
and by extension init_read_records
then go and fix the places where we weren't checking for errors.
This patch helped by Monty Widenius' patch to MariaDB. Greatly increased
confidence that we either handle the errors correctly or are at least
on par with bugs :)