diff mbox series

libtirpc: add missing extern

Message ID 20220919235954.14011-1-rosenp@gmail.com (mailing list archive)
State New, archived
Headers show
Series libtirpc: add missing extern | expand

Commit Message

Rosen Penev Sept. 19, 2022, 11:59 p.m. UTC
Fixes compilation warning.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
---
 src/svc_auth.c | 3 +++
 1 file changed, 3 insertions(+)

Comments

Steve Dickson Sept. 20, 2022, 7:28 p.m. UTC | #1
On 9/19/22 7:59 PM, Rosen Penev wrote:
> Fixes compilation warning.
What was the warning? Plus AUTH_DES is no longer supported.

steved.
> 
> Signed-off-by: Rosen Penev <rosenp@gmail.com>
> ---
>   src/svc_auth.c | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/src/svc_auth.c b/src/svc_auth.c
> index ce8bbd8..789d6af 100644
> --- a/src/svc_auth.c
> +++ b/src/svc_auth.c
> @@ -66,6 +66,9 @@ static struct authsvc *Auths = NULL;
>   
>   extern SVCAUTH svc_auth_none;
>   
> +#ifdef AUTHDES_SUPPORT
> +extern enum auth_stat _svcauth_des(struct svc_req *rqst, struct rpc_msg *msg);
> +#endif
>   /*
>    * The call rpc message, msg has been obtained from the wire.  The msg contains
>    * the raw form of credentials and verifiers.  authenticate returns AUTH_OK
Rosen Penev Sept. 20, 2022, 8:31 p.m. UTC | #2
On Tue, Sep 20, 2022 at 12:28 PM Steve Dickson <steved@redhat.com> wrote:
>
>
>
> On 9/19/22 7:59 PM, Rosen Penev wrote:
> > Fixes compilation warning.
> What was the warning? Plus AUTH_DES is no longer supported.
Implicit function declaration as it's not in a header file or anything.

I found the issue from doing a meson conversion for libtirpc:
https://github.com/mesonbuild/wrapdb/pull/644

I have no idea about AUTH_DES or what it's used for.
>
> steved.
> >
> > Signed-off-by: Rosen Penev <rosenp@gmail.com>
> > ---
> >   src/svc_auth.c | 3 +++
> >   1 file changed, 3 insertions(+)
> >
> > diff --git a/src/svc_auth.c b/src/svc_auth.c
> > index ce8bbd8..789d6af 100644
> > --- a/src/svc_auth.c
> > +++ b/src/svc_auth.c
> > @@ -66,6 +66,9 @@ static struct authsvc *Auths = NULL;
> >
> >   extern SVCAUTH svc_auth_none;
> >
> > +#ifdef AUTHDES_SUPPORT
> > +extern enum auth_stat _svcauth_des(struct svc_req *rqst, struct rpc_msg *msg);
> > +#endif
> >   /*
> >    * The call rpc message, msg has been obtained from the wire.  The msg contains
> >    * the raw form of credentials and verifiers.  authenticate returns AUTH_OK
>
Rosen Penev Oct. 24, 2022, 7:39 p.m. UTC | #3
On Tue, Sep 20, 2022 at 1:31 PM Rosen Penev <rosenp@gmail.com> wrote:
>
> On Tue, Sep 20, 2022 at 12:28 PM Steve Dickson <steved@redhat.com> wrote:
> >
> >
> >
> > On 9/19/22 7:59 PM, Rosen Penev wrote:
> > > Fixes compilation warning.
> > What was the warning? Plus AUTH_DES is no longer supported.
> Implicit function declaration as it's not in a header file or anything.
>
> I found the issue from doing a meson conversion for libtirpc:
> https://github.com/mesonbuild/wrapdb/pull/644
>
> I have no idea about AUTH_DES or what it's used for.
ping.
> >
> > steved.
> > >
> > > Signed-off-by: Rosen Penev <rosenp@gmail.com>
> > > ---
> > >   src/svc_auth.c | 3 +++
> > >   1 file changed, 3 insertions(+)
> > >
> > > diff --git a/src/svc_auth.c b/src/svc_auth.c
> > > index ce8bbd8..789d6af 100644
> > > --- a/src/svc_auth.c
> > > +++ b/src/svc_auth.c
> > > @@ -66,6 +66,9 @@ static struct authsvc *Auths = NULL;
> > >
> > >   extern SVCAUTH svc_auth_none;
> > >
> > > +#ifdef AUTHDES_SUPPORT
> > > +extern enum auth_stat _svcauth_des(struct svc_req *rqst, struct rpc_msg *msg);
> > > +#endif
> > >   /*
> > >    * The call rpc message, msg has been obtained from the wire.  The msg contains
> > >    * the raw form of credentials and verifiers.  authenticate returns AUTH_OK
> >
Steve Dickson Oct. 25, 2022, 4:36 p.m. UTC | #4
On 9/19/22 7:59 PM, Rosen Penev wrote:
> Fixes compilation warning.
> 
> Signed-off-by: Rosen Penev <rosenp@gmail.com>
Committed...

steved.
> ---
>   src/svc_auth.c | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/src/svc_auth.c b/src/svc_auth.c
> index ce8bbd8..789d6af 100644
> --- a/src/svc_auth.c
> +++ b/src/svc_auth.c
> @@ -66,6 +66,9 @@ static struct authsvc *Auths = NULL;
>   
>   extern SVCAUTH svc_auth_none;
>   
> +#ifdef AUTHDES_SUPPORT
> +extern enum auth_stat _svcauth_des(struct svc_req *rqst, struct rpc_msg *msg);
> +#endif
>   /*
>    * The call rpc message, msg has been obtained from the wire.  The msg contains
>    * the raw form of credentials and verifiers.  authenticate returns AUTH_OK
diff mbox series

Patch

diff --git a/src/svc_auth.c b/src/svc_auth.c
index ce8bbd8..789d6af 100644
--- a/src/svc_auth.c
+++ b/src/svc_auth.c
@@ -66,6 +66,9 @@  static struct authsvc *Auths = NULL;
 
 extern SVCAUTH svc_auth_none;
 
+#ifdef AUTHDES_SUPPORT
+extern enum auth_stat _svcauth_des(struct svc_req *rqst, struct rpc_msg *msg);
+#endif
 /*
  * The call rpc message, msg has been obtained from the wire.  The msg contains
  * the raw form of credentials and verifiers.  authenticate returns AUTH_OK