Skip to content

Commit 97b707f

Browse files
committed
[test] Switch to github.com/moby/sys/capability v0.1.2
This is to test moby/sys#149 Signed-off-by: Kir Kolyshkin <[email protected]>
1 parent b3cbc1e commit 97b707f

File tree

16 files changed

+194
-231
lines changed

16 files changed

+194
-231
lines changed

go.mod

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,14 +10,14 @@ require (
1010
github.com/cyphar/filepath-securejoin v0.2.4
1111
github.com/docker/go-units v0.5.0
1212
github.com/godbus/dbus/v5 v5.1.0
13+
github.com/moby/sys/capability v0.1.1
1314
github.com/moby/sys/mountinfo v0.7.1
1415
github.com/moby/sys/user v0.1.0
1516
github.com/mrunalp/fileutils v0.5.1
1617
github.com/opencontainers/runtime-spec v1.2.0
1718
github.com/opencontainers/selinux v1.11.0
1819
github.com/seccomp/libseccomp-golang v0.10.0
1920
github.com/sirupsen/logrus v1.9.3
20-
github.com/syndtr/gocapability v0.0.0-20200815063812-42c35b437635
2121
github.com/urfave/cli v1.22.14
2222
github.com/vishvananda/netlink v1.1.0
2323
golang.org/x/net v0.24.0
@@ -31,3 +31,6 @@ require (
3131
github.com/vishvananda/netns v0.0.0-20191106174202-0a2b9b5464df // indirect
3232
golang.org/x/exp v0.0.0-20230224173230-c95f2b4c22f2 // indirect
3333
)
34+
35+
// https://github.com/moby/sys/pull/149
36+
replace github.com/moby/sys/capability => github.com/kolyshkin/sys/capability v0.0.0-20240912003154-6a5bb117f70e

go.sum

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,8 @@ github.com/google/go-cmp v0.5.5/go.mod h1:v8dTdLbMG2kIc/vJvl+f65V22dbkXbowE6jgT/
2626
github.com/google/go-cmp v0.5.9 h1:O2Tfq5qg4qc4AmwVlvv0oLiVAGB7enBSJ2x2DqQFi38=
2727
github.com/google/go-cmp v0.5.9/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY=
2828
github.com/inconshreveable/mousetrap v1.0.0/go.mod h1:PxqpIevigyE2G7u3NXJIT2ANytuPF1OarO4DADm73n8=
29+
github.com/kolyshkin/sys/capability v0.0.0-20240912003154-6a5bb117f70e h1:nlkrAErJGbzaphFzBOnjKwuIsV6h/V4uV18A5osgCUU=
30+
github.com/kolyshkin/sys/capability v0.0.0-20240912003154-6a5bb117f70e/go.mod h1:4g9IK291rVkms3LKCDOoYlnV8xKwoDTpIrNEE35Wq0I=
2931
github.com/kr/pretty v0.3.1 h1:flRD4NNwYAUpkphVc1HcthR4KEIFJ65n8Mw5qdRn3LE=
3032
github.com/kr/pretty v0.3.1/go.mod h1:hoEshYVHaxMs3cyo3Yncou5ZscifuDolrwPKZanG3xk=
3133
github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY=
@@ -60,8 +62,6 @@ github.com/stretchr/testify v1.7.1/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/
6062
github.com/stretchr/testify v1.8.0/go.mod h1:yNjHg4UonilssWZ8iaSj1OCr/vHnekPRkoO+kdMU+MU=
6163
github.com/stretchr/testify v1.8.4 h1:CcVxjf3Q8PM0mHUKJCdn+eZZtm5yQwehR5yeSVQQcUk=
6264
github.com/stretchr/testify v1.8.4/go.mod h1:sz/lmYIOXD/1dqDmKjjqLyZ2RngseejIcXlSw2iwfAo=
63-
github.com/syndtr/gocapability v0.0.0-20200815063812-42c35b437635 h1:kdXcSzyDtseVEc4yCz2qF8ZrQvIDBJLl4S1c3GCXmoI=
64-
github.com/syndtr/gocapability v0.0.0-20200815063812-42c35b437635/go.mod h1:hkRG7XYTFWNJGYcbNJQlaLq0fg1yr4J4t/NcTQtrfww=
6565
github.com/urfave/cli v1.22.14 h1:ebbhrRiGK2i4naQJr+1Xj92HXZCrK7MsyTS/ob3HnAk=
6666
github.com/urfave/cli v1.22.14/go.mod h1:X0eDS6pD6Exaclxm99NJ3FiCDRED7vIHpx2mDOHLvkA=
6767
github.com/vishvananda/netlink v1.1.0 h1:1iyaYNBLmP6L0220aDnYQpo1QEV4t4hJ+xEEhhJH8j0=

libcontainer/capabilities/capabilities.go

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,9 @@ import (
88
"strings"
99
"sync"
1010

11+
"github.com/moby/sys/capability"
1112
"github.com/opencontainers/runc/libcontainer/configs"
1213
"github.com/sirupsen/logrus"
13-
"github.com/syndtr/gocapability/capability"
1414
)
1515

1616
const allCapabilityTypes = capability.CAPS | capability.BOUNDING | capability.AMBIENT
@@ -24,15 +24,19 @@ var (
2424
capability.AMBIENT,
2525
}
2626

27-
capMap = sync.OnceValue(func() map[string]capability.Cap {
28-
cm := make(map[string]capability.Cap, capability.CAP_LAST_CAP+1)
27+
capMap = sync.OnceValues(func() (map[string]capability.Cap, error) {
28+
last, err := capability.LastCap()
29+
if err != nil {
30+
return nil, err
31+
}
32+
cm := make(map[string]capability.Cap, last+1)
2933
for _, c := range capability.List() {
30-
if c > capability.CAP_LAST_CAP {
34+
if c > last {
3135
continue
3236
}
3337
cm["CAP_"+strings.ToUpper(c.String())] = c
3438
}
35-
return cm
39+
return cm, nil
3640
})
3741
)
3842

@@ -51,12 +55,12 @@ func KnownCapabilities() []string {
5155
// or Capabilities that are unavailable in the current environment are ignored,
5256
// printing a warning instead.
5357
func New(capConfig *configs.Capabilities) (*Caps, error) {
54-
var (
55-
err error
56-
c Caps
57-
)
58+
var c Caps
5859

59-
cm := capMap()
60+
cm, err := capMap()
61+
if err != nil {
62+
return nil, err
63+
}
6064
unknownCaps := make(map[string]struct{})
6165
ignoredCaps := make(map[string]struct{})
6266
// capSlice converts the slice of capability names in caps, to their numeric

libcontainer/capabilities/capabilities_linux_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,10 @@ import (
55
"os"
66
"testing"
77

8+
"github.com/moby/sys/capability"
89
"github.com/opencontainers/runc/libcontainer/configs"
910
"github.com/sirupsen/logrus"
1011
"github.com/sirupsen/logrus/hooks/test"
11-
"github.com/syndtr/gocapability/capability"
1212
)
1313

1414
func TestNew(t *testing.T) {

vendor/github.com/moby/sys/capability/.codespellrc

Lines changed: 3 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/github.com/moby/sys/capability/.golangci.yml

Lines changed: 6 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/github.com/moby/sys/capability/CHANGELOG.md

Lines changed: 41 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/github.com/moby/sys/capability/README.md

Lines changed: 11 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/github.com/syndtr/gocapability/capability/capability.go renamed to vendor/github.com/moby/sys/capability/capability.go

Lines changed: 24 additions & 24 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)