Re: [PATCH 2/2][next] nouveau/svm: Split assignment from if conditional

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]<

To: "Gustavo A. R. Silva" <gustavoars@xxxxxxxxxx>
Subject: Re: [PATCH 2/2][next] nouveau/svm: Split assignment from if conditional
From: Kees Cook <keescook@xxxxxxxxxxxx>
Date: Wed, 16 Aug 2023 13:38:58 -0700
Cc: nouveau@xxxxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, dri-devel@xxxxxxxxxxxxxxxxxxxxx, Ben Skeggs <bskeggs@xxxxxxxxxx>, Daniel Vetter <daniel@xxxxxxxx>, linux-hardening@xxxxxxxxxxxxxxx
In-reply-to: <6b900e80b5587187c68efc788f5b042ca747d374.1692208802.git.gustavoars@kernel.org>
References: <cover.1692208802.git.gustavoars@kernel.org> <6b900e80b5587187c68efc788f5b042ca747d374.1692208802.git.gustavoars@kernel.org>

On Wed, Aug 16, 2023 at 12:05:06PM -0600, Gustavo A. R. Silva wrote:
> Fix checkpatch.pl ERROR: do not use assignment in if condition.
> 
> Signed-off-by: Gustavo A. R. Silva <gustavoars@xxxxxxxxxx>

Reviewed-by: Kees Cook <keescook@xxxxxxxxxxxx>

-- 
Kees Cook

References:
- [PATCH 0/2][next] nouveau/svm: Replace one-element array with flexible-array member
  - From: Gustavo A. R. Silva
- [PATCH 2/2][next] nouveau/svm: Split assignment from if conditional
  - From: Gustavo A. R. Silva

Prev by Date: Re: [PATCH 1/2][next] nouveau/svm: Replace one-element array with flexible-array member in struct nouveau_svm
Next by Date: Re: [PATCH] drm/nouveau/disp: fix use-after-free in error handling of nouveau_connector_create
Previous by thread: [PATCH 2/2][next] nouveau/svm: Split assignment from if conditional
Next by thread: Re: [PATCH 0/2][next] nouveau/svm: Replace one-element array with flexible-array member
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Security] [Bugtraq] [Linux] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]

Powered by Linux