You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ab...@apache.org on 2019/04/09 15:27:09 UTC

[mesos] branch 1.5.x updated (136178e -> 89e9505)

This is an automated email from the ASF dual-hosted git repository.

abudnik pushed a change to branch 1.5.x
in repository https://gitbox.apache.org/repos/asf/mesos.git.


    from 136178e  Added MESOS-9529 to 1.5.4 CHANGELOG.
     new d24d0c7  Fix `link::lo()` to deal with `None` value.
     new 89e9505  Added MESOS-9707 to the 1.5.4 CHANGELOG.

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 CHANGELOG                       | 1 +
 src/linux/routing/link/link.cpp | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)


[mesos] 01/02: Fix `link::lo()` to deal with `None` value.

Posted by ab...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

abudnik pushed a commit to branch 1.5.x
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit d24d0c71f6e4b6f5c8b98c0f90ec2728fc0372b2
Author: Pavel Kirillov <49...@users.noreply.github.com>
AuthorDate: Tue Apr 9 15:44:24 2019 +0200

    Fix `link::lo()` to deal with `None` value.
    
    Some network links may cause `link::internal::test(link, IFF_LOOPBACK)`
    to be `None`.
    `Link::lo` should deal with `None` values of test var.
    
    This closes #330
---
 src/linux/routing/link/link.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/linux/routing/link/link.cpp b/src/linux/routing/link/link.cpp
index 5388a3d..ef83bc8 100644
--- a/src/linux/routing/link/link.cpp
+++ b/src/linux/routing/link/link.cpp
@@ -99,7 +99,7 @@ Result<string> lo()
     Result<bool> test = link::internal::test(link, IFF_LOOPBACK);
     if (test.isError()) {
       return Error("Failed to check the flag on link: " + link);
-    } else if (test.get()) {
+    } else if (test.isSome() && test.get()) {
       return link;
     }
   }


[mesos] 02/02: Added MESOS-9707 to the 1.5.4 CHANGELOG.

Posted by ab...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

abudnik pushed a commit to branch 1.5.x
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit 89e9505637726f38cf115c5ed52dec9f2a4a17e0
Author: Andrei Budnik <ab...@mesosphere.com>
AuthorDate: Tue Apr 9 17:26:16 2019 +0200

    Added MESOS-9707 to the 1.5.4 CHANGELOG.
---
 CHANGELOG | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGELOG b/CHANGELOG
index ba2f739..023c927 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -4,6 +4,7 @@ Release Notes - Mesos - Version 1.5.4 (WIP)
 
 ** Bug
   * [MESOS-9529] - `/proc` should be remounted even if a nested container set `share_pid_namespace` to true.
+  * [MESOS-9707] - Calling link::lo() may cause runtime error
 
 
 Release Notes - Mesos - Version 1.5.3