This is an automated email from the ASF dual-hosted git repository.

morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new 2dd1704e0da [Fix](orc-reader) Fix StringRef nullptr data by add 
checking string_values empty. (#42061)
2dd1704e0da is described below

commit 2dd1704e0da81b1341f32432a1bebe9f5e424eb2
Author: Qi Chen <[email protected]>
AuthorDate: Mon Oct 21 12:30:40 2024 +0800

    [Fix](orc-reader) Fix StringRef nullptr data by add checking string_values 
empty. (#42061)
    
    ## Proposed changes
    
    Added solution to #40857, fix StringRef nullptr data by add checking
    string_values empty in orc reader.
---
 be/src/vec/exec/format/orc/vorc_reader.cpp | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/be/src/vec/exec/format/orc/vorc_reader.cpp 
b/be/src/vec/exec/format/orc/vorc_reader.cpp
index 7c05f934bd5..761ddf3a5f9 100644
--- a/be/src/vec/exec/format/orc/vorc_reader.cpp
+++ b/be/src/vec/exec/format/orc/vorc_reader.cpp
@@ -1184,7 +1184,9 @@ Status 
OrcReader::_decode_string_non_dict_encoded_column(const std::string& col_
             }
         }
     }
-    data_column->insert_many_strings(&string_values[0], num_values);
+    if (!string_values.empty()) {
+        data_column->insert_many_strings(&string_values[0], num_values);
+    }
     return Status::OK();
 }
 
@@ -1288,8 +1290,10 @@ Status 
OrcReader::_decode_string_dict_encoded_column(const std::string& col_name
             }
         }
     }
-    data_column->insert_many_strings_overflow(&string_values[0], 
string_values.size(),
-                                              max_value_length);
+    if (!string_values.empty()) {
+        data_column->insert_many_strings_overflow(&string_values[0], 
string_values.size(),
+                                                  max_value_length);
+    }
     return Status::OK();
 }
 
@@ -2402,7 +2406,9 @@ MutableColumnPtr 
OrcReader::_convert_dict_column_to_string_column(
             }
         }
     }
-    res->insert_many_strings_overflow(&string_values[0], num_values, 
max_value_length);
+    if (!string_values.empty()) {
+        res->insert_many_strings_overflow(&string_values[0], num_values, 
max_value_length);
+    }
     return res;
 }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to