From patchwork Mon Dec 25 10:18:30 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Markus Elfring X-Patchwork-Id: 183167 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:6f82:b0:100:9c79:88ff with SMTP id tb2csp323631dyb; Mon, 25 Dec 2023 02:18:59 -0800 (PST) X-Google-Smtp-Source: AGHT+IHrcqynNmdyhDNaPGz0dks6KN9C0nQaTaj1hIhQGPPCtbMwJBZ5pxXzTsFG5s73yizp9Gni X-Received: by 2002:ac8:7f0c:0:b0:412:c2a:eaef with SMTP id f12-20020ac87f0c000000b004120c2aeaefmr7355554qtk.11.1703499539024; Mon, 25 Dec 2023 02:18:59 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703499539; cv=none; d=google.com; s=arc-20160816; b=jIfjWpPUyzO+54UJth7W9NK5sAnrLX1nMErBiV9x7qxaMMyGUpyf2hiyRdpxmmUJ/b /1h2EZD+yHQyJiGxE0GW/6RGRIeUxL5cccjV16TRv4mOIQOHMgFNRf4yWl3Auv+/0R91 2xt75EN+BPU0HS1gD/k99vgWGCI1pFJ/vmjHkxMiaveZlGZE+jolw4XdRyYrvdpJfsW9 eNN4EsprOsEUbk84zCYPfnO5u7x7mSv+tGEhO6ot8fwFa2v2+Pp/8eza6vbJscKPiGG5 gvxjvnCdeRaeZ0fUBehyFzbw2zJbE3TXixnxatAlPl9YIRH/AYmyzIIrjHVc6zwbrZns dfrQ== 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:references :cc:to:from:content-language:subject:user-agent:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:date:message-id :dkim-signature; bh=ySWAI4kP7vmZs3Y+VlnIHKgTIesun+Yl241EM1zS2CU=; fh=ksZansD5U+4SwnCe5XSfBs20ubp6qS0cOKRNB4rLVWY=; b=u7ysI8nULsvJ0IJJRcEAdZsEUhfYXjcYkYh7wt+Ep3jcrHmGLMx7uEt5EARViwZ/tK NOfuAOtocQy3DJLpa6G9g5x2E7HvJk0BHo0iOvAWu7HWt9AqXpT6wHazMFFlzI3H/Ti4 8Ik9OzEis3Jw2HLHXAzGCbhTAulRNzbDeJ/2xCVrNb74ynBAv6QnKzmoVpbss5YAOdm9 7xUBQXCQEAayb5NeIzGjJhXH/AlRgexUmeDVqnAl1l44dwiqkfoqvgHsNTBtQ39j21qS SESlaCqtfz0qgNrXg0Fc4S0DUTnxqtHA9QPlI0RtJPmgleLXY3Vm0ZzK6YCdtnJHaWzv PsOg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=AqR1cWYt; spf=pass (google.com: domain of linux-kernel+bounces-10984-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-10984-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id z12-20020ac87f8c000000b0042790b60db2si9801709qtj.620.2023.12.25.02.18.58 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 25 Dec 2023 02:18:59 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-10984-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=AqR1cWYt; spf=pass (google.com: domain of linux-kernel+bounces-10984-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-10984-ouuuleilei=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 ny.mirrors.kernel.org (Postfix) with ESMTPS id CB4891C218AE for ; Mon, 25 Dec 2023 10:18:58 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 8FF8E35F12; Mon, 25 Dec 2023 10:18:43 +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="AqR1cWYt" X-Original-To: linux-kernel@vger.kernel.org Received: from mout.web.de (mout.web.de [212.227.15.4]) (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 CF0D41D683; Mon, 25 Dec 2023 10:18:38 +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=1703499511; x=1704104311; i=markus.elfring@web.de; bh=HUXBB4x97jqAAp/XJdLxZnTbCwGahqmDUfoq0o7TY0Y=; h=X-UI-Sender-Class:Date:Subject:From:To:Cc:References: In-Reply-To; b=AqR1cWYtd7+eKcVdEjj1jDRU90bU3sIoJYTdsYIuRUHa05DCoczn9hKRkw35Sd9n aV8azjz4esk7zxbfKkfzLsweIPy6Qph8ZtGvVNZhTjtB1r15Dczg9C60qNG1V+51Y POG+U6YcjzhEOVyzx3BebImbfBjg6JfGuBOvdWmZ8j6sfbyEZz477gA1v1DB8dMLe 7r5cfIdT4rGRWcrbW2vwYkTeeAUoZaQx79m80HJXbeIkVuNmazK8UakCL0xuyZSb7 XOIpdxXRmdBYAQU4VRnVWl+vtbKytqFtpvpDYaAF31y5C0OorWpuZzDgOhRzSHcvg wWuzCg6+vbxJVBSnEg== X-UI-Sender-Class: 814a7b36-bfc1-4dae-8640-3722d8ec6cd6 Received: from [192.168.178.21] ([94.31.85.95]) by smtp.web.de (mrweb006 [213.165.67.108]) with ESMTPSA (Nemesis) id 1M2ggd-1rGt5s2LFf-0040rC; Mon, 25 Dec 2023 11:18:31 +0100 Message-ID: <4b7b5eed-d00e-43ea-afb6-1f563fb2577c@web.de> Date: Mon, 25 Dec 2023 11:18:30 +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: [PATCH 1/3] dmaengine: timb_dma: Return directly after a failed kzalloc() in td_alloc_init_desc() Content-Language: en-GB From: Markus Elfring To: dmaengine@vger.kernel.org, kernel-janitors@vger.kernel.org, Vinod Koul Cc: LKML , cocci@inria.fr References: In-Reply-To: X-Provags-ID: V03:K1:kzWqifzdC9upknspVDEsGqa/iK/LiQ+tcH1RpaEvLdJZMpNhh8m Ys3H+MVK7Ztd9cpyz3ZRXbetOfAcRVo9gGT610atXbS+5HOqpzQGP5DnFvMwwD5u6hGRd43 cJWIpvazYhfCVaSo0wYwQVMyf6y5Vj7iFNG1UxATRmzUMKUKQBIRgr+/uUh3IyJEk54a6fr UouIH+E8jv+jvI/c31o3w== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:5WXMyCpU9ac=;VbG3CJm/62DrSyXNdzthHO8TcpT 1/uF+efGVkVTZz2xgTh5sY0R6uotfXQmiFkdOgVyajsiF+N+eIpvGq+NM0Oblg5N42D3JPXJw G8qveNjhgONtQUfwxRGkj3FoxPFP0IcY/9DKhwTZnVZm6TF5RS6K/rzKLdhJE8gjq2zPiYnUV Hsp/D31XyTekbDwyLI2/PpiIfadDn8h78MJeM/6H8fhF0l1ZPEuJjgZ37J9nDPOlIWnIFKOXV 1jz2dbJgkHOKlpIYC0Yclg5rBAMb2RF+td8Q55B32MP6vaVdmsciELyDPmOPapizkp6My3xeS MhaRjvCPIx8UJV6iX1wx7V3VQ6CbSNKHo5HL4WaIUAlMprgkMwKBSsV6QIiW75gqEggWma/xE HuSewG2EsF5XXtL4hgefQUp0VZTAlYjZ2SjGKlYfnBMapMVeMXd28mktGq84vXoeXTXTE1hmk 6zNVaEQFiRjBNP+0/ROnI0RZcukdBO7MGDFNaUXuWFwHFXGcUv2crddDd0XHifljTLhp8mA9p vu2pp7tcSHbX11EpkQ9ELuQ26wiApMbj3uwJyWd6vpyjqakbgjnrCM6zS1Umw+6Q0NIVRne9y gDJGsC/javSUel4AmlNnnuV+Kdhd25qiluARVGS+zw46nLWEyfrxsFFwalmR/SIayFLv1l5Bi bYy02+KYi6oQOCaWLcsbPJpUusCn7jMyvBaxA65kvsqLqxb1wYcMeT/xstXng3HbW36iYWotn kkIa8miKrk0X4LfSPnYnTeqqfOQFyHZwlNLj2ba1vyYhju5BpBtrjfeZ4u7C7dcvhJlvJMAJR sTdY6TNNnxbMl7gbYnNZnzfH4OC8f14PLMpY7WtkEIUkMqTQOtnlZnn67bjwD1357BNZw0CrN 71IWRzpxX1Y/jm7kBI9vwbSwH8sFKO6ehNJr2kSvnsILHmFsKAD0z4ODF6SRyaKZp06UA4wo/ vAVJ9Q== X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1786248732836657793 X-GMAIL-MSGID: 1786248732836657793 From: Markus Elfring Date: Mon, 25 Dec 2023 10:40:12 +0100 1. Return directly after a call of the function “kzalloc” failed at the beginning. 2. Delete the label “out” which became unnecessary with this refactoring. Signed-off-by: Markus Elfring --- drivers/dma/timb_dma.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) -- 2.43.0 diff --git a/drivers/dma/timb_dma.c b/drivers/dma/timb_dma.c index 7410025605e0..abcab0b1ad3b 100644 --- a/drivers/dma/timb_dma.c +++ b/drivers/dma/timb_dma.c @@ -327,7 +327,7 @@ static struct timb_dma_desc *td_alloc_init_desc(struct timb_dma_chan *td_chan) td_desc = kzalloc(sizeof(struct timb_dma_desc), GFP_KERNEL); if (!td_desc) - goto out; + return NULL; td_desc->desc_list_len = td_chan->desc_elems * TIMB_DMA_DESC_SIZE; @@ -352,7 +352,6 @@ static struct timb_dma_desc *td_alloc_init_desc(struct timb_dma_chan *td_chan) err: kfree(td_desc->desc_list); kfree(td_desc); -out: return NULL; } From patchwork Mon Dec 25 10:20:18 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Markus Elfring X-Patchwork-Id: 183168 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:6f82:b0:100:9c79:88ff with SMTP id tb2csp324135dyb; Mon, 25 Dec 2023 02:20:47 -0800 (PST) X-Google-Smtp-Source: AGHT+IFG9UOI27WLkK0psALMEY+XAhzo9k0q/X+mny0zmLM/LBZaA5UcwAUVh6D8ZnDCLp7Ak+dr X-Received: by 2002:a50:a41b:0:b0:550:5173:22f0 with SMTP id u27-20020a50a41b000000b00550517322f0mr4308234edb.46.1703499647262; Mon, 25 Dec 2023 02:20:47 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703499647; cv=none; d=google.com; s=arc-20160816; b=upvjYzhtdo493/W8AbrjO9aIPlJCKprXuWqxzSY4NA7Mq5RgXZvmXrpiwwmPvTkudQ 9h8iclP4RJbSaTlCxPunmMqWriTqm9nsw1opQfkCBKrvPOth0fZawgfozwQ5P1gFwUd2 4rWvbn7ReP1IppKvaOa78qji0jMPFiokj8+HoP462QFWCeciYvv2yLzD5LlJG7nrBXQ5 Wcn4RCGjCven5hvmZbnJTFrs8QsJNHj8sGxwb5nUorYIaE9bIjhpiBLgKgxR1BTSEMDH VX+TO4iyRJ70/ZjG0SamP08WOcIDB/JM06OKlGz4uI9yn92moyoMJwQHylaNX0dJLATH H1AA== 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:references :cc:to:from:content-language:subject:user-agent:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:date:message-id :dkim-signature; bh=v7V9n1U3DQ5dMKf3Ht8pi3eoz6BvR5RoupOzzD7Q7Io=; fh=ksZansD5U+4SwnCe5XSfBs20ubp6qS0cOKRNB4rLVWY=; b=wP78k7ljPT2WvPmcNGioog1ERfh73WOdyNi3BfZVjWOIc3ObgNluaneSUWvn6vws/1 vF4PAv1U8o0FP2sT+P49EK1jgy7JQmfHoZLCKr8kqpNun7i9XZe6ckGXDzXIf6hPh6Qc /cKmDbSEpzEXl0Kwfedva7le8nBfMHC08mOHGVHeBoSa8PdfWgF2iC6D6gGmK1j2e+VC 1IXwf96DAZanXnQL6eVSZmQnnbbMhOwHwZg4aa2I4YpEcnJObrouSw2ZpegMfnjTeA76 3WfrmknJ2tq/br3ru2agpIddJ65iMqz9IJW6ZCoyGpYYQFEOOlb+vG4s8OHyaIgWl3cY my7g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=njxlL7Ss; spf=pass (google.com: domain of linux-kernel+bounces-10985-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-10985-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [2604:1380:4601:e00::3]) by mx.google.com with ESMTPS id cb17-20020a0564020b7100b005545b670e4esi3100831edb.525.2023.12.25.02.20.47 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 25 Dec 2023 02:20:47 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-10985-ouuuleilei=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=njxlL7Ss; spf=pass (google.com: domain of linux-kernel+bounces-10985-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-10985-ouuuleilei=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 DC44A1F22017 for ; Mon, 25 Dec 2023 10:20:46 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 7B02835F13; Mon, 25 Dec 2023 10:20:30 +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="njxlL7Ss" 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 138722E41A; Mon, 25 Dec 2023 10:20:25 +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=1703499619; x=1704104419; i=markus.elfring@web.de; bh=BcDjKBOyEaRKAxrqJ8HhD9NG+93EcGaceaR8TsHG4xc=; h=X-UI-Sender-Class:Date:Subject:From:To:Cc:References: In-Reply-To; b=njxlL7SsWq3EoKB7t/Stasqu/iiTq+qCDkeVWW0/9HtseAk/GOLk5ethgcI2gcTt B5H0dLfJmNZa9TrD/nGl0zsSLEBuevJ1XHeWDmXl7qHJYbFJXMpU/psZmIbOBcVXK xQgTBdyuajcPq7j5rKQu7AZjKbudA2ipENUZqm2s3f/i6sw59D5TPq/1Hmevvqj15 13fcmAFP9ikphFnmWyN5ntPfs0z0w5MNGsI0A12YbrrZkHGM2ZgOy14m5lD1S+c6d cfXx5XIvWcesJ9b1FaNm8Kamh7c1t9avy3NcCOogGx6GjvOnyFjXNAb6t2dNG5fN6 x9i4gb4pMI+gj72EiQ== X-UI-Sender-Class: 814a7b36-bfc1-4dae-8640-3722d8ec6cd6 Received: from [192.168.178.21] ([94.31.85.95]) by smtp.web.de (mrweb006 [213.165.67.108]) with ESMTPSA (Nemesis) id 1MWz8v-1rjzMA2EUD-00XEuK; Mon, 25 Dec 2023 11:20:19 +0100 Message-ID: <04ced4fa-90b5-4dd7-8f50-2f898b2c0855@web.de> Date: Mon, 25 Dec 2023 11:20:18 +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: [PATCH 2/3] dmaengine: timb_dma: Improve a size determination in td_alloc_init_desc() Content-Language: en-GB From: Markus Elfring To: dmaengine@vger.kernel.org, kernel-janitors@vger.kernel.org, Vinod Koul Cc: LKML , cocci@inria.fr References: In-Reply-To: X-Provags-ID: V03:K1:Hj+qur51bNboQvrcogJaAioT17jm3VMz9CFbZ6eqKgDxp9sPq3v IV5MHrUZQYKCpcBdE18ug+tw4+ty1LiaE29qw3xnn2FYGrLUa26k3TOgQPDQWkO1u0mtQ0j 2E72Wh6j/y1CYPwhaYjGTjmjvdcAB8vNwxCKDCkCTuqMy+yW6A22nTZLq8/n2nmoff6ijgv 2EMq14MgnJDhDnp6sXY0g== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:EechiKC28UA=;Avaih5tdOdZMgr/C+RawOPJ/OYI hYAadsHCz8Zdqx78XOgZrEtDT56dqhsT2AU9NKxcF5+HhzCvnOehzkgLI/JFnOPX2OeiICYMg izn2Xhj+qRneoJQ7TbEs6ebZItXTH+SmH8Hzs8qZv1qc/hWk1WmQIjTzN7GSVsj6cWTNuweOi 8HlhABAvO7XG+Z62u/YSetdFK3fO5VoY+EXaRWPsQtA3dk2VEAxtFd0LK/iRKR4qsb2eRbyd9 2by6EQc75pvclmCaYge+Hga3wQT8jIua77L7kgJzyeiy3drArTULhzoLz26yfrofUWT3b4x9D w+A1B+W0eOV//sUMuNiMz6ZCaKz/xwkHbjpJNH94SlRC+TzjKOS3968VseQjrRemqpzqBdQsS QHpqSK0REDz3doOtBwUuvyuL8LItkQNM9FlRAtdRqCBT265AG1UOteUgBraMR4fjHln8S6xN8 dmN37eRWGmsqUlsWS3Wo1eXV2H2P8keXRmO1OUIk2RJCV2ML7hb9VDARLWF7Gneg3C0/CUcr3 dsMML2P5nceDpesSCfuYhS+i3guv/NyV7A9IurMEBxgq1jJkpLY1BJbLxK1YGKMnl+szQzFCS onzYdHxoCEswNPTEKQZii0/eFkEzxxR1XVYlVu+Y+9MGuNY8Xv1p48hxyPHSCMWTHmk+b+Ot7 31iXQUuyb1kENTNO4Mls/Vv6rW2X1+hmgI/diazCEDUToODgqExarvjMPX95cTbKV8KnAByfE Xp2G3cZ5cc7caLfi6TTtH57b1l51ipCPuchjKppqsO+KoMfcQduv3IDQUAEcbWAMWKtVRURTc ylK/bQJ8XKcFDGXgcU7AwQvli1AY0SHAchwNh0sh9vi3zLi/7nOHlFQmK+C9qmAMQkgL5wkRr BKvQ8HXlqHaacDoSB2jOM2ZD8EjuVbrg7OKajEQJPGQ8+w3CuVLyaGUNqVcYhId3ObhRI8Kgo so3qbT8p6m0SA+mA/B6lR8wAmxk= X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1786248845952015203 X-GMAIL-MSGID: 1786248845952015203 From: Markus Elfring Date: Mon, 25 Dec 2023 10:50:10 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. Signed-off-by: Markus Elfring --- drivers/dma/timb_dma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.43.0 diff --git a/drivers/dma/timb_dma.c b/drivers/dma/timb_dma.c index abcab0b1ad3b..fc1f67bf9c06 100644 --- a/drivers/dma/timb_dma.c +++ b/drivers/dma/timb_dma.c @@ -325,7 +325,7 @@ static struct timb_dma_desc *td_alloc_init_desc(struct timb_dma_chan *td_chan) struct timb_dma_desc *td_desc; int err; - td_desc = kzalloc(sizeof(struct timb_dma_desc), GFP_KERNEL); + td_desc = kzalloc(sizeof(*td_desc), GFP_KERNEL); if (!td_desc) return NULL; From patchwork Mon Dec 25 10:22:24 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Markus Elfring X-Patchwork-Id: 183169 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:6f82:b0:100:9c79:88ff with SMTP id tb2csp324678dyb; Mon, 25 Dec 2023 02:22:53 -0800 (PST) X-Google-Smtp-Source: AGHT+IH5LEYlt2d86D0UEOogc82t6KxM3u3iAtcc/VLX5DcqPJgSmo7JP7WvC+mPPmZpSHjzfRS5 X-Received: by 2002:a05:6808:13c1:b0:3b8:bb95:97ef with SMTP id d1-20020a05680813c100b003b8bb9597efmr5655103oiw.48.1703499773108; Mon, 25 Dec 2023 02:22:53 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703499773; cv=none; d=google.com; s=arc-20160816; b=zZBuf3itjCs1uZ6rPwpyP7zS58Xwcz79a+7YVvZHhJ51YECNjvdThc8FxdRJZ6A7JP Rc7yXcIYOlJvgwigT+M154NyuCj6RmQnqGYu4rfexIrR2iNsRDi6+T1gULhtS6V4CzUh MdW9HIvJHcge1lenKV5MbaRonv8zSFOC0i0x9now57RY21lKXWaiEBkkJQLaJvQ2FB3f kt/H7h0Jk9FREqDkq/fjkzVlTQFTB/Y7eSwwTgu7Erv6HFO2DErdeV0vNAfr+Ib8M8Xr PfOcXQCool/OF4obVsJ7qNcG6UayqQ0Ov8yiKNflFnD5HrxhZW1sGm3sk2eIpIjyOH2w Bvgg== 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:references :cc:to:from:content-language:subject:user-agent:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:date:message-id :dkim-signature; bh=NB5WKIRnokEv8QZ/VdZaEjhlOe5tueLbLCFc1azf0KA=; fh=ksZansD5U+4SwnCe5XSfBs20ubp6qS0cOKRNB4rLVWY=; b=IUALZ6xEgjYMAaO41SVkDsXSEqEQI68K/xUy0coXsK4gmBWNgooSk8/OU5FwwN368j v4zK9Zx7np0Kscca2+Y7QAUOihYq2UYRG0iMHdNCWNSTO48Czcmz46ZYKs/x8f+gOJSL GsaKzscLx1eWe3/V0AHJpUDtakXYNBhKdNNMhNgYYb3bmL2bDdVcZrB2NBFWWRuXdgc4 6btph7GyBmyL9DmtIbJKNfdOmBO1ps7Fu32IdS9gMGuQqRri2dBA9gdG9zNO4OyVGEPg iZjwOyA4fFqdx6DlYz7S7xuH/Yik428Sn3fGaoRTsyV+FBIBhr4ITgfZ//jY+PV6hf3F ipbQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=oEszm892; spf=pass (google.com: domain of linux-kernel+bounces-10987-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-10987-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [2604:1380:45e3:2400::1]) by mx.google.com with ESMTPS id b191-20020a6334c8000000b005ce01410dddsi4091771pga.638.2023.12.25.02.22.52 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 25 Dec 2023 02:22:53 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-10987-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1; Authentication-Results: mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=oEszm892; spf=pass (google.com: domain of linux-kernel+bounces-10987-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-10987-ouuuleilei=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 sv.mirrors.kernel.org (Postfix) with ESMTPS id A8A95281C9C for ; Mon, 25 Dec 2023 10:22:52 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 2604F35F15; Mon, 25 Dec 2023 10:22:37 +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="oEszm892" 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 9A69935EF3; Mon, 25 Dec 2023 10:22:31 +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=1703499745; x=1704104545; i=markus.elfring@web.de; bh=ehzvFIgSeJSu3nZAjymVW905vwyKajE8awgJ7Isa/V4=; h=X-UI-Sender-Class:Date:Subject:From:To:Cc:References: In-Reply-To; b=oEszm8925iw94XfHKFlwJrQA3oD8t9iCFVEqmEKTAJn/d1LpUVHuRzeCtKqRSVop IDnFqQMAvI3kAaIOoVEmlDgOQyaAmlFWJNLZeLXGI6sb7sQE+j/OExm3fpVb1ygIb rmJGL0JuqwYEE1i4bmUKtlrlrSjEPZ2Jo+n/3pu7+TkKggu0n2qWDDWPDM+yGgEo8 K3dfYiV+T0Sa+s9ZA0887eMXL5SeQhF8jDos9A0bvHGW/9lk/8pywFlVdLcfHuf3/ 7ZlhgUL3tXfdcibkx0FY/Q025QRi3yaR8qPwVJK8ix3kO1zBKdvmnM65ZWtXVfSeQ 1hbRuY2FnOyb0KmseA== 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 1MeDQZ-1qk1702OPf-00auKj; Mon, 25 Dec 2023 11:22:25 +0100 Message-ID: <3184e80d-9056-4a0f-b0c0-8c1fefa62196@web.de> Date: Mon, 25 Dec 2023 11:22:24 +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: [PATCH 3/3] dmaengine: timb_dma: One function call less in td_alloc_init_desc() after error detection Content-Language: en-GB From: Markus Elfring To: dmaengine@vger.kernel.org, kernel-janitors@vger.kernel.org, Vinod Koul Cc: LKML , cocci@inria.fr References: In-Reply-To: X-Provags-ID: V03:K1:Rl1oMsp960JeDUAqYdAIMvOPYzMZfHe1CPjrgQS8GX9vY8IVCkF CO9ZPyUVlIUdzV1NfxsNHpGudjLmqX5lzPWD7LxC6H1HFeQ3L4oqImoT3Y/gWNHLFu5dyKz 3oYYWmS3TF5mA57xcx4hY8VEcZaVyew4VcCz1/Nu/QTUUZ3KIYkbZE4P1MywpCwQzl2BmU7 6wCx9mmTI5E005sLqqBXA== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:DM+q4z1Nt9I=;Yiy9v3GEYKJ7vkCTt0gbBFxvhr6 QQmXDJcB9xFHFIV/evpPM9LazU4h7+QdcqGygL7EizOHIuTT1vdcmXN5tzm502oFecGQzM/hM 27qsHBQR0sC6JnWnYUacaNYqdAVyidi7gG9vz38XV8PdrWvhFhnwtJVTkwVj59DLenAdPcrZt FgbvJ1spHeU9GQwE9PW84AhL6MjzinuMmWc2awjrhIcCWscB9lN2BMgTmacJtFOrHAbydXwIR PsF6/hQCrpFpBbetPGX8VdiLlDL/yb7IPr4Mqn7xBkVMQLrudPY635aAOSpm4tDyw/tU6PSMM z8kVNLlZRjXuYAe1PVx126xa8FchFE1tHGfhwByiJY66g+HzNbBPNyAbbqc9PqUJt6b/7cBNH VzPHmAoPLAQSAUhEP3Ye856LSNHveP6PdzT/mstARIomDHwKl3buDlKmwfa3ZwLt5Ot8PEA6K IxUyXxQJ/QP4Lm1jhCYDG0ew6a6/3jVMockeFZtQO5iPAMZI2L4b5kTW25QI24+ppXdNdXXIB 164iSOVFSGTKFa97n5+ASz35gV+FkVtHDwnPZMn6tCvUW0DjvXqE+tscu8bKHmuql72hfFAu2 /XWy/thQlMekjkNHpTdRhg4fqsQbkloBawkz70Hu3XivxMkbruJ63qWqYx66SPDoRFdHsQSHN e0QlVCxY9NmYA8rNFg7Qcf/XRdWtteakBztpRv0S+EjY9QtB9ynoXM5VJfCYVNmuUJyggfZER f2T2c/xAy4ptlWk6mdLAOxc3/jjZ6AjuXp12ukm6O9vrmjF0Ihc4mqTU3MTl7IAWOHPV53I77 3bJg7CTTQqYaKHLvK3cpDgxmVrbO0tCPEzH+9woWVn/0lAs7+9Vtpcku4G4yUUzP/XWomgsd6 6dBojfTz18rfvRP4+nffu5WWSMUf9eTSy9RxuxaPL+Dc/PRhupM5f99a46u9SK704wTtFDd07 xA/PZJj6AbZw62+6YvDgBaexhdU= X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1786248978431180114 X-GMAIL-MSGID: 1786248978431180114 From: Markus Elfring Date: Mon, 25 Dec 2023 10:55:52 +0100 The kfree() function was called in one case by the td_alloc_init_desc() function during error handling even if the passed data structure member contained a null pointer. This issue was detected by using the Coccinelle software. Thus use another label. Signed-off-by: Markus Elfring --- drivers/dma/timb_dma.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) -- 2.43.0 diff --git a/drivers/dma/timb_dma.c b/drivers/dma/timb_dma.c index fc1f67bf9c06..831c67af0237 100644 --- a/drivers/dma/timb_dma.c +++ b/drivers/dma/timb_dma.c @@ -333,7 +333,7 @@ static struct timb_dma_desc *td_alloc_init_desc(struct timb_dma_chan *td_chan) td_desc->desc_list = kzalloc(td_desc->desc_list_len, GFP_KERNEL); if (!td_desc->desc_list) - goto err; + goto free_td_desc; dma_async_tx_descriptor_init(&td_desc->txd, chan); td_desc->txd.tx_submit = td_tx_submit; @@ -351,4 +351,5 @@ static struct timb_dma_desc *td_alloc_init_desc(struct timb_dma_chan *td_chan) return td_desc; err: kfree(td_desc->desc_list); +free_td_desc: kfree(td_desc);