Re: [PATCH v3] staging: ks7010: Add null pointer check for skb

From: Dan Carpenter
Date: Fri Sep 28 2018 - 04:22:55 EST


On Thu, Sep 27, 2018 at 07:04:43PM +0100, Aymen Qader wrote:
> Retraction: in hindsight I see that with the current usage of this
> function, there is already a check for the socket buffer so this check
> is unnecessary. However, I'm not sure if it's considered good practice
> to keep this check anyway--in any case, ENOMEM isn't the right error
> to return.

When we find inconsistent NULL checks then we fix it to make sense.
Generally, we prefer a minimal style, with no extra code for future
proofing. (The future seldom goes the way you expect and those extra
NULL checks would be easy to add back).

So, yes, do remove the NULL check but also fix the indenting while
you're at it.

Take your time to write patches. I write them, then I sit on them over
night then I send them in the morning. It means that sometimes other
people have already sent it but that's fine. If you have to redo a
patch then don't send the v2 patch on the same day. v2 patches are
stressful and you imagine that everyone is waiting for you to send it
or something. We are not waiting for you. We don't care if you wait
until next week to send these...

So when you write a v2 patch wait until the next day to send it. Then
you will be calm when you review it.

regards,
dan carpenter