* [nfs-utils PATCH] Set right error after getaddrinfo fails bz 1287468 [not found] <1449635027-10764-1-git-send-email-rkothiya@redhat.com> @ 2015-12-09 4:29 ` Rinku Kothiya 2015-12-21 4:41 ` Rinku Kothiya 2016-01-16 21:59 ` [nfs-utils PATCH] Set right error after getaddrinfo fails bz 1287468 Steve Dickson 0 siblings, 2 replies; 4+ messages in thread From: Rinku Kothiya @ 2015-12-09 4:29 UTC (permalink / raw) To: steved; +Cc: Mohit Agrawal, linux-nfs From: Rinku Kothiya <rkothiya@redhat.com> Signed-off-by: Rinku Kothiya <rkothiya@redhat.com> Signed-off-by: Mohit Agrawal <moagrawa@redhat.com> --- nfs-utils-1.3.0-set-errno.patch | 29 +++++++++++++++++++++++++++++ nfs-utils.spec | 23 ++++++++++++----------- 2 files changed, 41 insertions(+), 11 deletions(-) create mode 100644 nfs-utils-1.3.0-set-errno.patch diff --git a/nfs-utils-1.3.0-set-errno.patch b/nfs-utils-1.3.0-set-errno.patch new file mode 100644 index 0000000..7c9d381 --- /dev/null +++ b/nfs-utils-1.3.0-set-errno.patch @@ -0,0 +1,29 @@ +diff -up nfs-utils-1.3.0/support/export/hostname.c.orig nfs-utils-1.3.0/support/export/hostname.c +--- nfs-utils-1.3.0/support/export/hostname.c.orig 2015-12-08 15:24:36.705047453 +0530 ++++ nfs-utils-1.3.0/support/export/hostname.c 2015-12-08 15:25:46.614047453 +0530 +@@ -179,11 +179,11 @@ + case 0: + return ai; + case EAI_SYSTEM: +- xlog(D_GENERAL, "%s: failed to resolve %s: (%d) %m", ++ xlog(L_WARNING, "%s: failed to resolve %s: (%d) %m", + __func__, hostname, errno); + break; + default: +- xlog(D_GENERAL, "%s: failed to resolve %s: %s", ++ xlog(L_WARNING, "%s: failed to resolve %s: %s", + __func__, hostname, gai_strerror(error)); + break; + } +diff -up nfs-utils-1.3.0/support/nfs/client.c.orig nfs-utils-1.3.0/support/nfs/client.c +--- nfs-utils-1.3.0/support/export/client.c.orig 2015-12-08 17:08:45.469047453 +0530 ++++ nfs-utils-1.3.0/support/export/client.c 2015-12-08 17:09:01.159047453 +0530 +@@ -277,7 +277,7 @@ + if (htype == MCL_FQDN && !canonical) { + ai = host_addrinfo(hname); + if (!ai) { +- xlog(L_ERROR, "Failed to resolve %s", hname); ++ xlog(L_WARNING, "Failed to resolve %s", hname); + goto out; + } + hname = ai->ai_canonname; diff --git a/nfs-utils.spec b/nfs-utils.spec index 2fe1cc5..e881914 100644 --- a/nfs-utils.spec +++ b/nfs-utils.spec @@ -2,7 +2,7 @@ Summary: NFS utilities and supporting clients and daemons for the kernel NFS ser Name: nfs-utils URL: http://sourceforge.net/projects/nfs Version: 1.3.0 -Release: 0.22%{?dist} +Release: 0.22%{?dist}_sfdc_01536441 Epoch: 1 # group all 32bit related archs @@ -61,6 +61,7 @@ Patch100: nfs-utils-1.2.1-statdpath-man.patch Patch101: nfs-utils-1.2.1-exp-subtree-warn-off.patch Patch102: nfs-utils-1.2.3-sm-notify-res_init.patch Patch103: nfs-utils-1.2.5-idmap-errmsg.patch +Patch104: nfs-utils-1.3.0-set-errno.patch Group: System Environment/Daemons Provides: exportfs = %{epoch}:%{version}-%{release} @@ -101,16 +102,12 @@ Requires: gssproxy >= 0.3.0-0 Conflicts: gssproxy < 0.3.0-10 %description -The nfs-utils package provides a daemon for the kernel NFS server and -related tools, which provides a much higher level of performance than the -traditional Linux NFS server used by most users. - -This package also contains the showmount program. Showmount queries the -mount daemon on a remote host for information about the NFS (Network File -System) server on the remote host. For example, showmount can display the -clients which are mounted on that host. - -This package also contains the mount.nfs and umount.nfs program. +This RPM has been provided by Red Hat for testing purposes only and is +NOT supported for any other use. This RPM may contain changes that are +necessary for debugging but that are not appropriate for other uses, +or that are not compatible with third-party hardware or software. This +RPM should NOT be deployed for purposes other than testing and +debugging. %prep %setup -q @@ -185,6 +182,7 @@ This package also contains the mount.nfs and umount.nfs program. %patch101 -p1 %patch102 -p1 %patch103 -p1 +%patch104 -p1 # Remove .orig files find . -name "*.orig" | xargs rm -f @@ -418,6 +416,9 @@ fi /sbin/umount.nfs4 %changelog +* Tue Dec 8 2015 Rinku Kothiya <rkothiya@redhat.com> 1.3.0-0.22_sfdc_01536441 +- exportfs: Set the correct error type after getaddrinfo fails in host_addrinfo (bz 1287468) + * Thu Dec 3 2015 Steve Dickson <steved@redhat.com> 1.3.0-0.22 - nfsidmap: Correct a failure to set key timeout values (bz 1161222) - exportfs: Restore the EAI_NONAME check in host_pton() (bz 1276099) -- 1.9.3 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [nfs-utils PATCH] Set right error after getaddrinfo fails bz 1287468 2015-12-09 4:29 ` [nfs-utils PATCH] Set right error after getaddrinfo fails bz 1287468 Rinku Kothiya @ 2015-12-21 4:41 ` Rinku Kothiya 2015-12-22 9:09 ` [nfs-utils PATCH] Set right error after getaddrinfo fails bz 1287468, Please find the same patch attached to this email as well Rinku Kothiya 2016-01-16 21:59 ` [nfs-utils PATCH] Set right error after getaddrinfo fails bz 1287468 Steve Dickson 1 sibling, 1 reply; 4+ messages in thread From: Rinku Kothiya @ 2015-12-21 4:41 UTC (permalink / raw) To: Steve Dickson, Mohit Agrawal, linux-nfs From: Rinku Kothiya <rkothiya@redhat.com> Signed-off-by: Rinku Kothiya <rkothiya@redhat.com> Signed-off-by: Mohit Agrawal <moagrawa@redhat.com> --- nfs-utils-1.3.0-set-errno.patch | 29 +++++++++++++++++++++++++++++ nfs-utils.spec | 23 ++++++++++++----------- 2 files changed, 41 insertions(+), 11 deletions(-) create mode 100644 nfs-utils-1.3.0-set-errno.patch diff --git a/nfs-utils-1.3.0-set-errno.patch b/nfs-utils-1.3.0-set-errno.patch new file mode 100644 index 0000000..7c9d381 --- /dev/null +++ b/nfs-utils-1.3.0-set-errno.patch @@ -0,0 +1,29 @@ +diff -up nfs-utils-1.3.0/support/export/hostname.c.orig nfs-utils-1.3.0/support/export/hostname.c +--- nfs-utils-1.3.0/support/export/hostname.c.orig 2015-12-08 15:24:36.705047453 +0530 ++++ nfs-utils-1.3.0/support/export/hostname.c 2015-12-08 15:25:46.614047453 +0530 +@@ -179,11 +179,11 @@ + case 0: + return ai; + case EAI_SYSTEM: +- xlog(D_GENERAL, "%s: failed to resolve %s: (%d) %m", ++ xlog(L_WARNING, "%s: failed to resolve %s: (%d) %m", + __func__, hostname, errno); + break; + default: +- xlog(D_GENERAL, "%s: failed to resolve %s: %s", ++ xlog(L_WARNING, "%s: failed to resolve %s: %s", + __func__, hostname, gai_strerror(error)); + break; + } +diff -up nfs-utils-1.3.0/support/nfs/client.c.orig nfs-utils-1.3.0/support/nfs/client.c +--- nfs-utils-1.3.0/support/export/client.c.orig 2015-12-08 17:08:45.469047453 +0530 ++++ nfs-utils-1.3.0/support/export/client.c 2015-12-08 17:09:01.159047453 +0530 +@@ -277,7 +277,7 @@ + if (htype == MCL_FQDN && !canonical) { + ai = host_addrinfo(hname); + if (!ai) { +- xlog(L_ERROR, "Failed to resolve %s", hname); ++ xlog(L_WARNING, "Failed to resolve %s", hname); + goto out; + } + hname = ai->ai_canonname; diff --git a/nfs-utils.spec b/nfs-utils.spec index 2fe1cc5..e881914 100644 --- a/nfs-utils.spec +++ b/nfs-utils.spec @@ -2,7 +2,7 @@ Summary: NFS utilities and supporting clients and daemons for the kernel NFS ser Name: nfs-utils URL: http://sourceforge.net/projects/nfs Version: 1.3.0 -Release: 0.22%{?dist} +Release: 0.22%{?dist}_sfdc_01536441 Epoch: 1 # group all 32bit related archs @@ -61,6 +61,7 @@ Patch100: nfs-utils-1.2.1-statdpath-man.patch Patch101: nfs-utils-1.2.1-exp-subtree-warn-off.patch Patch102: nfs-utils-1.2.3-sm-notify-res_init.patch Patch103: nfs-utils-1.2.5-idmap-errmsg.patch +Patch104: nfs-utils-1.3.0-set-errno.patch Group: System Environment/Daemons Provides: exportfs = %{epoch}:%{version}-%{release} @@ -101,16 +102,12 @@ Requires: gssproxy >= 0.3.0-0 Conflicts: gssproxy < 0.3.0-10 %description -The nfs-utils package provides a daemon for the kernel NFS server and -related tools, which provides a much higher level of performance than the -traditional Linux NFS server used by most users. - -This package also contains the showmount program. Showmount queries the -mount daemon on a remote host for information about the NFS (Network File -System) server on the remote host. For example, showmount can display the -clients which are mounted on that host. - -This package also contains the mount.nfs and umount.nfs program. +This RPM has been provided by Red Hat for testing purposes only and is +NOT supported for any other use. This RPM may contain changes that are +necessary for debugging but that are not appropriate for other uses, +or that are not compatible with third-party hardware or software. This +RPM should NOT be deployed for purposes other than testing and +debugging. %prep %setup -q @@ -185,6 +182,7 @@ This package also contains the mount.nfs and umount.nfs program. %patch101 -p1 %patch102 -p1 %patch103 -p1 +%patch104 -p1 # Remove .orig files find . -name "*.orig" | xargs rm -f @@ -418,6 +416,9 @@ fi /sbin/umount.nfs4 %changelog +* Tue Dec 8 2015 Rinku Kothiya <rkothiya@redhat.com> 1.3.0-0.22_sfdc_01536441 +- exportfs: Set the correct error type after getaddrinfo fails in host_addrinfo (bz 1287468) + * Thu Dec 3 2015 Steve Dickson <steved@redhat.com> 1.3.0-0.22 - nfsidmap: Correct a failure to set key timeout values (bz 1161222) - exportfs: Restore the EAI_NONAME check in host_pton() (bz 1276099) -- 1.9.3 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [nfs-utils PATCH] Set right error after getaddrinfo fails bz 1287468, Please find the same patch attached to this email as well 2015-12-21 4:41 ` Rinku Kothiya @ 2015-12-22 9:09 ` Rinku Kothiya 0 siblings, 0 replies; 4+ messages in thread From: Rinku Kothiya @ 2015-12-22 9:09 UTC (permalink / raw) To: Steve Dickson, linux-nfs; +Cc: Mohit Agrawal [-- Attachment #1: Type: text/plain, Size: 1780 bytes --] From: Rinku Kothiya <rkothiya@redhat.com> Signed-off-by: Rinku Kothiya <rkothiya@redhat.com> Signed-off-by: Mohit Agrawal <moagrawa@redhat.com> --- nfs-utils-1.3.0-set-errno.patch | 29 +++++++++++++++++++++++++++++ nfs-utils.spec | 23 ++++++++++++----------- 2 files changed, 41 insertions(+), 11 deletions(-) create mode 100644 nfs-utils-1.3.0-set-errno.patch diff --git a/nfs-utils-1.3.0-set-errno.patch b/nfs-utils-1.3.0-set-errno.patch new file mode 100644 index 0000000..7c9d381 --- /dev/null +++ b/nfs-utils-1.3.0-set-errno.patch @@ -0,0 +1,29 @@ +diff -up nfs-utils-1.3.0/support/export/hostname.c.orig nfs-utils-1.3.0/support/export/hostname.c +--- nfs-utils-1.3.0/support/export/hostname.c.orig 2015-12-08 15:24:36.705047453 +0530 ++++ nfs-utils-1.3.0/support/export/hostname.c 2015-12-08 15:25:46.614047453 +0530 +@@ -179,11 +179,11 @@ + case 0: + return ai; + case EAI_SYSTEM: +- xlog(D_GENERAL, "%s: failed to resolve %s: (%d) %m", ++ xlog(L_WARNING, "%s: failed to resolve %s: (%d) %m", + __func__, hostname, errno); + break; + default: +- xlog(D_GENERAL, "%s: failed to resolve %s: %s", ++ xlog(L_WARNING, "%s: failed to resolve %s: %s", + __func__, hostname, gai_strerror(error)); + break; + } +diff -up nfs-utils-1.3.0/support/nfs/client.c.orig nfs-utils-1.3.0/support/nfs/client.c +--- nfs-utils-1.3.0/support/export/client.c.orig 2015-12-08 17:08:45.469047453 +0530 ++++ nfs-utils-1.3.0/support/export/client.c 2015-12-08 17:09:01.159047453 +0530 +@@ -277,7 +277,7 @@ + if (htype == MCL_FQDN && !canonical) { + ai = host_addrinfo(hname); + if (!ai) { +- xlog(L_ERROR, "Failed to resolve %s", hname); ++ xlog(L_WARNING, "Failed to resolve %s", hname); + goto out; + } + hname = ai->ai_canonname; [-- Attachment #2: nfs-utils-1.3.0-set-errno.patch --] [-- Type: text/x-patch, Size: 1165 bytes --] diff -up nfs-utils-1.3.0/support/nfs/hostname.c.orig nfs-utils-1.3.0/support/nfs/hostname.c --- nfs-utils-1.3.0/support/export/hostname.c.orig 2015-12-08 15:24:36.705047453 +0530 +++ nfs-utils-1.3.0/support/export/hostname.c 2015-12-08 15:25:46.614047453 +0530 @@ -179,11 +179,11 @@ case 0: return ai; case EAI_SYSTEM: - xlog(D_GENERAL, "%s: failed to resolve %s: (%d) %m", + xlog(L_WARNING, "%s: failed to resolve %s: (%d) %m", __func__, hostname, errno); break; default: - xlog(D_GENERAL, "%s: failed to resolve %s: %s", + xlog(L_WARNING, "%s: failed to resolve %s: %s", __func__, hostname, gai_strerror(error)); break; } diff -up nfs-utils-1.3.0/support/nfs/client.c.orig nfs-utils-1.3.0/support/nfs/client.c --- nfs-utils-1.3.0/support/export/client.c.orig 2015-12-08 17:08:45.469047453 +0530 +++ nfs-utils-1.3.0/support/export/client.c 2015-12-08 17:09:01.159047453 +0530 @@ -277,7 +277,7 @@ if (htype == MCL_FQDN && !canonical) { ai = host_addrinfo(hname); if (!ai) { - xlog(L_ERROR, "Failed to resolve %s", hname); + xlog(L_WARNING, "Failed to resolve %s", hname); goto out; } hname = ai->ai_canonname; ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [nfs-utils PATCH] Set right error after getaddrinfo fails bz 1287468 2015-12-09 4:29 ` [nfs-utils PATCH] Set right error after getaddrinfo fails bz 1287468 Rinku Kothiya 2015-12-21 4:41 ` Rinku Kothiya @ 2016-01-16 21:59 ` Steve Dickson 1 sibling, 0 replies; 4+ messages in thread From: Steve Dickson @ 2016-01-16 21:59 UTC (permalink / raw) To: Rinku Kothiya; +Cc: Mohit Agrawal, linux-nfs On 12/08/2015 11:29 PM, Rinku Kothiya wrote: > > > From: Rinku Kothiya <rkothiya@redhat.com> > > Signed-off-by: Rinku Kothiya <rkothiya@redhat.com> > Signed-off-by: Mohit Agrawal <moagrawa@redhat.com> > --- > nfs-utils-1.3.0-set-errno.patch | 29 +++++++++++++++++++++++++++++ > nfs-utils.spec | 23 ++++++++++++----------- > 2 files changed, 41 insertions(+), 11 deletions(-) > create mode 100644 nfs-utils-1.3.0-set-errno.patch We finally got this committed! ;-) steved > > diff --git a/nfs-utils-1.3.0-set-errno.patch b/nfs-utils-1.3.0-set-errno.patch > new file mode 100644 > index 0000000..7c9d381 > --- /dev/null > +++ b/nfs-utils-1.3.0-set-errno.patch > @@ -0,0 +1,29 @@ > +diff -up nfs-utils-1.3.0/support/export/hostname.c.orig nfs-utils-1.3.0/support/export/hostname.c > +--- nfs-utils-1.3.0/support/export/hostname.c.orig 2015-12-08 15:24:36.705047453 +0530 > ++++ nfs-utils-1.3.0/support/export/hostname.c 2015-12-08 15:25:46.614047453 +0530 > +@@ -179,11 +179,11 @@ > + case 0: > + return ai; > + case EAI_SYSTEM: > +- xlog(D_GENERAL, "%s: failed to resolve %s: (%d) %m", > ++ xlog(L_WARNING, "%s: failed to resolve %s: (%d) %m", > + __func__, hostname, errno); > + break; > + default: > +- xlog(D_GENERAL, "%s: failed to resolve %s: %s", > ++ xlog(L_WARNING, "%s: failed to resolve %s: %s", > + __func__, hostname, gai_strerror(error)); > + break; > + } > +diff -up nfs-utils-1.3.0/support/nfs/client.c.orig nfs-utils-1.3.0/support/nfs/client.c > +--- nfs-utils-1.3.0/support/export/client.c.orig 2015-12-08 17:08:45.469047453 +0530 > ++++ nfs-utils-1.3.0/support/export/client.c 2015-12-08 17:09:01.159047453 +0530 > +@@ -277,7 +277,7 @@ > + if (htype == MCL_FQDN && !canonical) { > + ai = host_addrinfo(hname); > + if (!ai) { > +- xlog(L_ERROR, "Failed to resolve %s", hname); > ++ xlog(L_WARNING, "Failed to resolve %s", hname); > + goto out; > + } > + hname = ai->ai_canonname; > diff --git a/nfs-utils.spec b/nfs-utils.spec > index 2fe1cc5..e881914 100644 > --- a/nfs-utils.spec > +++ b/nfs-utils.spec > @@ -2,7 +2,7 @@ Summary: NFS utilities and supporting clients and daemons for the kernel NFS ser > Name: nfs-utils > URL: http://sourceforge.net/projects/nfs > Version: 1.3.0 > -Release: 0.22%{?dist} > +Release: 0.22%{?dist}_sfdc_01536441 > Epoch: 1 > > # group all 32bit related archs > @@ -61,6 +61,7 @@ Patch100: nfs-utils-1.2.1-statdpath-man.patch > Patch101: nfs-utils-1.2.1-exp-subtree-warn-off.patch > Patch102: nfs-utils-1.2.3-sm-notify-res_init.patch > Patch103: nfs-utils-1.2.5-idmap-errmsg.patch > +Patch104: nfs-utils-1.3.0-set-errno.patch > > Group: System Environment/Daemons > Provides: exportfs = %{epoch}:%{version}-%{release} > @@ -101,16 +102,12 @@ Requires: gssproxy >= 0.3.0-0 > Conflicts: gssproxy < 0.3.0-10 > > %description > -The nfs-utils package provides a daemon for the kernel NFS server and > -related tools, which provides a much higher level of performance than the > -traditional Linux NFS server used by most users. > - > -This package also contains the showmount program. Showmount queries the > -mount daemon on a remote host for information about the NFS (Network File > -System) server on the remote host. For example, showmount can display the > -clients which are mounted on that host. > - > -This package also contains the mount.nfs and umount.nfs program. > +This RPM has been provided by Red Hat for testing purposes only and is > +NOT supported for any other use. This RPM may contain changes that are > +necessary for debugging but that are not appropriate for other uses, > +or that are not compatible with third-party hardware or software. This > +RPM should NOT be deployed for purposes other than testing and > +debugging. > > %prep > %setup -q > @@ -185,6 +182,7 @@ This package also contains the mount.nfs and umount.nfs program. > %patch101 -p1 > %patch102 -p1 > %patch103 -p1 > +%patch104 -p1 > > # Remove .orig files > find . -name "*.orig" | xargs rm -f > @@ -418,6 +416,9 @@ fi > /sbin/umount.nfs4 > > %changelog > +* Tue Dec 8 2015 Rinku Kothiya <rkothiya@redhat.com> 1.3.0-0.22_sfdc_01536441 > +- exportfs: Set the correct error type after getaddrinfo fails in host_addrinfo (bz 1287468) > + > * Thu Dec 3 2015 Steve Dickson <steved@redhat.com> 1.3.0-0.22 > - nfsidmap: Correct a failure to set key timeout values (bz 1161222) > - exportfs: Restore the EAI_NONAME check in host_pton() (bz 1276099) ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-01-16 21:59 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1449635027-10764-1-git-send-email-rkothiya@redhat.com>
2015-12-09 4:29 ` [nfs-utils PATCH] Set right error after getaddrinfo fails bz 1287468 Rinku Kothiya
2015-12-21 4:41 ` Rinku Kothiya
2015-12-22 9:09 ` [nfs-utils PATCH] Set right error after getaddrinfo fails bz 1287468, Please find the same patch attached to this email as well Rinku Kothiya
2016-01-16 21:59 ` [nfs-utils PATCH] Set right error after getaddrinfo fails bz 1287468 Steve Dickson
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox