On 3/4/15 12:59 PM, Верещагин Алексей wrote: > Bash Version: 4.3.33 > File: lib/sh/shquote.c > Function: sh_backslash_quote > Line: if (backslash_table[c] == 1) > > Description > ----------- > Variable "c" has signed integer type and may be negative (if "string" is > not only ASCII characters string). But "c" used as an index in > "backslash_table" array. This causes out of range error and produce > undefined behavior. > > Possible solution > ----------------- > Cast variable "c" to unsigned char type: > if (backslash_table[(unsigned char)c] == 1)
Thanks for the report. This is a good fix for the remaining cases where sh_backslash_quote is called with arbitrary filenames. Chet -- ``The lyf so short, the craft so long to lerne.'' - Chaucer ``Ars longa, vita brevis'' - Hippocrates Chet Ramey, ITS, CWRU c...@case.edu http://cnswww.cns.cwru.edu/~chet/