[C++] Potential buffer overflow when CSV line contains too many columns

Describe the bug, including details regarding any error messages, version, and platform.

This snippet only has a debug-mode check, but the condition might fail if the CSV line currently parsed has more than num_cols columns.

arrow/cpp/src/arrow/csv/parser.cc

Lines 177 to 180 in f8a0902

    
           void PushValue(ParsedValueDesc v) { 
        
             DCHECK_LT(values_size_, values_capacity_); 
        
             values_[values_size_++] = v; 
        
           }

Component(s)

C++

Issue resolved by pull request 45498
#45498

Navigation Menu

Repository navigation

[C++] Potential buffer overflow when CSV line contains too many columns #45497

Describe the bug, including details regarding any error messages, version, and platform.

Component(s)

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

	void PushValue(ParsedValueDesc v) {
	DCHECK_LT(values_size_, values_capacity_);
	values_[values_size_++] = v;
	}

[C++] Potential buffer overflow when CSV line contains too many columns #45497

Description

Describe the bug, including details regarding any error messages, version, and platform.

Component(s)

Activity

pitrou commented on Feb 12, 2025

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions