diff mbox

Add requirement for newer libgssglue for svcgssd -n option

Message ID 20110629183721.23198.57774.stgit@jazz.citi.umich.edu (mailing list archive)
State New, archived
Headers show

Commit Message

Kevin Coffman June 29, 2011, 6:37 p.m. UTC
SNAFU.  This came to my attention minutes after 1.2.4 was
released...

Changes in commit d6c1b35c require that gss_acquire_cred()
is now called when the "-n" option is used.  This requires an
updated libgssglue which properly handles name GSS_C_NO_NAME
as input to gss_import_name()/gss_acquire_cred().

Add a requirement for the newer version.

Without the newer libgssglue, when svcgssd is started with "-n"
you will see the error message, "ERROR: GSS-API: error in
gss_acquire_cred(): GSS_S_BAD_NAME (An invalid name was supplied)
- Unknown error"
---

 aclocal/rpcsec_vers.m4 |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Kevin Coffman June 30, 2011, 12:38 a.m. UTC | #1
On Wed, Jun 29, 2011 at 2:37 PM, Kevin Coffman <kwc@citi.umich.edu> wrote:
> SNAFU.  This came to my attention minutes after 1.2.4 was
> released...
>
> Changes in commit d6c1b35c require that gss_acquire_cred()
> is now called when the "-n" option is used.  This requires an
> updated libgssglue which properly handles name GSS_C_NO_NAME
> as input to gss_import_name()/gss_acquire_cred().
>
> Add a requirement for the newer version.
>
> Without the newer libgssglue, when svcgssd is started with "-n"
> you will see the error message, "ERROR: GSS-API: error in
> gss_acquire_cred(): GSS_S_BAD_NAME (An invalid name was supplied)
> - Unknown error"
> ---
>
>  aclocal/rpcsec_vers.m4 |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/aclocal/rpcsec_vers.m4 b/aclocal/rpcsec_vers.m4
> index 25902ca..8218372 100644
> --- a/aclocal/rpcsec_vers.m4
> +++ b/aclocal/rpcsec_vers.m4
> @@ -1,7 +1,7 @@
>  dnl Checks librpcsec version
>  AC_DEFUN([AC_RPCSEC_VERSION], [
>
> -  PKG_CHECK_MODULES([GSSGLUE], [libgssglue >= 0.1])
> +  PKG_CHECK_MODULES([GSSGLUE], [libgssglue >= 0.3])
>
>   dnl TI-RPC replaces librpcsecgss
>   if test "$enable_tirpc" = no; then
>

I just realized that I failed to add a Signed-off-by: in both patches
I sent today.  Let me know if you need me to resend them...

K.C.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Steve Dickson June 30, 2011, 3:24 p.m. UTC | #2
On 06/29/2011 02:37 PM, Kevin Coffman wrote:
> SNAFU.  This came to my attention minutes after 1.2.4 was
> released...
> 
> Changes in commit d6c1b35c require that gss_acquire_cred()
> is now called when the "-n" option is used.  This requires an
> updated libgssglue which properly handles name GSS_C_NO_NAME
> as input to gss_import_name()/gss_acquire_cred().
> 
> Add a requirement for the newer version.
> 
> Without the newer libgssglue, when svcgssd is started with "-n"
> you will see the error message, "ERROR: GSS-API: error in
> gss_acquire_cred(): GSS_S_BAD_NAME (An invalid name was supplied)
> - Unknown error"
> ---
Committed.. Unfortunately post 1.2.4 release... 

steved.

> 
>  aclocal/rpcsec_vers.m4 |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/aclocal/rpcsec_vers.m4 b/aclocal/rpcsec_vers.m4
> index 25902ca..8218372 100644
> --- a/aclocal/rpcsec_vers.m4
> +++ b/aclocal/rpcsec_vers.m4
> @@ -1,7 +1,7 @@
>  dnl Checks librpcsec version
>  AC_DEFUN([AC_RPCSEC_VERSION], [
>  
> -  PKG_CHECK_MODULES([GSSGLUE], [libgssglue >= 0.1])
> +  PKG_CHECK_MODULES([GSSGLUE], [libgssglue >= 0.3])
>  
>    dnl TI-RPC replaces librpcsecgss
>    if test "$enable_tirpc" = no; then
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/aclocal/rpcsec_vers.m4 b/aclocal/rpcsec_vers.m4
index 25902ca..8218372 100644
--- a/aclocal/rpcsec_vers.m4
+++ b/aclocal/rpcsec_vers.m4
@@ -1,7 +1,7 @@ 
 dnl Checks librpcsec version
 AC_DEFUN([AC_RPCSEC_VERSION], [
 
-  PKG_CHECK_MODULES([GSSGLUE], [libgssglue >= 0.1])
+  PKG_CHECK_MODULES([GSSGLUE], [libgssglue >= 0.3])
 
   dnl TI-RPC replaces librpcsecgss
   if test "$enable_tirpc" = no; then