Invalid files cause irrecoverable crashes in `ROOTReader::openFile(...)` about podio

[tmadlener]

Thanks for reporting this. Given that the ROOTReader and the EventStore are deprecated now, we will probably not fix these. However, a similar issue exists in the new Frame based readers as well, where the only check that exists with a "nice" exception is whether the file exists.

This doesn't fix anything for you, but the python bindings deal with this slightly better as we have implemented some functionality there for detecting this:

from podio.reading import get_reader
reader = get_reader("<filename>")
# ...

podio/python/podio/reading.py

Lines 33 to 57 in 9cd0373

	def get_reader(filename):
	"""Get an appropriate reader for the passed file.
	Args:
	filename (str): The input file
	Returns:
	root_io.[Legacy]Reader, sio_io.[Legacy]Reader: an initialized reader that
	is able to process the input file.
	Raises:
	ValueError: If the file cannot be recognized, or if podio has not been
	built with the necessary backend I/O support
	"""
	if filename.endswith('.sio'):
	if _is_frame_sio_file(filename):
	return sio_io.Reader(filename)
	return sio_io.LegacyReader(filename)
	if filename.endswith('.root'):
	if _is_frame_root_file(filename):
	return root_io.Reader(filename)
	return root_io.LegacyReader(filename)
	raise ValueError('file must end on .root or .sio')

[tmadlener]

Closing this since the EventStore based Readers and Writers have been removed in #485