Improve duckdb_arrow_scan args check

sgrebnov · sgrebnov · commit ba3dca57ab6e · 2025-10-12T18:56:12.000-07:00
diff --git a/src/main/capi/arrow-c.cpp b/src/main/capi/arrow-c.cpp
@@ -287,6 +287,10 @@ duckdb_state Ingest(duckdb_connection connection, const char *table_name, struct
 duckdb_state duckdb_arrow_scan(duckdb_connection connection, const char *table_name, duckdb_arrow_stream arrow) {
 	auto stream = reinterpret_cast<ArrowArrayStream *>(arrow);
 
+	if (!stream) {
+ 		return DuckDBError;
+ 	}
+
 	return arrow_array_stream_wrapper::Ingest(connection, table_name, stream);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -287,6 +287,10 @@ duckdb_state Ingest(duckdb_connection connection, const char *table_name, struct`
`287`	`287`	`duckdb_state duckdb_arrow_scan(duckdb_connection connection, const char *table_name, duckdb_arrow_stream arrow) {`
`288`	`288`	`auto stream = reinterpret_cast<ArrowArrayStream *>(arrow);`
`289`	`289`
	`290`	`+ if (!stream) {`
	`291`	`+ return DuckDBError;`
	`292`	`+ }`
	`293`	`+`
`290`	`294`	`return arrow_array_stream_wrapper::Ingest(connection, table_name, stream);`
`291`	`295`	`}`
`292`	`296`