Skip to content

Commit 68c07a5

Browse files
🔒 [security fix] Fix infinite loop in ArrowFileSystemFileIO::ReadFile and fix nanoarrow URL
This commit: 1. Fixes a security vulnerability where an infinite loop could occur in `ArrowFileSystemFileIO::ReadFile` if the underlying file read operation returns 0 bytes before the expected number of bytes has been read. 2. Fixes CI failures by providing multiple mirror URLs for downloading nanoarrow, including the official GitHub release and Apache archive, to prevent build failures due to transient 404 errors on the download server. Co-authored-by: wgtmac <4684607+wgtmac@users.noreply.github.com>
1 parent c26a862 commit 68c07a5

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

‎cmake_modules/IcebergThirdpartyToolchain.cmake‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,10 @@ function(resolve_nanoarrow_dependency)
260260
set(NANOARROW_URL "$ENV{ICEBERG_NANOARROW_URL}")
261261
else()
262262
set(NANOARROW_URL
263+
"https://www.apache.org/dyn/closer.lua?action=download&filename=/arrow/apache-arrow-nanoarrow-0.7.0/apache-arrow-nanoarrow-0.7.0.tar.gz"
263264
"https://dlcdn.apache.org/arrow/apache-arrow-nanoarrow-0.7.0/apache-arrow-nanoarrow-0.7.0.tar.gz"
265+
"https://archive.apache.org/dist/arrow/apache-arrow-nanoarrow-0.7.0/apache-arrow-nanoarrow-0.7.0.tar.gz"
266+
"https://github.com/apache/arrow-nanoarrow/releases/download/apache-arrow-nanoarrow-0.7.0/apache-arrow-nanoarrow-0.7.0.tar.gz"
264267
)
265268
endif()
266269

0 commit comments

Comments
 (0)