<div dir="ltr">Hi Fedor!,<div><br><div>Sure, In fact, there were two scenarios from the beginning, either <br>redundant condition or potential NULL pointer dereference.I relied on <br>the condition to determine the type of issue because I did not find <br>it logical to add a useless condition, but based on the words Wolfram <br>Sang "we trust ourselves here." then the scenario will change to <br>redundant condition, so i'll write a new patch and send it in new <br>thread.<br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Feb 4, 2024 at 11:54 AM Fedor Pchelkin <<a href="mailto:pchelkin@ispras.ru">pchelkin@ispras.ru</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">On 24/02/03 09:44PM, Wolfram Sang wrote:<br>
> <br>
> > If I'm not missing something, npcm_i2c_reg_slave() is called via a<br>
> > function pointer ->reg_slave here [1]. And seems `client` can't be NULL<br>
> > there. Other drivers implementing ->reg_slave function don't check its<br>
> > argument.<br>
> <br>
> Correct, we trust ourselves here.<br>
> <br>
> > Maybe we should just drop `if (!bus->slave)` check?<br>
> <br>
> Yes.<br>
> <br>
<br>
Okay, thanks for confirmation.<br>
<br>
Rand, would you like to prepare the patch, please?<br>
<br>
</blockquote></div>