[sane-devel] [Janitorial] Zero(!) compiler warnings
Olaf Meeuwissen
paddy-hack at member.fsf.org
Sun Sep 25 08:12:38 UTC 2016
Hi Allan,
Olaf Meeuwissen writes:
> m. allan noah writes:
>
>> On Wed, Sep 21, 2016 at 9:12 AM, Olaf Meeuwissen
>> <paddy-hack at member.fsf.org> wrote:
>
>>> ... next are four linker warnings about the use of `mktemp`, which, as
>>> we all know, is racey and should be avoided. Like the plague, so I'd
>>> like to see these get fixed.
>>>
>>> # I have a patch for the avision backend already. By all means, tackle
>>> # any of the canon, canon_dr and dc25 backends' use of mktemp.
>>
>> Note: it is canon that is busted, canon_dr does not use mktemp.
>
> The linker thinks it does. Could be that both backends link the same
> file in. I haven't really looked at it yet.
My bad! The CI builds run with -j2 so the output is sometimes a bit
jumbled up. I should look more carefully.
It's not the canon_dr backend but the canon630u backend.
Hope this helps,
--
Olaf Meeuwissen, LPIC-2 FSF Associate Member since 2004-01-27
GnuPG key: F84A2DD9/B3C0 2F47 EA19 64F4 9F13 F43E B8A4 A88A F84A 2DD9
Support Free Software https://my.fsf.org/donate
Join the Free Software Foundation https://my.fsf.org/join
More information about the sane-devel
mailing list