[U-Boot] [PATCH v2]net: null check after pointer dereference

[Vishwas Srivastava]

Author: Vishwas Srivastava <vishu.kernel at gmail.com>
Date:   Tue Jan 26 12:46:42 2016 +0530

null check after pointer dereference

NULL check is made after the pointer dereference in
davinci-emac driver. This patch fixes this issue.

Signed-off-by: Vishwas Srivastava <vishu.kernel at gmail.com>
CC: Joe Hershberger <joe.hershberger at ni.com>
Reviewed-by: Tom Rini <trini at konsulko.com>

Changes for v2:
        -cleaned up the style format
        -addressed various comments given by Joe <joe.hershberger at ni.com>
        on the first version of the patch.

diff --git a/drivers/net/davinci_emac.c b/drivers/net/davinci_emac.c
index 3f54a3f..a61374d 100644
--- a/drivers/net/davinci_emac.c
+++ b/drivers/net/davinci_emac.c
@@ -692,8 +692,10 @@ static int davinci_eth_rcv_packet (struct eth_device
*dev)
        davinci_invalidate_rx_descs();

        rx_curr_desc = emac_rx_active_head;
+       if (!rx_curr_desc)
+               return 0;
        status = rx_curr_desc->pkt_flag_len;
-       if ((rx_curr_desc) && ((status & EMAC_CPPI_OWNERSHIP_BIT) == 0)) {
+       if ((status & EMAC_CPPI_OWNERSHIP_BIT) == 0) {
                if (status & EMAC_CPPI_RX_ERROR_FRAME) {
                        /* Error in packet - discard it and requeue desc */
                        printf ("WARN: emac_rcv_pkt: Error in packet\n");


On Thu, Jan 28, 2016 at 1:35 AM, Joe Hershberger <joe.hershberger at gmail.com>
wrote:

> On Tue, Jan 26, 2016 at 1:26 AM, Vishwas Srivastava
> <vishu.kernel at gmail.com> wrote:
> > Author: Vishwas Srivastava <vishu.kernel at gmail.com>
> > Date:   Tue Jan 26 12:46:42 2016 +0530
>
> Please fix the subject of this patch. No space in the "net :".
>
> >
> >     null check after pointer dereference
>
> All of your commit logs should not have spaces in them (at the start of
> lines).
>
> >
> >     NULL check is made after the pointer dereference is davinci-
> >     emac driver. This patch fixes this issue.
> >
> >     CC: Sergey Kubushyn <ksi at koi8.net>;Joe Hershberger <
> > joe.hershberger at ni.com>
>
> Fix this format. One "Cc:" per line.
>
> >     Signed-off-by: Vishwas Srivastava <vishu.kernel at gmail.com>
>
> This belongs first... before the Cc:
>
> >
> > diff --git a/drivers/net/davinci_emac.c b/drivers/net/davinci_emac.c
> > index 3f54a3f..1966eb9 100644
> > --- a/drivers/net/davinci_emac.c
> > +++ b/drivers/net/davinci_emac.c
> > @@ -692,8 +692,10 @@ static int davinci_eth_rcv_packet (struct eth_device
> > *dev)
> >         davinci_invalidate_rx_descs();
> >
> >         rx_curr_desc = emac_rx_active_head;
>
> You need to run your patches through scripts/checkpatch.pl.
>
> > +       if (!rx_curr_desc)
> > +           return 0;
> >         status = rx_curr_desc->pkt_flag_len;
> > -       if ((rx_curr_desc) && ((status & EMAC_CPPI_OWNERSHIP_BIT) == 0))
> {
> > +       if ((status & EMAC_CPPI_OWNERSHIP_BIT) == 0) {
> >                 if (status & EMAC_CPPI_RX_ERROR_FRAME) {
> >                         /* Error in packet - discard it and requeue desc
> */
> >                         printf ("WARN: emac_rcv_pkt: Error in packet\n");
>
> Consider using tools/patman.
>
> Thanks,
> -Joe
>