[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