This is an automated email from the ASF dual-hosted git repository.
yiguolei 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 3f228da5897 [bugfix][be][cppcheck] Possible NULL pointer access
(#31025) (#31026)
3f228da5897 is described below
commit 3f228da589799765eb28e1ecae75c517897e5cb2
Author: Vallish Pai <[email protected]>
AuthorDate: Thu Feb 15 06:16:40 2024 +0530
[bugfix][be][cppcheck] Possible NULL pointer access (#31025) (#31026)
---
be/src/vec/exec/format/orc/vorc_reader.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/be/src/vec/exec/format/orc/vorc_reader.cpp
b/be/src/vec/exec/format/orc/vorc_reader.cpp
index d1b908dbc22..95160c94037 100644
--- a/be/src/vec/exec/format/orc/vorc_reader.cpp
+++ b/be/src/vec/exec/format/orc/vorc_reader.cpp
@@ -2170,10 +2170,10 @@ MutableColumnPtr
OrcReader::_convert_dict_column_to_string_column(
const int* dict_data = dict_column->get_data().data();
string_values.reserve(num_values);
size_t max_value_length = 0;
- auto* null_map_data = null_map->data();
if (orc_column_type->getKind() == orc::TypeKind::CHAR) {
// Possibly there are some zero padding characters in CHAR type, we
have to strip them off.
if (null_map) {
+ auto* null_map_data = null_map->data();
for (int i = 0; i < num_values; ++i) {
if (!null_map_data[i]) {
char* val_ptr;
@@ -2207,6 +2207,7 @@ MutableColumnPtr
OrcReader::_convert_dict_column_to_string_column(
}
} else {
if (null_map) {
+ auto* null_map_data = null_map->data();
for (int i = 0; i < num_values; ++i) {
if (!null_map_data[i]) {
char* val_ptr;
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
