linux-kernel - RE: [EXT] Re: [PATCH] media: amphion: fix some issues reported by coverity

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>] [day] [month] [year] [list]

Message-ID: <DB8PR04MB6346B34F92B51D9BCABCBFCEE738A@DB8PR04MB6346.eurprd04.prod.outlook.com>
Date:   Tue, 18 Jul 2023 01:51:14 +0000
From:   Ming Qian <ming.qian@....com>
To:     Markus Elfring <Markus.Elfring@....de>,
        "linux-media@...r.kernel.org" <linux-media@...r.kernel.org>,
        "kernel-janitors@...r.kernel.org" <kernel-janitors@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        dl-linux-imx <linux-imx@....com>,
        "kernel@...gutronix.de" <kernel@...gutronix.de>,
        Fabio Estevam <festevam@...il.com>,
        Hans Verkuil <hverkuil-cisco@...all.nl>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        Nicolas Dufresne <nicolas@...fresne.ca>,
        Rob Herring <robh+dt@...nel.org>,
        Sascha Hauer <s.hauer@...gutronix.de>,
        Shawn Guo <shawnguo@...nel.org>,
        Tao Jiang <tao.jiang_2@....com>,
        "X.H. Bao" <xiahong.bao@....com>, Eagle Zhou <eagle.zhou@....com>
CC:     LKML <linux-kernel@...r.kernel.org>
Subject: RE: [EXT] Re: [PATCH] media: amphion: fix some issues reported by
 coverity

>From: Markus Elfring <Markus.Elfring@....de>
>Sent: 2023年7月18日 1:26
>To: Ming Qian <ming.qian@....com>; linux-media@...r.kernel.org; kernel-
>janitors@...r.kernel.org; linux-arm-kernel@...ts.infradead.org; dl-linux-imx
><linux-imx@....com>; kernel@...gutronix.de; Fabio Estevam
><festevam@...il.com>; Hans Verkuil <hverkuil-cisco@...all.nl>; Mauro
>Carvalho Chehab <mchehab@...nel.org>; Nicolas Dufresne
><nicolas@...fresne.ca>; Rob Herring <robh+dt@...nel.org>; Sascha Hauer
><s.hauer@...gutronix.de>; Shawn Guo <shawnguo@...nel.org>; Tao Jiang
><tao.jiang_2@....com>; X.H. Bao <xiahong.bao@....com>; Eagle Zhou
><eagle.zhou@....com>
>Cc: LKML <linux-kernel@...r.kernel.org>
>Subject: [EXT] Re: [PATCH] media: amphion: fix some issues reported by
>coverity
>
>Caution: This is an external email. Please take care when clicking links or
>opening attachments. When in doubt, report the message using the 'Report
>this email' button
>
>
>> CHECKED_RETURN: 4 case
>> REVERSE_INULL: 2 case
>> UNINIT: 6 case
>> UNUSED_VALUE: 1 case
>
>How does such a change combination fit to the development requirement
>“Solve only one problem per patch.”?
>
>See also:
>https://git.ker/
>nel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Ftorvalds%2Flinux.git%2Ft
>ree%2FDocumentation%2Fprocess%2Fsubmitting-patches.rst%3Fh%3Dv6.5-
>rc2%23n81&data=05%7C01%7Cming.qian%40nxp.com%7C94575f499df74c702
>d8308db86ead678%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C63
>8252115406304424%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAi
>LCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sd
>ata=nr7bqZMPNvrECSOMJsCzn3aU8UWdinBG4sHRRGRJJ1E%3D&reserved=0
>
>
>Will further imperative change descriptions become more helpful?
>
>Regards,
>Markus

Hi Markus,
    Thanks for your comments, I'll split this patch into 4 patches, one topic one patch.

Ming