summaryrefslogtreecommitdiff
path: root/libpod
diff options
context:
space:
mode:
authorMatthew Heon <matthew.heon@pm.me>2020-06-02 16:31:01 -0400
committerMatthew Heon <matthew.heon@pm.me>2020-06-02 17:58:52 -0400
commit42505f64d29e73106dc29658fe7e852b9a5421c5 (patch)
tree15a9c8c7edccb631f7dfd31ab81c1dcba241c8a0 /libpod
parentc4ccd7cbc1509bab6183c47f740cbf2cc4ee0424 (diff)
downloadpodman-42505f64d29e73106dc29658fe7e852b9a5421c5.tar.gz
podman-42505f64d29e73106dc29658fe7e852b9a5421c5.tar.bz2
podman-42505f64d29e73106dc29658fe7e852b9a5421c5.zip
Properly follow linked namespace container for stats
Podman containers can specify that they get their network namespace from another container. This is automatic in pods, but any container can do it. The problem is that these containers are not guaranteed to have a network namespace of their own; it is perfectly valid to join the network namespace of a --net=host container, and both containers will end up in the host namespace. The code for obtaining network stats did not account for this, and could cause segfaults as a result. Fortunately, the fix is simple - the function we use to get said stats already performs appropriate checks, so we just need to recursively call it. Fixes #5652 Signed-off-by: Matthew Heon <matthew.heon@pm.me>
Diffstat (limited to 'libpod')
-rw-r--r--libpod/networking_linux.go2
1 files changed, 1 insertions, 1 deletions
diff --git a/libpod/networking_linux.go b/libpod/networking_linux.go
index 83344ebbe..0c9d28701 100644
--- a/libpod/networking_linux.go
+++ b/libpod/networking_linux.go
@@ -550,7 +550,7 @@ func getContainerNetNS(ctr *Container) (string, error) {
if err = c.syncContainer(); err != nil {
return "", err
}
- return c.state.NetNS.Path(), nil
+ return getContainerNetNS(c)
}
return "", nil
}