Re: SASL 2.1.27 pre-release 1

Ken Murchison via Cyrus-sasl <[email protected]>
Newsgroups gmane.comp.security.cyrus.sasl
Organization Carnegie Mellon University
Message-ID <[email protected]>
Julien,

Can you try this patch for OSX:

diff --git a/plugins/plugin_common.c b/plugins/plugin_common.c
index f2b26bd..1211fea 100644
--- a/plugins/plugin_common.c
+++ b/plugins/plugin_common.c
@@ -88,6 +88,10 @@ static void sockaddr_unmapped(
      uint32_t addr;
      int port;

+#if defined(__FreeBSD__) && !defined(s6_addr32)
+#define s6_addr32 __u6_addr.__u6_addr32
+#endif
+
      if (sa->sa_family != AF_INET6)
      return;
      sin6 = (struct sockaddr_in6 *)sa;


On 11/23/2016 04:53 PM, Julien ÉLIE via Cyrus-sasl wrote:
> Hi Ken,
>
>> I have just built a first pre-release of SASL 2.1.27 which can be
>> downloaded from here for the time being:
>> http://www.contrib.andrew.cmu.edu/~murch/
>>
>> At this point I'm mostly concerned with build issues and any binary
>> incompatibilities (I don't believe there are any).
>
> When building on macOS 10.12.1 (Sierra), I see:
>
> ../plugins/plugin_common.c:97:42: error: no member named 's6_addr32' in
>       'struct in6_addr'
>     addr = *(uint32_t *)&sin6->sin6_addr.s6_addr32[3];
>                          ~~~~~~~~~~~~~~~ ^
> 1 error generated.
>
>
> It seems that s6_addr is not supported on macOS (and also OSX).
>
> struct in6_addr {
>         union {
>                 __uint8_t   __u6_addr8[16];
>                 __uint16_t  __u6_addr16[8];
>                 __uint32_t  __u6_addr32[4];
>         } __u6_addr;                    /* 128-bit IP6 address */
> };
>
>
>
>
> Building on a few other Linux and BSD systems is OK.
> Linking Cyrus SASL library against the NNTP news server INN also works 
> fine.
>
>
> In the release process, could you add the update of config.guess and 
> config.sub?  They date back to January 2015 in the pre-release tar ball.
>
>
> Thanks again for your work!
>

-- 
Kenneth Murchison
Principal Systems Software Engineer
Carnegie Mellon University
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.