linux-kernel - Re: linux-next: Tree for Jul 26 (media/usb/dvb-usb/pctv452e.c)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <7995d469-34a8-2574-5137-18a509187d23@infradead.org>
Date:   Mon, 29 Jul 2019 11:04:40 -0700
From:   Randy Dunlap <rdunlap@...radead.org>
To:     Dominik Kuhlen <dkuhlen@....net>
Cc:     Stephen Rothwell <sfr@...b.auug.org.au>,
        Linux Next Mailing List <linux-next@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        linux-media <linux-media@...r.kernel.org>,
        Andre Weidemann <Andre.Weidemann@....de>,
        "Michael H. Schimek" <mschimek@....at>
Subject: Re: linux-next: Tree for Jul 26 (media/usb/dvb-usb/pctv452e.c)

On 7/29/19 10:52 AM, Dominik Kuhlen wrote:
> Hi Randy,
> 
> On Friday 26 July 2019 16:57:50 Randy Dunlap wrote:
>> On 7/25/19 10:23 PM, Stephen Rothwell wrote:
>>> Hi all,
>>>
>>> Changes since 20190725:
>>>
>>
>> on x86_64:
>>
>> In file included from ../drivers/media/usb/dvb-usb/pctv452e.c:16:0:
>> ../drivers/media/usb/dvb-usb/pctv452e.c: In function ‘pctv452e_frontend_attach’:
>> ../drivers/media/dvb-frontends/stb0899_drv.h:139:36: error: weak declaration of ‘stb0899_attach’ being applied to a already existing, static definition
>>  static inline struct dvb_frontend *stb0899_attach(struct stb0899_config *config,
>>                                     ^~~~~~~~~~~~~~
> 
>  cat config-r6623 | grep -i pctv45
> CONFIG_DVB_USB_PCTV452E=y
> 
>  cat config-r6623 | grep -i stb0899
> # CONFIG_DVB_STB0899 is not set
> 
> -> mismatching configuration:
> The pctv452e needs the stb0899 frontend driver to work.
> And the CONFIG_DVB_STB6100 must also be set.

This build error is from a randconfig.  The Kconfig rules for this (any)
driver should not allow build errors to be generated.

For DVB_USB_PCTV452E (and probably most of drivers/media/), there is an
attempt to handle this by the use of MEDIA_SUBDRV_AUTOSELECT (which is
not set/enabled in this particular randconfig).  So I am reporting that
the Kconfig entry for DVB_USB_PCTV452E is not sufficient.
(although I expect that some people won't care)


thanks.
-- 
~Randy