* [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 an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.