<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 23 Jun 2024 at 14:22, Joshua Ashton <<a href="mailto:joshua@froggi.es">joshua@froggi.es</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Maybe that wasn't you or whatever, but your last patch that did this got <br>
some CVE assigned to it that didn't really make any sense, given this is <br>
just a null deref that'd end up as an oops?<br>
<br>
It can only happen if the kzalloc in drm_mode_create fails.<br>
<br>
I imagine that the `continue` is not the best cause of action anyway, <br>
it's probably not worth adding some broken connector with a bunch of <br>
missing modes.<br>
It's just going to make things appear more broken and be unexpected to <br>
userspace.<br>
<br>
Maybe this is some new thing that we do now I am out of the loop on...<br></blockquote><div><br></div><div>Kernel CVE handling changed a lot, for background:</div><div><br></div><div> <a href="https://lwn.net/Articles/978711/">https://lwn.net/Articles/978711/</a></div><div><br></div><div>No one needs to ask for CVEs, you'll all get them pretty much automatically for everything now.</div><div><br></div><div>Cheers, Sima</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
- Joshie 🐸✨<br>
<br>
On 6/23/24 10:20 AM, Joshua Ashton wrote:<br>
> Are you planning on submitting a bogus CVE for this patch too?<br>
> <br>
> - Joshie 🐸✨<br>
> <br>
> On June 22, 2024 9:22:19 AM GMT+01:00, Ma Ke <<a href="mailto:make24@iscas.ac.cn" target="_blank">make24@iscas.ac.cn</a>> wrote:<br>
>> In amdgpu_connector_add_common_modes(), the return value of drm_cvt_mode()<br>
>> is assigned to mode, which will lead to a NULL pointer dereference on<br>
>> failure of drm_cvt_mode(). Add a check to avoid npd.<br>
>><br>
>> Signed-off-by: Ma Ke <<a href="mailto:make24@iscas.ac.cn" target="_blank">make24@iscas.ac.cn</a>><br>
>> ---<br>
>> drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c | 2 ++<br>
>> 1 file changed, 2 insertions(+)<br>
>><br>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c<br>
>> index 9caba10315a8..6cf946adb6fe 100644<br>
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c<br>
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c<br>
>> @@ -458,6 +458,8 @@ static void amdgpu_connector_add_common_modes(struct drm_encoder *encoder,<br>
>>                      continue;<br>
>><br>
>>              mode = drm_cvt_mode(dev, common_modes[i].w, common_modes[i].h, 60, false, false, false);<br>
>> +            if (!mode)<br>
>> +                    continue;<br>
>>              drm_mode_probed_add(connector, mode);<br>
>>      }<br>
>> }<br>
<br>
</blockquote></div><br clear="all"><br><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature"><div dir="ltr">Daniel Vetter<br>Software Engineer, Intel Corporation<br><a href="http://blog.ffwll.ch" target="_blank">http://blog.ffwll.ch</a></div></div></div>