Received: by 2002:a05:7412:b995:b0:f9:9502:5bb8 with SMTP id it21csp4425550rdb; Fri, 29 Dec 2023 00:57:18 -0800 (PST) X-Google-Smtp-Source: AGHT+IE6HJ8kdAmydlK60Hct8kNQ87f4GXPUjsdzx6/wBjlt+70Q6wpIiD/Kqkh+vV8rIzS3kc90 X-Received: by 2002:a05:6512:3da3:b0:50e:78f3:dbed with SMTP id k35-20020a0565123da300b0050e78f3dbedmr3642787lfv.91.1703840237851; Fri, 29 Dec 2023 00:57:17 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703840237; cv=none; d=google.com; s=arc-20160816; b=LiKVokFtFSZ8zMrOjTh3nHgqe2NnAk9QS9YNbDEk+JwIbDfh80wabEAsWX5diQF6vI y5ffaHLnBR/HQTjTgAPqC+EgyaB2OfxbhpgDlG6fTD70Q3Or/Rb4vvt3aLLIZ8IdJIt1 wi9HDNHbSzUUnHc5R9TT6yYTOH5me1Ao5O6ETzrzllaISNoeqcjLROx8sOK1kZkoPyvm hFOUlYu8OZ2VdaADNaw0Hki8+iITNs76XRlUeDMZcM7nTbn0SP4zoLBsPEPkpMEqJ7E/ IIGZ2seo8ilNlol+VUbnmksmt6Mvd5V2pBfoCUjZnoiSR6bn+qoyGTofY71MHOgB1GxX 6RkQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=ui-outboundreport:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:date:message-id :dkim-signature; bh=TLIBnCVR41anr31BNytRhjf9vcw4oSOtATyNR7teNzA=; fh=0Ya/csuxPGb6Oi0gj9tu+yEPH7XbU7DUMI16bfAZI+Q=; b=nOklZU9q74a7fb1yqeYbHRnX9/ZLGKAIqxIfRyzMqFxntS2egfugGruVShjVshyHRR yuQxSEge7w6bGDUnbDOrb7yBScEZLoDfYO5VN3veaWrctRMNM8y2EAp59p5r/aqtnEsc eQMViPpwWVJp8oewDCRtsHtJjZkMpYGhyJmfMBLM/rkLh5Z0+KjXHTW+hPv0CI9DpRGM Im8T6YFbRa/meA9swNoKdqn2aFheSPEK9CPHY45rC7eQrfXIqKD0DLET5fvttafTCNDr y67K6VKrvhEQQ3/kP+jouVYpaeObPnxSmBtQ8D+3ghxU3nWz2N8a00ohz741yfB/cnGy QYmw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=Aw12T9BT; spf=pass (google.com: domain of linux-kernel+bounces-13009-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-13009-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [2604:1380:4601:e00::3]) by mx.google.com with ESMTPS id n3-20020a5099c3000000b005534be58c54si7815669edb.160.2023.12.29.00.57.17 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 29 Dec 2023 00:57:17 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-13009-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3; Authentication-Results: mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=Aw12T9BT; spf=pass (google.com: domain of linux-kernel+bounces-13009-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-13009-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id 93F0A1F227FE for ; Fri, 29 Dec 2023 08:57:17 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 001BC1097B; Fri, 29 Dec 2023 08:57:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=web.de header.i=markus.elfring@web.de header.b="Aw12T9BT" X-Original-To: linux-kernel@vger.kernel.org Received: from mout.web.de (mout.web.de [212.227.15.14]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0342210A00; Fri, 29 Dec 2023 08:57:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=web.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=web.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=s29768273; t=1703840183; x=1704444983; i=markus.elfring@web.de; bh=TLIBnCVR41anr31BNytRhjf9vcw4oSOtATyNR7teNzA=; h=X-UI-Sender-Class:Date:Subject:To:Cc:References:From: In-Reply-To; b=Aw12T9BTOms3c731LbdPhLBu7QQ7WYYcMSe2es/lXoYMLdm++riAyrgre1jcn8UP L7TJ1Pnh6ndqUhlrzK8XdOAKm2tTJ46d6pVzEpP7igdZtGZHnHoMh6xxgrXtHKwxv K7PlYnZAWdciF8jqULSKGI/5ObrdyQwzBNc3HjiMU7JMVxpF5++ktZ28YwuC+TNVl QhXm3mLN7h2W6NJFP1Ll+GkRIsoVOuv+4BMUtOi+Mn7wLM0y8zB13dNDlXSRMkVEs kh4w2HUL4jihQacQ2R0qKSFRCZBayRVZYdkAoLRh6kEmVMK7b6nr7M3SL2TVulMtp D7Ng0RHiJYsKnTd8Pg== X-UI-Sender-Class: 814a7b36-bfc1-4dae-8640-3722d8ec6cd6 Received: from [192.168.178.21] ([94.31.85.95]) by smtp.web.de (mrweb005 [213.165.67.108]) with ESMTPSA (Nemesis) id 1MF2gW-1rUBH511se-00Foq7; Fri, 29 Dec 2023 09:56:23 +0100 Message-ID: Date: Fri, 29 Dec 2023 09:56:22 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: Input: usbtouchscreen - Return directly after a failed kmalloc() in nexio_init() Content-Language: en-GB To: Dmitry Torokhov , linux-input@vger.kernel.org Cc: kernel-janitors@vger.kernel.org, Oliver Graute , =?UTF-8?Q?Uwe_Kleine-K=C3=B6nig?= , ye xingchen , LKML , cocci@inria.fr References: <9365c845-baa1-44d1-add9-ec8ca4d365eb@web.de> From: Markus Elfring In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:K3N/JgCX21RpM3f27fdRKwJDDoLXRmG2i5YFyXwrN/krCUMYcE8 0a5Sm2ZiF2RAnDHWOVUfRNQocEqoUuEdGbCpW0AwENZ0siJiN2CZb1WhSRGstr71sZTop03 YVBB/G4J+DC6PDg7Y6EZ/NfJptbJCWz11dmhHXIAUgd/JnfFF6YS0h4cZsXkH2XcVF0m7Q3 npARVFm6PJrP/sPvwEfZw== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:abzbliDP3o4=;nsgH/UvmEiwsgW3As5IIowSgbT3 2dziVncnGh89DkBPVaZaZWtbkwNZHy0do+5q49KULlmBPD36s0DxDXTr0an4CHWpsRroF8K2A fw8uiAJMMUzbxxFw713oFHoyIOWRCsq08TnGzuL6lQee+KuV3aeHYJehHUPB9eJLxfqbp7qKL 16+4cDQ+JDQ/TENMnl/CQIYA/E+TWM2WxtDXDQzO0qli0FK3U+Xbp34F3Hqnr3fWWEivv+J2W b1mYJ/JOwaE6zwcORbqMrDp3dQH/DhroaIYzOenrZ6bOgxBGhiyzQXxtLZZpbBBKYEDlnsRbk P2DpgkDfd0/R4rm6VjYlhoVOC5CVlzC3s5zxeThJX1x5WN+VvJc7n2eUU1TBT0d74WVryfTfM cWAlgHJsXtshVbVIdNaC2qtIc2x91PvwtqmBSfgsrmbn/c84VmiIcggyjGGyDl4nQv3sVqxE7 ootKcDr3tLbTQC3ezv5a6S+dlUwUFQ9RLTm+AT4xXxxv2afDr2PqIn3J8hmLJqVXqtYSrVKG/ r0AM2p58lEzFJPkfXFEzhA9oBeKOcz95oz45VcnEPvVKeGmW4pfRmzVpDa0+dfrJztGF5lqxk F06e21MB03swuaIPjvkZkn/Ln7GlBmUhJMRUywuh9xrXPWnk4G+EogfOWr/0jk4NgiIizvpAp EsHc01nvBcgZZ94ooSqGJHaKThnYCCDLE5MzsMi4zTCGFnLtmuj8R2HE58zjY/JOAwm7k3vsM Sw0F/4dN3ev3KYugyJpB/3pS/SjyKZxvcGMXQ4HcRwbhcgUJSwGoLxyQA6G8l4TIj42IkSUkT Cz6t3jUb6oSZkgcdkWaKqgi7zKQ0hTSGGASKgI/EVP8ECKKKsKIKZ3rrdlo3PTxGBqKW78d98 DWTlpiig38SFkivZCzJIRChdZ/pVwLBB3ZOYFdmMDJxo28cNe6WarHg90aT0nnY6JRVBXhKT5 Nejg0PDxdxx+BxqKTbo1dR9jM6U= >> The kfree() function was called in one case by >> the nexio_init() function during error handling >> even if the passed variable contained a null pointer. > > Which is perfectly valid thing to do, like free(), kfree() accepts NULL = argument. I find such a function call with this special parameter not so useful. >> Thus return directly after a call of the function =E2=80=9Ckmalloc=E2= =80=9D failed >> at the beginning. > > This is simply a matter of preference, the original author preferred > that style, I see no objective reason to change it. Would you ever like to avoid redundant data processing a bit more? Regards, Markus