[Fix](load) Restrict the import of VARCHAR(0) data to avoid coredump #40940 (#40974)

liaoxin01 · web-flow · commit f2b93d58964e · 2024-09-19T17:40:46.000+08:00
cherry pick from #40940
diff --git a/be/src/vec/sink/vtablet_block_convertor.cpp b/be/src/vec/sink/vtablet_block_convertor.cpp
@@ -218,7 +218,7 @@ Status OlapTableBlockConvertor::_internal_validate_column(
 
         size_t limit = config::string_type_length_soft_limit_bytes;
         // when type.len is negative, std::min will return overflow value, so we need to check it
-        if (type.len > 0) {
+        if (type.len >= 0) {
             limit = std::min(config::string_type_length_soft_limit_bytes, type.len);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -218,7 +218,7 @@ Status OlapTableBlockConvertor::_internal_validate_column(`
`218`	`218`
`219`	`219`	`size_t limit = config::string_type_length_soft_limit_bytes;`
`220`	`220`	`// when type.len is negative, std::min will return overflow value, so we need to check it`
`221`		`- if (type.len > 0) {`
	`221`	`+ if (type.len >= 0) {`
`222`	`222`	`limit = std::min(config::string_type_length_soft_limit_bytes, type.len);`
`223`	`223`	`}`
`224`	`224`