[ovs-build] Broken: booxter/ovn#3 (dhcp-broadcast - 591a6bc)
builds at travis-ci.org
Thu Mar 5 19:55:59 UTC 2020
Build Update for booxter/ovn
Duration: 17 mins and 17 secs
Commit: 591a6bc (dhcp-broadcast)
Author: Ihar Hrachyshka
Message: Broadcast DHCPREPLY when BROADCAST flag is set
As per RFC2131, section 4.1:
A server or relay agent sending or relaying a DHCP message directly
to a DHCP client (i.e., not to a relay agent specified in the
'giaddr' field) SHOULD examine the BROADCAST bit in the 'flags'
field. If this bit is set to 1, the DHCP message SHOULD be sent as
an IP broadcast using an IP broadcast address (preferably 0xffffffff)
as the IP destination address and the link-layer broadcast address as
the link-layer destination address.
This patch changes destination IP address to 255.255.255.255 when client
set BROADCAST flag in their DHCPREQUEST. Note: the offered IP address is
still part of the DHCP payload.
Signed-off-by: Ihar Hrachyshka <ihrachys at redhat.com>
* Introduced DHCP_BROADCAST_FLAG macro.
* Converted DHCP_FLAGS_BROADCAST into a static function.
* Used full sentences in comments (added missing dot).
* Negated if-condition to handle least common case in else branch.
* Fixed build with --enable-sparse --enable-Werror.
* Updated ovn-northd documentation to reflect new flow actions.
* Fixed line length warning reported by checkpatch.
View the changeset: https://github.com/booxter/ovn/compare/b0db764dfaef...591a6bc496db
View the full build log and details: https://travis-ci.org/booxter/ovn/builds/658846761?utm_medium=notification&utm_source=email
You can unsubscribe from build emails from the booxter/ovn repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=27978448&utm_medium=notification&utm_source=email.
Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notification&utm_source=email.
Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the build