diff --git a/VERSION-vendor b/VERSION-vendor index 64cf6b5..e2b32f2 100644 --- a/VERSION-vendor +++ b/VERSION-vendor @@ -1 +1 @@ -18.09.0.325 +18.09.0.326 diff --git a/docker.spec b/docker.spec index 8bcf885..9e5932e 100644 --- a/docker.spec +++ b/docker.spec @@ -1,6 +1,6 @@ Name: docker-engine Version: 18.09.0 -Release: 325 +Release: 326 Epoch: 2 Summary: The open-source application container engine Group: Tools/Docker @@ -229,6 +229,12 @@ fi %endif %changelog +* Thu Jun 29 2023 chenjiankun - 18.09.0-326 +- Type:bugfix +- CVE:NA +- SUG:NA +- DESC:repalce unix.Rmdir with os.RemoveAll when remove mount point dir + * Tue Jun 27 2023 chenjiankun - 18.09.0-325 - Type:bugfix - CVE:NA diff --git a/git-commit b/git-commit index 9f8649e..df0c3f6 100644 --- a/git-commit +++ b/git-commit @@ -1 +1 @@ -67cbcd3bf8881fc8434a3e35e778cbe6791e7727 +e1a8917f6034ae01a9bc2172799dda8a2f34a882 diff --git a/patch/0260-docker-repalce-unix.Rmdir-with-os.RemoveAll-when-rem.patch b/patch/0260-docker-repalce-unix.Rmdir-with-os.RemoveAll-when-rem.patch new file mode 100644 index 0000000..f46b328 --- /dev/null +++ b/patch/0260-docker-repalce-unix.Rmdir-with-os.RemoveAll-when-rem.patch @@ -0,0 +1,38 @@ +From 9f1aa90a8c1b3856c4bb6ec422b8f67ac5afc31e Mon Sep 17 00:00:00 2001 +From: chenjiankun +Date: Mon, 12 Jun 2023 16:09:06 +0800 +Subject: [PATCH] docker: repalce unix.Rmdir with os.RemoveAll when remove + mount point dir + +In some cases, there are some files in the mount point dir, so we can't use +unix.Rmdir to remove mount point dir. +--- + components/engine/daemon/graphdriver/devmapper/driver.go | 5 +++-- + 1 file changed, 3 insertions(+), 2 deletions(-) + +diff --git a/components/engine/daemon/graphdriver/devmapper/driver.go b/components/engine/daemon/graphdriver/devmapper/driver.go +index 3005b84eb..a1a6e17af 100644 +--- a/components/engine/daemon/graphdriver/devmapper/driver.go ++++ b/components/engine/daemon/graphdriver/devmapper/driver.go +@@ -19,7 +19,6 @@ import ( + "github.com/docker/go-units" + "github.com/pkg/errors" + "github.com/sirupsen/logrus" +- "golang.org/x/sys/unix" + ) + + func init() { +@@ -196,7 +195,9 @@ func (d *Driver) Remove(id string) error { + // mount point is not important and should not be treated + // as a failure to remove the container. + mp := path.Join(d.home, "mnt", id) +- err := unix.Rmdir(mp) ++ // In some cases, there are some files in the mount point dir, so we can't use ++ // unix.Rmdir to remove mount point dir. os.RemoveAll is more appropriate ++ err := os.RemoveAll(mp) + if err != nil && !os.IsNotExist(err) { + logrus.WithField("storage-driver", "devicemapper").Warnf("unable to remove mount point %q: %s", mp, err) + } +-- +2.23.0 + diff --git a/series.conf b/series.conf index 577c0bb..99b63ec 100644 --- a/series.conf +++ b/series.conf @@ -257,4 +257,5 @@ patch/0256-docker-libnet-d-overlay-document-some-encryption-code.patch patch/0257-docker-libnet-d-overlay-add-BPF-powered-VNI-matcher.patch patch/0258-docker-thinpool-full-because-kill-docker-daemon-when.patch patch/0259-backport-fix-blockThreshold-full-bug.patch +patch/0260-docker-repalce-unix.Rmdir-with-os.RemoveAll-when-rem.patch #end