<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEEDINFO "
title="NEEDINFO - [BAT][BRW] NULL pointer dereference in snd_hda_codec_generic"
href="https://bugs.freedesktop.org/show_bug.cgi?id=101246#c8">Comment # 8</a>
on <a class="bz_bug_link
bz_status_NEEDINFO "
title="NEEDINFO - [BAT][BRW] NULL pointer dereference in snd_hda_codec_generic"
href="https://bugs.freedesktop.org/show_bug.cgi?id=101246">bug 101246</a>
from <span class="vcard"><a class="email" href="mailto:libin.yang@intel.com" title="Libin Yang <libin.yang@intel.com>"> <span class="fn">Libin Yang</span></a>
</span></b>
<pre>(In reply to Martin Peres from <a href="show_bug.cgi?id=101246#c7">comment #7</a>)
<span class="quote">> (In reply to Libin Yang from <a href="show_bug.cgi?id=101246#c6">comment #6</a>)
> > My suggestion test is to try to find why it is using generic driver.
>
> Hmm, I get your point, but whatever the reason why, it should still not make
> any null pointer de-reference.</span >
Yes, there seems 2 issues: 1) why using generic; 2) the null pointer.
For the second issue, it seems you've already find a solution. I recommend you
submit a patch.
<span class="quote">>
> However, did you see that this is for the i965G platform, which is 10 years
> old? Is the non-generic driver supporting this platform?</span >
We did not do any test on i965G platform. It's not our scope.
<span class="quote">>
> > However, if you can fix it in generic driver, please make a patch and send
> > it to community.
>
> Well, I already wrote a patch that papers around the issue and will still
> report it in the logs[1], but this is not a proper patch.
>
> I do not have any background in the sound subsystem, so I do not know what
> are the correct assumptions to have, especially around the binding and
> unbinding of the driver. I am merely doing some QA/CI job here, and
> reporting the issue to you, who own this code.
>
> I am however willing to help, but not by configuring the system differently
> in order to get rid of the problem. This would be a workaround, not a fix.
>
> [1] <a href="https://patchwork.freedesktop.org/patch/160525/">https://patchwork.freedesktop.org/patch/160525/</a></span >
Please feel free to make the patch and submit it. Takashi is very professional
and nice. I did think we need check the null pointer. But maybe it is not as
your patch. Maybe it is in another place.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
<li>You are on the CC list for the bug.</li>
</ul>
</body>
</html>