diff options
author | gVisor bot <gvisor-bot@google.com> | 2020-12-12 00:05:43 +0000 |
---|---|---|
committer | gVisor bot <gvisor-bot@google.com> | 2020-12-12 00:05:43 +0000 |
commit | 445e57ac75a4c48e5455fdaf5b3e963594c15346 (patch) | |
tree | c2794d96cddf521e40e74bc0d26d000133cf3139 /pkg/sentry/fsimpl/kernfs | |
parent | 0aa7d2a4b264764152ef79db960e920f2e101780 (diff) | |
parent | 1e92732eb19ac5cfa3df6ff01cc1ea71d80a9198 (diff) |
Merge release-20201208.0-36-g1e92732eb (automated)
Diffstat (limited to 'pkg/sentry/fsimpl/kernfs')
-rw-r--r-- | pkg/sentry/fsimpl/kernfs/static_directory_refs.go | 9 | ||||
-rw-r--r-- | pkg/sentry/fsimpl/kernfs/synthetic_directory_refs.go | 9 |
2 files changed, 4 insertions, 14 deletions
diff --git a/pkg/sentry/fsimpl/kernfs/static_directory_refs.go b/pkg/sentry/fsimpl/kernfs/static_directory_refs.go index cdf6374a3..88cba9456 100644 --- a/pkg/sentry/fsimpl/kernfs/static_directory_refs.go +++ b/pkg/sentry/fsimpl/kernfs/static_directory_refs.go @@ -54,11 +54,6 @@ func (r *StaticDirectoryRefs) LogRefs() bool { return StaticDirectoryenableLogging } -// EnableLeakCheck enables reference leak checking on r. -func (r *StaticDirectoryRefs) EnableLeakCheck() { - refsvfs2.Register(r) -} - // ReadRefs returns the current number of references. The returned count is // inherently racy and is unsafe to use without external synchronization. func (r *StaticDirectoryRefs) ReadRefs() int64 { @@ -115,7 +110,7 @@ func (r *StaticDirectoryRefs) TryIncRef() bool { func (r *StaticDirectoryRefs) DecRef(destroy func()) { v := atomic.AddInt64(&r.refCount, -1) if StaticDirectoryenableLogging { - refsvfs2.LogDecRef(r, v+1) + refsvfs2.LogDecRef(r, v) } switch { case v < 0: @@ -132,6 +127,6 @@ func (r *StaticDirectoryRefs) DecRef(destroy func()) { func (r *StaticDirectoryRefs) afterLoad() { if r.ReadRefs() > 0 { - r.EnableLeakCheck() + refsvfs2.Register(r) } } diff --git a/pkg/sentry/fsimpl/kernfs/synthetic_directory_refs.go b/pkg/sentry/fsimpl/kernfs/synthetic_directory_refs.go index 69b41668a..4409edd35 100644 --- a/pkg/sentry/fsimpl/kernfs/synthetic_directory_refs.go +++ b/pkg/sentry/fsimpl/kernfs/synthetic_directory_refs.go @@ -54,11 +54,6 @@ func (r *syntheticDirectoryRefs) LogRefs() bool { return syntheticDirectoryenableLogging } -// EnableLeakCheck enables reference leak checking on r. -func (r *syntheticDirectoryRefs) EnableLeakCheck() { - refsvfs2.Register(r) -} - // ReadRefs returns the current number of references. The returned count is // inherently racy and is unsafe to use without external synchronization. func (r *syntheticDirectoryRefs) ReadRefs() int64 { @@ -115,7 +110,7 @@ func (r *syntheticDirectoryRefs) TryIncRef() bool { func (r *syntheticDirectoryRefs) DecRef(destroy func()) { v := atomic.AddInt64(&r.refCount, -1) if syntheticDirectoryenableLogging { - refsvfs2.LogDecRef(r, v+1) + refsvfs2.LogDecRef(r, v) } switch { case v < 0: @@ -132,6 +127,6 @@ func (r *syntheticDirectoryRefs) DecRef(destroy func()) { func (r *syntheticDirectoryRefs) afterLoad() { if r.ReadRefs() > 0 { - r.EnableLeakCheck() + refsvfs2.Register(r) } } |