]> git.proxmox.com Git - ovs.git/commit - AUTHORS.rst
rstp: Eliminate BPDU padding and uninitialized bytes.
authorBen Pfaff <blp@ovn.org>
Mon, 4 Jun 2018 20:42:10 +0000 (13:42 -0700)
committerBen Pfaff <blp@ovn.org>
Tue, 5 Jun 2018 18:17:13 +0000 (11:17 -0700)
commit36549a912b6b2fe90f4116420a3938e2062efd43
tree642715d308e1ffbd0cf8882f8519bdccb25b1680
parentbca4ff53aef16d38aeb1569edaaca6ac4feac6e8
rstp: Eliminate BPDU padding and uninitialized bytes.

When the RSTP implementation sent BPDUs, it failed to initialize some of
their bytes.  None of the code initialized an array of 7 padding bytes, and
some of it also failed to initialize the version1_length field.  In
addition, the padding bytes confused some implementations that did not
correctly ignore extra bytes.

This commit fixes both problems, by removing the padding bytes and
initializing every byte in outgoing messages.

Reported-by: David van Moolenbroek <dvmoolenbroek@aimvalley.nl>
Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2018-June/046864.html
Tested-by: David van Moolenbroek <dvmoolenbroek@aimvalley.nl>
Signed-off-by: Ben Pfaff <blp@ovn.org>
AUTHORS.rst
lib/rstp-common.h
lib/rstp-state-machines.c