paleolimbot opened a new issue, #909: URL: https://github.com/apache/arrow-nanoarrow/issues/909
In https://github.com/apache/arrow-nanoarrow/pull/906 , we fixed a compile error where ENODATA wasn't defined on BSD platforms. I didn't know that ENODATA was problematic at the time, but this internal sentinel is not ideal as it's not actually part of the standard and is equivalent with ENOATTR in some places. The internal use of this is OK with any integer value; however, there are potentially a few code paths used by advanced users of the low-level decoder that could leak this value to users, and if it's returned to an end user it should be a valid errno value. We can either pick a new sentinel or rewrite that code path to avoid using a sentinel at all or ensure that any internal sentinel doesn't get returned to end users. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
