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

Joe Hershberger joe.hershberger at gmail.com
Wed Jan 27 21:05:28 CET 2016


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


More information about the U-Boot mailing list