Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933170AbbLXHjC (ORCPT ); Thu, 24 Dec 2015 02:39:02 -0500 Received: from relmlor2.renesas.com ([210.160.252.172]:34027 "EHLO relmlie1.idc.renesas.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S933090AbbLXHjA (ORCPT ); Thu, 24 Dec 2015 02:39:00 -0500 X-IronPort-AV: E=Sophos;i="5.20,472,1444662000"; d="scan'";a="201187589" Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=kuninori.morimoto.gx@renesas.com; Message-ID: <87d1twfgdi.wl%kuninori.morimoto.gx@renesas.com> From: Kuninori Morimoto To: Andrzej Hajda CC: Liam Girdwood , Mark Brown , Bartlomiej Zolnierkiewicz , Marek Szyprowski , Jaroslav Kysela , Takashi Iwai , "moderated list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM..." , open list Subject: Re: [PATCH v2] ASoC: rsnd: fix usrcnt decrementing bug In-Reply-To: <1450940559-16516-1-git-send-email-a.hajda@samsung.com> References: <87k2o4fkad.wl%kuninori.morimoto.gx@renesas.com> <1450940559-16516-1-git-send-email-a.hajda@samsung.com> User-Agent: Wanderlust/2.15.9 Emacs/24.3 Mule/6.0 MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset="US-ASCII" Date: Thu, 24 Dec 2015 07:38:54 +0000 X-Originating-IP: [211.11.155.144] X-ClientProxiedBy: OS1PR01CA0009.jpnprd01.prod.outlook.com (25.161.225.147) To SG2PR06MB1020.apcprd06.prod.outlook.com (25.161.9.28) X-Microsoft-Exchange-Diagnostics: 1;SG2PR06MB1020;2:daCxrF6k+cSEvFrGN269pimMDJgOtxOK8KpZm57InIEKp8wgdLRR5XALzRuzgUhaXkIg+GeLtRpRqFgRQxI3RZpb7b9ucJO9AkkWR3ZxjMFn+BFspmNEYxpiTanWDgRR3zpU2F+oHBTmhZs7JgfJIw==;3:NIWl++5IBNs+gwKPA1Ci5gewugx5xIoICpRoX/3xOYgrpsCC5T2FonTWH9goqieJBYHrJ6LkoWaVaBcu0sj7ppCze+br2lZiqXeXpZhxBIQ7ce0Gi94EV654ZcgV8XX6;25:o6n9RqX2S4VqKNwIsJe6cXUuBCFSu7IM4ODykjZ3STYLMD24uZPOkT/PKtYJmT3QwUhLGn/kkgPinDP3huzw/jRjoXJOAaRkOUeWCFO6UUzDEtMcCb1YCXHVdcKCageFXhVJPLVdPPy2sIun6eZwmmPKEbWqBO5gqe2JjLhwQajaP6OI51jMf7kCM+G0XeAFlQpZ7s9oXRWvc1g8KRBbRxYzlYeNx+IFeCMq04DgNKF0Rby570LqW44O7vKnYbjed6iqT+cle3KY2U6A1rDidA== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:SG2PR06MB1020; X-Microsoft-Exchange-Diagnostics: 1;SG2PR06MB1020;20:92QENULJbyKnubzpKCMslejCS8FJE+wQKdoN+VkLI6bFmIA67jwHdvdgFjbulWzc72hbgRYb3gV/5GxTAdyFnMgPA4o5iUHstpjmjj7jGE3crTQwg29IJqpGzbY8YJR8TzqD77TwXgOQbHrEc+lttria/ENJ6fQsIW4blFKLZ7XU3txMXzsJ9IDHUe3kXbE4Z7wDm/yL4AwMw9zjPQ067K0nV5NNm4LAjD1vzeoLjPK5TV/vqL625ho/FzaYh4tj3cbUMLMtpmmkn4wPq49gvrrrfnr6hpKKU8mPGVcHiNNt/ywUkbugaAf4StQWzG+exWRNCUjMbTts/bNvNZGGFYwy2A8LKCmXiSlZR/311rh/Y9sO2oJ80iD5j2DTcTgYaAU1xeigrD+bSPsE1Z/d7GZCEYL0iQ1HDnUwL3b/IZgE4Gk4UK4FFdUWxY0B/hhx0Gp4MQtIEb2tcRZSFlEMB8OumEXcMrH24s91EXbJVRn8BXLyQzSL0zvtkvPYgH12;4:2US1rPjPCdZH8sfuHgr0Wrt+aIGXWOBPHfwfnoZkXu2vS0DA4JxxPW1E5Wva9wv8wZlzpWH3sPA6IbeaRYUviHDSzTvaNEjYZ9imE8HC38t1v5qI8EN4qtnzmJAJmOF70nnK9oj0HKpH3i+KK0FfjkbivrS03l65xVEwEPpk8oeBS1GjDFMPgzXCbJzKcL7ABh7hHSXy4ffcy0tyUuCaRkxcWEcrioSIpF46vwyK9a6IF4T1o4TEG/muHMU73oEfARByT/LgY2PMQL+kHrJvD5MPVI36Guvkk4iSoZ88ztRY2qXhRw11HTSmCfAVA1CMkl2sL7VISHs4cQI6aYsTvUnOARpz28a33iIyZNj+ltOfOQsoBPxb0Lm8E/Ynw6BAiIEFcUWHfwVAnT2XVGvqW1J6FJXbVSj5peZA1oSgOnvUnBfwuQOIhnkToZtF/LY+ X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(85106069007906); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(5005006)(520078)(8121501046)(10201501046)(3002001);SRVR:SG2PR06MB1020;BCL:0;PCL:0;RULEID:;SRVR:SG2PR06MB1020; X-Forefront-PRVS: 0800C0C167 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(6009001)(189002)(199003)(42186005)(54356999)(2950100001)(1096002)(53416004)(86362001)(33646002)(87976001)(586003)(50466002)(92566002)(5004730100002)(46406003)(23726003)(3846002)(6116002)(5008740100001)(77096005)(15975445007)(122386002)(189998001)(4001350100001)(110136002)(83506001)(97736004)(81156007)(5001960100002)(106356001)(19580395003)(19580405001)(105586002)(36756003)(69596002)(66066001)(101416001)(47776003)(76176999)(50986999)(40100003);DIR:OUT;SFP:1102;SCL:1;SRVR:SG2PR06MB1020;H:morimoto-PC.renesas.com;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;SG2PR06MB1020;23:OCmpf/aLEvzQrlH5F/tGH88hDhsv3LHnvytKBYchx?= =?us-ascii?Q?giK3Pu2hRsQRrLTqsQpVTRJRzKOfCw7RBhQPGLD5RfKRuOmgiC9mYrvD98fH?= =?us-ascii?Q?BTGsiikxeavnX4kJQX3vdxyy3D8zXwcJM3qIY6Rmg9meVjYAPGXoc2yJS291?= =?us-ascii?Q?5VVaa2SEWvtCTHFzOaDIjVBA95QO68O5Z8SRzI5W2MkudwQE72poQpPRD82s?= =?us-ascii?Q?/o6wRkxlflrf8W4lcswhjDuMFzqLzLHUJbiTOR4BS5vULZuaqSrTDWWGSOVH?= =?us-ascii?Q?dD49ZKhCviUEA6kid0wyZA93WxEtFi1xOGBLRPvhQ5UxpDKrYilB24s4E4ch?= =?us-ascii?Q?HpMJDnbUUWZQpIlLrOlIPvGPw/j9mcosAdxOietWJ22kOmV82JorGen4SVQO?= =?us-ascii?Q?aaZ6F3klLAH9AIWAa4nUbHzDaQ+kfMv98sVRj8P5y0gFBldtfTDlej3UGCDy?= =?us-ascii?Q?Fesa9OZzwNkpxVnsF8UoXb9x9S9fq9Z7VYKJw5n80IOJH8W1/ZbNZ4J32N/G?= =?us-ascii?Q?btMXBZwwwE3Jxt3i0aLk6/VhBMcFkQMew//E2eH9PUoIUuQSF/ih7M9VkStW?= =?us-ascii?Q?MAdOcJGSY7qMt8zh86qltyL4MfqY3JAMO5tMT/Wmj9sItwjJZsMZts8zmBwa?= =?us-ascii?Q?62epDj442pG8pi3bnzgioF4zLgUj/XBEQ3Km2q9HpMkfooqz/Q9PjZcFR4vM?= =?us-ascii?Q?odt6nTYBN+dJdoOX+yXS1JG+XtT1arrugPXMgwHLRXHxwaZ+/g1bjc1cflKr?= =?us-ascii?Q?76UCHerPRUnjmweJcwHSmHxTjiWMvCq2vHsQjVs7h3ZhAkFCQo8qwwqv0VkR?= =?us-ascii?Q?vmMrDc1clTUceCXRXFDyKU7QIK4nmk/S6yFn3e7HNkZNuGeU9s33/HsP7a3D?= =?us-ascii?Q?J9x/cSfAt2g0oFfmE2o3rqDlC9Fi4YIz2T8YpZEGbQs0k8fxuyFa8qejZDCw?= =?us-ascii?Q?+q+X+Z5gx9OWvcIfyA4KMvS6Nawxf/GXNElwIlt2ALXTHDMeCLftYAhk/mII?= =?us-ascii?Q?NqT3BtVadbL4vdO+jam+Pvu/6MRu6Kg8mPUxiJm080p0GaWq47x9XZ0NmrW7?= =?us-ascii?Q?pD5+38F8mtkB0UWjFEsFztrZX2VssAL3bzKsQE6UQnru8pdzpkQxby6EsMN4?= =?us-ascii?Q?u3wZdLlcTU=3D?= X-Microsoft-Exchange-Diagnostics: 1;SG2PR06MB1020;5:LnappWLwbnnc9x9OE7ukuKSIwDvW4xcZI9JGNtx9l/rXNfhBStQixDDvlnPrdJtpKUKe3BaBTc2f2MNIfi2vB7fVD6zFm3pptKnVbOxFBU8stJEWfPkGSVQQq90C8xCmYw+3FnXBqf3ZOLnhlq6s/w==;24:E6qf9xN5hovDDG6Sj7JUYpJZjYV6zPHFWkBniGyS7tC683HAEHxZIw1uMfqIXGZcZmj/2PX0RtohuT69Cd70BXaJDa95K7RYWiClwY40XNk=;20:9z2lye30ZuXdmGyFiB9/YCsLzOTBKOTgqhZiPXVVkQDkeMHYP+4WniIGUvmo6coIkSQMPNqFJipJgyrqnoC4vt4QV3LnIoLNjJixnfcnmg08AwFjWAkry4FJjLZ6YbIST4U5P3UHAVciRgdUgDiEGg2VrRfW2U76gMHWbs2InlM= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: renesas.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Dec 2015 07:38:54.4842 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: SG2PR06MB1020 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 765 Lines: 22 Hi > > Field usrcnt is unsigned so it cannot be lesser than zero. > The patch fixes the check, moves it to the beginning of the function > and changes return value to -EIO in case of usercnt error. > > The problem has been detected using proposed semantic patch > scripts/coccinelle/tests/unsigned_lesser_than_zero.cocci [1]. > > [1]: http://permalink.gmane.org/gmane.linux.kernel/2038576 > > Signed-off-by: Andrzej Hajda > --- Acked-by: Kuninori Morimoto -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/