[ovs-dev] [PATCH] ovs-lldp: Remove unused variable 'lldp_size'.

Dennis Flynn dflynn20 at gmail.com
Fri Apr 17 14:49:09 UTC 2015


Acked-by: Dennis Flynn <drflynn at avaya.com>

On Thu, Apr 16, 2015 at 4:44 PM, Ben Pfaff <blp at nicira.com> wrote:

> Doesn't seem to be good for anything.
>
> Found by LLVM scan-build.
>
> Reported-by: Kevin Lo <kevlo at FreeBSD.org>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
>  lib/ovs-lldp.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/lib/ovs-lldp.c b/lib/ovs-lldp.c
> index 9ee639e..61534a4 100644
> --- a/lib/ovs-lldp.c
> +++ b/lib/ovs-lldp.c
> @@ -728,7 +728,6 @@ lldp_put_packet(struct lldp *lldp, struct dp_packet
> *packet,
>  {
>      struct lldpd *mylldpd = lldp->lldpd;
>      struct lldpd_hardware *hw = lldpd_first_hardware(mylldpd);
> -    uint32_t lldp_size = 0;
>      static const uint8_t eth_addr_lldp[6] =
>          {0x01, 0x80, 0xC2, 0x00, 0x00, 0x0e};
>
> @@ -736,10 +735,7 @@ lldp_put_packet(struct lldp *lldp, struct dp_packet
> *packet,
>
>      eth_compose(packet, eth_addr_lldp, eth_src, ETH_TYPE_LLDP, 0);
>
> -    lldp_size = lldpd_send(hw, packet);
> -    if (lldp_size + ETH_HEADER_LEN < MINIMUM_ETH_PACKET_SIZE) {
> -        lldp_size = MINIMUM_ETH_PACKET_SIZE;
> -    }
> +    lldpd_send(hw, packet);
>
>      timer_set_duration(&lldp->tx_timer,
> lldp->lldpd->g_config.c_tx_interval);
>      ovs_mutex_unlock(&mutex);
> --
> 2.1.3
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>



More information about the dev mailing list