Bug#890873: lintian: gcc-8-cross-ports makes lintian fork-bomb, consume all memory and fill the disk
Chris Lamb:
> Hi Niels,
>
>> Thanks for having a look at this.
>
> Thanks for the review. I've updated my patch here:
>
> https://gist.githubusercontent.com/lamby/996d85ac0e6ba0a18af2758e79912299/raw
>
>> I have one design concern; I think the blacklist should be global
>> not per archive.
>
> ACK, updated.
>
>> On the more minor side of things, I would have liked to see lintian
>> still record the packages that are blacklisted.
>
> In lieu of marking them as such on the reports, I've made it spit out log
> messages when it skips something. (For the offending packages, it will at
> least still show "processed with Lintian $very_old_version" so it should
> not be /too/ magic.)
>
>
>> We will probably want to map the blacklist into a hashref so we can do
>> O(1) testing to see if a source is blacklisted if the blacklist grows
>> more than a handful of packages.
>
> I hope this is unnecessary too (!), but I'm a bit stubborn so have
> implemented it regardless.
>
> Looking forward to your feedback... :)
>
>
> Best wishes,
>
Thanks, looks good to me with a quick review.
I think we should try that variant and see what breaks. :) Honestly, I
do not remember if you can use $foo->{'bar'} in a string interpolation,
but I guess are about to find out. :)
Thanks,
~Niels
Reply to: