[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [lintian] 01/01: coll/unpacked: Gracefully handle symlink targets with newlines



On 2014-10-17 10:42, Jakub Wilk wrote:
> * Niels Thykier <nthykier@moszumanska.debian.org>, 2014-10-17, 07:53:
>> +            # Link targets can have newlines in them.  A lession
>> learned by
>> +            # clasp (#765311)
> 
> s/lession/lesson/
> 

Thanks, fixed.

>> +            '|',
>> +            ['perl', '-l', '-0', '-pe', 's/\n/\\\n/g'],
> 
> Hmm. Shouldn't we escape other special characters, too?
> 

Possibly, but I suspect the effect is less problematic there because the
raw ones should be passed through just fine.  The sole exception is
probably "\\" which should be escaped as we unconditionally dequote the
result.  Do you have any other characters that might be problematic?

~Niels



Reply to: