Skip Menu |
 

Date: Fri, 27 Feb 2004 14:38:54 -0600
From: "Douglas E. Engert" <deengert@anl.gov>
To: krb5-bugs@mit.edu
Subject: krb5-1.3.2 and HP UX - foreachaddr.c


HP UX 11i and IA64 HP UX 11.23 define SIOCGLIFCONF
and SIOCGLIFNUM, but define the structures used by these
features differently. Thus foreachaddr.c does not compile.

The comments on the #ifdef imply for Solaris 8 and later only.
This patch makes sure the HP versions will not try to use this feature.



--- ,foreachaddr.c Mon Sep 29 14:07:29 2003
+++ foreachaddr.c Fri Feb 27 13:21:59 2004
@@ -259,7 +259,7 @@
return ret;
}

-#ifdef SIOCGLIFCONF /* Solaris */
+#if defined(SIOCGLIFCONF) && !defined(__hpux) /* Solaris */
static int
get_lifconf (int af, int s, size_t *lenp, /*@out@*/ char *buf)
/*@modifies *buf,*lenp@*/
@@ -431,7 +431,7 @@
return 0;
}

-#elif defined (SIOCGLIFNUM) /* Solaris 8 and later; Sol 7? */
+#elif defined (SIOCGLIFNUM) && !defined(__hpux) /* Solaris 8 and later; Sol 7? */

static int
foreach_localaddr (/*@null@*/ void *data,

--

Douglas E. Engert <DEEngert@anl.gov>
Argonne National Laboratory
9700 South Cass Avenue
Argonne, Illinois 60439
(630) 252-5444
Date: Sat, 28 Feb 2004 14:12:37 -0500
Subject: Re: [krbdev.mit.edu #2300] krb5-1.3.2 and HP UX - foreachaddr.c
From: Ken Raeburn <raeburn@MIT.EDU>
To: rt@krbdev.mit.edu
RT-Send-Cc:
On Friday, Feb 27, 2004, at 15:38 US/Eastern, DEEngert@anl.gov via RT
wrote:

Show quoted text
> The comments on the #ifdef imply for Solaris 8 and later only.

That's because it was the only system I had encountered that had
SIOCGLIFCONF, not because I thought there were systems where we should
avoid using it.

As Tom and I were joking yesterday, our HP-UX build system hasn't
reported any errors.
But that's because it's turned off. It's a slow machine running an old
version of the OS, not very interesting...

Show quoted text
> This patch makes sure the HP versions will not try to use this feature.

I needed it on Solaris because the SIOCGIFCONF interface didn't seem to
get me the IPv6 info. Does IPv6 support work with your change, or do
we need to figure out how to use SIOCGLIFCONF on HP-UX?

Ken
To: rt@krbdev.mit.edu
Subject: Re: [krbdev.mit.edu #2300] krb5-1.3.2 and HP UX - foreachaddr.c
From: Sam Hartman <hartmans@mit.edu>
Date: Sun, 29 Feb 2004 01:10:29 -0500
RT-Send-Cc:


Even if Doug's patch does work, I'd really prefer a feature test for a
broken interface than something that is os-specific.
Date: Sun, 29 Feb 2004 19:28:57 -0600
From: "Douglas E. Engert" <deengert@anl.gov>
To: rt-comment@krbdev.mit.edu
Cc: krb5-prs@mit.edu
Subject: Re: [krbdev.mit.edu #2300] krb5-1.3.2 and HP UX - foreachaddr.c
RT-Send-Cc:


Ken Raeburn via RT wrote:
Show quoted text
>
> On Friday, Feb 27, 2004, at 15:38 US/Eastern, DEEngert@anl.gov via RT
> wrote:
>
> > The comments on the #ifdef imply for Solaris 8 and later only.
>
> That's because it was the only system I had encountered that had
> SIOCGLIFCONF, not because I thought there were systems where we should
> avoid using it.
>
> As Tom and I were joking yesterday, our HP-UX build system hasn't
> reported any errors.
> But that's because it's turned off. It's a slow machine running an old
> version of the OS, not very interesting...
>
> > This patch makes sure the HP versions will not try to use this feature.
>
> I needed it on Solaris because the SIOCGIFCONF interface didn't seem to
> get me the IPv6 info. Does IPv6 support work with your change, or do
> we need to figure out how to use SIOCGLIFCONF on HP-UX?

Don't know if it will work or not. We don't have and IPv6 on any HPs
that I know of. I was just trying to get it to comple.



Show quoted text
>
> Ken

--

Douglas E. Engert <DEEngert@anl.gov>
Argonne National Laboratory
9700 South Cass Avenue
Argonne, Illinois 60439
(630) 252-5444