[IMPALA-7833] Audit and fix other string builtins for long string handling

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Impala 2.11.0, Impala 3.0, Impala 3.1.0
• Fix Version/s: Impala 4.0.0, Impala 3.4.1
• Component/s: Backend
• Labels: crash ramp-up
• Target Version: Impala 4.0.0
• Epic Color: ghx-label-4
• Assignee: Akos Kovacs
• Reporter: Tim Armstrong
• Created: 08/Nov/18 00:02
• Updated: 05/Feb/21 13:19
• Resolved: 14/May/20 06:31

Description

Following on from IMPALA-7822, there are some other string builtins that seem to follow the same pattern of having a string size overflow an int passed into the StringVal constructor. I think in some cases we get lucky and it works out, but others it seems possible to crash given the right input values.

Here are some examples of cases where we can hit such bugs:

select lpad('foo', 17179869184 , ' ');
select rpad('foo', 17179869184 , ' ');
select space(17179869184 );