[ARROW-6126] [C++] IPC stream reader handling of empty streams potentially not robust

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.15.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/22521
• Assignee: Wes McKinney
• Reporter: Wes McKinney
• Created: 02/Aug/19 20:07
• Updated: 11/Jan/23 07:45
• Resolved: 23/Aug/19 16:46

Description

If dictionaries are expected in a stream, but the stream terminates, then "empty stream" logic is triggered to suppress errors, see

https://github.com/apache/arrow/blob/master/cpp/src/arrow/ipc/reader.cc#L482

It's probably esoteric but this "empty stream" logic will trigger if the stream terminates in the middle of the dictionary messages, which is a legitimate error. So we should only bail out early (concluding that we have an empty stream) if the first dictionary message is null

Issue Links

• links to: GitHub Pull Request #5146