linux-kernel - Re: [PATCH RESEND] drm/msm/dpu: Fix smatch warnings about variable dereferenced before check

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

Message-ID: <aXxqrD3ZzK_dTAmb@stanley.mountain>
Date: Fri, 30 Jan 2026 11:24:12 +0300
From: Dan Carpenter <dan.carpenter@...aro.org>
To: sunliming <sunliming@...ux.dev>
Cc: robin.clark@....qualcomm.com, lumag@...nel.org, airlied@...il.com,
	simona@...ll.ch, sean@...rly.run, marijn.suijten@...ainline.org,
	linux-arm-msm@...r.kernel.org, dri-devel@...ts.freedesktop.org,
	freedreno@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
	sunliming <sunliming@...inos.cn>, kernel test robot <lkp@...el.com>,
	Dan Carpenter <error27@...il.com>
Subject: Re: [PATCH RESEND] drm/msm/dpu: Fix smatch warnings about variable
 dereferenced before check

On Fri, Jan 30, 2026 at 01:30:35PM +0800, sunliming wrote:
> 
> On 2026/1/29 15:07, Dan Carpenter wrote:
> > On Thu, Jan 29, 2026 at 10:49:19AM +0800,sunliming@...ux.dev  wrote:
> > > From: sunliming<sunliming@...inos.cn>
> > > 
> > > Fix below smatch warnings:
> > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp_v13.c:161 dpu_hw_sspp_setup_pe_config_v13()
> > > warn: variable dereferenced before check 'ctx' (see line 159)
> > > 
> > > Reported-by: kernel test robot<lkp@...el.com>
> > > Reported-by: Dan Carpenter<error27@...il.com>
> > > Closes:https://lore.kernel.org/r/202601252214.oEaY3UZM-lkp@intel.com/
> > > Signed-off-by: sunliming<sunliming@...inos.cn>
> > > ---
> 
> Previously, a maintainer suggested that the name should match the email
> address,
> 

Yes.  But normally people change their email address instead of their
name.

> but community patches have consistently used the name "sunliming" since
> then.
> 

Consistency is less important than being correct...  It's just a weird
thing to say.  Are there other non-community patches with a different
name?  It's fine if that's your legal name and we have made exceptions
for people who aren't able to share their real name because of stalking
or whatever...

regards,
dan carpenter