[ovs-build] Passed: ovsrobot/ovs#251 (series_87396 - 023bd1c)

Travis CI builds at travis-ci.org
Mon Jan 21 21:57:04 UTC 2019


Build Update for ovsrobot/ovs
-------------------------------------

Build: #251
Status: Passed

Duration: 59 mins and 29 secs
Commit: 023bd1c (series_87396)
Author: David Marchand
Message: conntrack: fix ftp ipv4 address substitution.

When replacing the ipv4 address in repl_ftp_v4_addr(), the remaining size
was incorrectly calculated which could lead to the wrong replacement
adjustment.

This goes unnoticed most of the time, unless you choose carefully your
initial and replacement addresses.

Example fail address combination with 10.1.1.200 DNAT'd to 10.1.100.1.

Fix this by getting rid of remain_size since it is a derived variable
and use the primary variable byte_str instead.

A test is updated to exercise different initial and replacement addresses
and another test is added.

Fixes: bd5e81a0e596 ("Userspace Datapath: Add ALG infra and FTP.")
Co-authored-by: Darrell Ball <dlu998 at gmail.com>
Signed-off-by: Darrell Ball <dlu998 at gmail.com>
Signed-off-by: David Marchand <david.marchand at redhat.com>
Signed-off-by: 0-day Robot <robot at bytheb.org>

View the changeset: https://github.com/ovsrobot/ovs/commit/023bd1c5f1c1

View the full build log and details: https://travis-ci.org/ovsrobot/ovs/builds/482602227?utm_medium=notification&utm_source=email

--

You can unsubscribe from build emails from the ovsrobot/ovs repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=22285853&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...
URL: <http://mail.openvswitch.org/pipermail/ovs-build/attachments/20190121/006fdfde/attachment.html>


More information about the build mailing list