diff mbox series

[ovs-dev,v3,4/6] lldp: increase statsTLVsUnrecognizedTotal on unknown TLV

Message ID 20201112172533.271352-5-fdangelo@redhat.com
State Changes Requested
Headers show
Series Incorporate fixes from lldpd upstream | expand

Commit Message

Fabrizio D'Angelo Nov. 12, 2020, 5:25 p.m. UTC 
From: Vincent Bernat <vincent@bernat.ch>

Upstream commit:
	commit 109bcd423cd560545ec7940d73a50c5584aebb0c
	Author: Vincent Bernat <vincent@bernat.ch>
	Date: Sat, 6 Apr 2019 21:17:25 +0200

	This was done for organization TLVs, but not for other TLVs.

	Fix https://github.com/vincentbernat/lldpd/issues/323

Signed-off-by: Fabrizio D'Angelo <fdangelo@redhat.com>
Acked-by: Aaron Conole <aconole@redhat.com>
---
 lib/lldp/lldp.c | 1 +
 1 file changed, 1 insertion(+)

Comments

0-day Robot Nov. 12, 2020, 6:10 p.m. UTC | #1 
Bleep bloop.  Greetings Fabrizio D'Angelo, I am a robot and I have tried out your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
ERROR: Author Vincent Bernat <vincent@bernat.ch> needs to sign off.
WARNING: Unexpected sign-offs from developers who are not authors or co-authors or committers: Fabrizio D'Angelo <fdangelo@redhat.com>
Lines checked: 35, Warnings: 1, Errors: 1


Please check this out.  If you feel there has been an error, please email aconole@redhat.com

Thanks,
0-day Robot
diff mbox series

Patch

diff --git a/lib/lldp/lldp.c b/lib/lldp/lldp.c
index 5c9d34007..7a80c4aef 100644
--- a/lib/lldp/lldp.c
+++ b/lib/lldp/lldp.c
@@ -679,6 +679,7 @@  lldp_decode(struct lldpd *cfg OVS_UNUSED, char *frame, int s,
             VLOG_WARN("unknown tlv (%d) received on %s",
                       tlv_type,
                       hardware->h_ifname);
+            hardware->h_rx_unrecognized_cnt++;
             goto malformed;
         }
         if (pos > tlv + tlv_size) {