Script 'mail_helper' called by obssrc
Hello community,
here is the log from the commit of package nfs-utils for openSUSE:Factory
checked in at 2023-10-16 22:32:44
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/nfs-utils (Old)
and /work/SRC/openSUSE:Factory/.nfs-utils.new.20540 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "nfs-utils"
Mon Oct 16 22:32:44 2023 rev:179 rq:1117914 version:unknown
Changes:
--------
--- /work/SRC/openSUSE:Factory/nfs-utils/nfs-utils.changes 2023-04-24
22:30:56.575414227 +0200
+++ /work/SRC/openSUSE:Factory/.nfs-utils.new.20540/nfs-utils.changes
2023-10-16 22:32:45.683339031 +0200
@@ -1,0 +2,7 @@
+Mon Oct 16 00:43:42 UTC 2023 - Neil Brown <nfbr...@suse.com>
+
+- 0001-export-fix-handling-of-error-from-match_fsid.patch
+ Errors from match_fsid() shouldn't be fatal.
+ (bsc#1214540)
+
+-------------------------------------------------------------------
New:
----
0001-export-fix-handling-of-error-from-match_fsid.patch
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ nfs-utils.spec ++++++
--- /var/tmp/diff_new_pack.CtbObf/_old 2023-10-16 22:32:46.471367455 +0200
+++ /var/tmp/diff_new_pack.CtbObf/_new 2023-10-16 22:32:46.471367455 +0200
@@ -42,6 +42,7 @@
Source26: nfs.conf
Source27: nfs-kernel-server.tmpfiles.conf
Patch0: nfs-utils-1.0.7-bind-syntax.patch
+Patch1: 0001-export-fix-handling-of-error-from-match_fsid.patch
BuildRequires: e2fsprogs-devel
BuildRequires: gcc-c++
BuildRequires: libtool
++++++ 0001-export-fix-handling-of-error-from-match_fsid.patch ++++++
>From 0f46b6728d0f45393eb0e33a90fcfa2147271275 Mon Sep 17 00:00:00 2001
From: NeilBrown <ne...@suse.de>
Date: Wed, 11 Oct 2023 12:32:25 +1100
Subject: [PATCH] export: fix handling of error from match_fsid()
If match_fsid() returns -1 we shouldn't assume that the path definitely
doesn't match the fsid, though it might not.
This is a similar situation to where an export is expected to be a mount
point, but is found not to be one. So it can be handled the same way,
by setting 'dev_missing'.
This will only have an effect is no other path matched the fsid, which
is what we want.
The current code results in nothing being exported and any export point,
or any mount point beneath a crossmnt export point fail a 'stat'
request, which is too harsh.
Signed-off-by: NeilBrown <ne...@suse.de>
---
support/export/cache.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
--- a/support/export/cache.c
+++ b/support/export/cache.c
@@ -858,7 +858,8 @@ static void nfsd_fh(int f)
case 0:
continue;
case -1:
- goto out;
+ dev_missing ++;
+ continue;
}
if (is_ipaddr_client(dom)
&& !ipaddr_client_matches(exp, ai))
