* [Intel-gfx] [bug report] drm/i915/tc: Fix TypeC PHY connect/disconnect logic on ADL-P
@ 2021-10-01 13:25 Dan Carpenter
2021-10-01 14:20 ` Ville Syrjälä
0 siblings, 1 reply; 3+ messages in thread
From: Dan Carpenter @ 2021-10-01 13:25 UTC (permalink / raw)
To: imre.deak; +Cc: intel-gfx
Hello Imre Deak,
This is a semi-automatic email about new static checker warnings.
The patch 3e0abc7661c8: "drm/i915/tc: Fix TypeC PHY
connect/disconnect logic on ADL-P" from Sep 29, 2021, leads to the
following Smatch complaint:
drivers/gpu/drm/i915/display/intel_ddi.c:4028 intel_ddi_encoder_destroy()
warn: variable dereferenced before check 'dig_port' (see line 4020)
drivers/gpu/drm/i915/display/intel_ddi.c
4019 struct intel_digital_port *dig_port = enc_to_dig_port(to_intel_encoder(encoder));
4020 enum phy phy = intel_port_to_phy(i915, dig_port->base.port);
^^^^^^^^^^^^^^^^^^^
The patch adds a new unchecked dereference.
4021
4022 intel_dp_encoder_flush_work(encoder);
4023 if (intel_phy_is_tc(i915, phy))
4024 intel_tc_port_flush_work(dig_port);
4025 intel_display_power_flush_work(i915);
4026
4027 drm_encoder_cleanup(encoder);
4028 if (dig_port)
^^^^^^^^
But the existing code checked for NULL.
4029 kfree(dig_port->hdcp_port_data.streams);
4030 kfree(dig_port);
regards,
dan carpenter
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [Intel-gfx] [bug report] drm/i915/tc: Fix TypeC PHY connect/disconnect logic on ADL-P
2021-10-01 13:25 [Intel-gfx] [bug report] drm/i915/tc: Fix TypeC PHY connect/disconnect logic on ADL-P Dan Carpenter
@ 2021-10-01 14:20 ` Ville Syrjälä
2021-10-01 14:52 ` Dan Carpenter
0 siblings, 1 reply; 3+ messages in thread
From: Ville Syrjälä @ 2021-10-01 14:20 UTC (permalink / raw)
To: Dan Carpenter; +Cc: imre.deak, intel-gfx
On Fri, Oct 01, 2021 at 04:25:35PM +0300, Dan Carpenter wrote:
> Hello Imre Deak,
>
> This is a semi-automatic email about new static checker warnings.
>
> The patch 3e0abc7661c8: "drm/i915/tc: Fix TypeC PHY
> connect/disconnect logic on ADL-P" from Sep 29, 2021, leads to the
> following Smatch complaint:
>
> drivers/gpu/drm/i915/display/intel_ddi.c:4028 intel_ddi_encoder_destroy()
> warn: variable dereferenced before check 'dig_port' (see line 4020)
>
> drivers/gpu/drm/i915/display/intel_ddi.c
> 4019 struct intel_digital_port *dig_port = enc_to_dig_port(to_intel_encoder(encoder));
> 4020 enum phy phy = intel_port_to_phy(i915, dig_port->base.port);
> ^^^^^^^^^^^^^^^^^^^
> The patch adds a new unchecked dereference.
>
> 4021
> 4022 intel_dp_encoder_flush_work(encoder);
> 4023 if (intel_phy_is_tc(i915, phy))
> 4024 intel_tc_port_flush_work(dig_port);
> 4025 intel_display_power_flush_work(i915);
> 4026
> 4027 drm_encoder_cleanup(encoder);
> 4028 if (dig_port)
> ^^^^^^^^
> But the existing code checked for NULL.
That check is nonsense. Feel free to nuke it.
>
> 4029 kfree(dig_port->hdcp_port_data.streams);
> 4030 kfree(dig_port);
>
> regards,
> dan carpenter
--
Ville Syrjälä
Intel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Intel-gfx] [bug report] drm/i915/tc: Fix TypeC PHY connect/disconnect logic on ADL-P
2021-10-01 14:20 ` Ville Syrjälä
@ 2021-10-01 14:52 ` Dan Carpenter
0 siblings, 0 replies; 3+ messages in thread
From: Dan Carpenter @ 2021-10-01 14:52 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: imre.deak, intel-gfx
On Fri, Oct 01, 2021 at 05:20:17PM +0300, Ville Syrjälä wrote:
> On Fri, Oct 01, 2021 at 04:25:35PM +0300, Dan Carpenter wrote:
> > Hello Imre Deak,
> >
> > This is a semi-automatic email about new static checker warnings.
> >
> > The patch 3e0abc7661c8: "drm/i915/tc: Fix TypeC PHY
> > connect/disconnect logic on ADL-P" from Sep 29, 2021, leads to the
> > following Smatch complaint:
> >
> > drivers/gpu/drm/i915/display/intel_ddi.c:4028 intel_ddi_encoder_destroy()
> > warn: variable dereferenced before check 'dig_port' (see line 4020)
> >
> > drivers/gpu/drm/i915/display/intel_ddi.c
> > 4019 struct intel_digital_port *dig_port = enc_to_dig_port(to_intel_encoder(encoder));
> > 4020 enum phy phy = intel_port_to_phy(i915, dig_port->base.port);
> > ^^^^^^^^^^^^^^^^^^^
> > The patch adds a new unchecked dereference.
> >
> > 4021
> > 4022 intel_dp_encoder_flush_work(encoder);
> > 4023 if (intel_phy_is_tc(i915, phy))
> > 4024 intel_tc_port_flush_work(dig_port);
> > 4025 intel_display_power_flush_work(i915);
> > 4026
> > 4027 drm_encoder_cleanup(encoder);
> > 4028 if (dig_port)
> > ^^^^^^^^
> > But the existing code checked for NULL.
>
> That check is nonsense. Feel free to nuke it.
Thanks! Will do.
regards,
dan carpenter
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-10-01 14:53 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-01 13:25 [Intel-gfx] [bug report] drm/i915/tc: Fix TypeC PHY connect/disconnect logic on ADL-P Dan Carpenter
2021-10-01 14:20 ` Ville Syrjälä
2021-10-01 14:52 ` Dan Carpenter
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.