util: Fix race when checking whether a path is on a shared FS

[PATCH 0/4] util: Fix race when checking whether a path is on a shared FS

Jiri Denemark via Devel posted 4 patches 2 days, 20 hours ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1765292471.git.jdenemar@redhat.com

src/util/virfile.c  | 133 +++++++++++++++++++++++++++++---------------
tests/virfiletest.c |  69 +++++++++++++++++++++++
2 files changed, 156 insertions(+), 46 deletions(-)

Expand all Fold all

[PATCH 0/4] util: Fix race when checking whether a path is on a shared FS

Posted by Jiri Denemark via Devel 2 days, 20 hours ago

See 2/4 for details.

Jiri Denemark (4):
  tests: Test virFileIsSharedFSOverride
  util: Fix race condition in virFileIsSharedFSType
  util: Fix race condition in virFileIsSharedFSOverride
  util: Rework virFileIsSharedFSOverride using virFileCheckParents

 src/util/virfile.c  | 133 +++++++++++++++++++++++++++++---------------
 tests/virfiletest.c |  69 +++++++++++++++++++++++
 2 files changed, 156 insertions(+), 46 deletions(-)

-- 
2.52.0

Re: [PATCH 0/4] util: Fix race when checking whether a path is on a shared FS

Posted by Michal Prívozník via Devel 2 days, 20 hours ago

On 12/9/25 16:02, Jiri Denemark via Devel wrote:
> See 2/4 for details.
> 
> Jiri Denemark (4):
>   tests: Test virFileIsSharedFSOverride
>   util: Fix race condition in virFileIsSharedFSType
>   util: Fix race condition in virFileIsSharedFSOverride
>   util: Rework virFileIsSharedFSOverride using virFileCheckParents
> 
>  src/util/virfile.c  | 133 +++++++++++++++++++++++++++++---------------
>  tests/virfiletest.c |  69 +++++++++++++++++++++++
>  2 files changed, 156 insertions(+), 46 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal

Re: [PATCH 0/4] util: Fix race when checking whether a path is on a shared FS

Posted by Jiří Denemark via Devel 2 days ago

On Tue, Dec 09, 2025 at 16:41:37 +0100, Michal Prívozník wrote:
> On 12/9/25 16:02, Jiri Denemark via Devel wrote:
> > See 2/4 for details.
> > 
> > Jiri Denemark (4):
> >   tests: Test virFileIsSharedFSOverride
> >   util: Fix race condition in virFileIsSharedFSType
> >   util: Fix race condition in virFileIsSharedFSOverride
> >   util: Rework virFileIsSharedFSOverride using virFileCheckParents
> > 
> >  src/util/virfile.c  | 133 +++++++++++++++++++++++++++++---------------
> >  tests/virfiletest.c |  69 +++++++++++++++++++++++
> >  2 files changed, 156 insertions(+), 46 deletions(-)
> > 
> 
> Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Oops, I forgot to add this to the commits before pushing :-(

Jirka