* missing inet_makeaddr
@ 2012-11-20 1:18 Roy
2012-11-20 5:51 ` [PATCH] Re: [musl] " Isaac Dunham
0 siblings, 1 reply; 3+ messages in thread
From: Roy @ 2012-11-20 1:18 UTC (permalink / raw)
To: musl
samba > 3.3 needs inet_makeaddr which is missing in musl.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] Re: [musl] missing inet_makeaddr
2012-11-20 1:18 missing inet_makeaddr Roy
@ 2012-11-20 5:51 ` Isaac Dunham
2012-11-21 0:37 ` Rich Felker
0 siblings, 1 reply; 3+ messages in thread
From: Isaac Dunham @ 2012-11-20 5:51 UTC (permalink / raw)
To: musl
[-- Attachment #1: Type: text/plain, Size: 298 bytes --]
On Tue, 20 Nov 2012 01:18:05 +0000 (UTC)
Roy <roytam@gmail.com> wrote:
>
> samba > 3.3 needs inet_makeaddr which is missing in musl.
1: This is a portability issue that should be tested in configure.
2. Here's a patch for musl, based on the NetBSD code.
--
Isaac Dunham <idunham@lavabit.com>
[-- Attachment #2: makeaddr.diff --]
[-- Type: text/x-diff, Size: 3230 bytes --]
commit d95ac554fb5cd7ed48a4cb1768cc102e0b002c12
Author: Isaac Dunham <idunham@lavabit.com>
Date: Mon Nov 19 21:45:54 2012 -0800
Add inet_makeaddr.
This is a BSD network function which Samba wants.
diff --git a/include/arpa/inet.h b/include/arpa/inet.h
index 9293840..95f046e 100644
--- a/include/arpa/inet.h
+++ b/include/arpa/inet.h
@@ -29,6 +29,7 @@ int inet_pton (int, const char *__restrict, void *__restrict);
const char *inet_ntop (int, const void *__restrict, char *__restrict, socklen_t);
int inet_aton (const char *, struct in_addr *); /* nonstandard but widely used */
+struct in_addr inet_makeaddr (in_addr_t net, in_addr_t host);
#undef INET_ADDRSTRLEN
#undef INET6_ADDRSTRLEN
diff --git a/src/network/inet_makeaddr.c b/src/network/inet_makeaddr.c
new file mode 100644
index 0000000..e205168
--- /dev/null
+++ b/src/network/inet_makeaddr.c
@@ -0,0 +1,55 @@
+/* $NetBSD: inet_makeaddr.c,v 1.1 2004/05/20 23:13:02 christos Exp $ */
+
+/*
+ * Copyright (c) 1983, 1993
+ * The Regents of the University of California. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ * 3. Neither the name of the University nor the names of its contributors
+ * may be used to endorse or promote products derived from this software
+ * without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+ * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
+ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ */
+
+#define _BSD_SOURCE
+#include <netinet/in.h>
+#include <arpa/inet.h>
+
+/*
+ * Formulate an Internet address from network + host. Used in
+ * building addresses stored in the ifnet structure.
+ */
+struct in_addr inet_makeaddr(in_addr_t net, in_addr_t host)
+{
+ in_addr_t addr;
+ struct in_addr ret;
+
+ if (net < 128)
+ addr = (net << IN_CLASSA_NSHIFT) | (host & IN_CLASSA_HOST);
+ else if (net < 65536)
+ addr = (net << IN_CLASSB_NSHIFT) | (host & IN_CLASSB_HOST);
+ else if (net < 16777216L)
+ addr = (net << IN_CLASSC_NSHIFT) | (host & IN_CLASSC_HOST);
+ else
+ addr = net | host;
+ ret.s_addr = htonl(addr);
+ return ret;
+}
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Re: [musl] missing inet_makeaddr
2012-11-20 5:51 ` [PATCH] Re: [musl] " Isaac Dunham
@ 2012-11-21 0:37 ` Rich Felker
0 siblings, 0 replies; 3+ messages in thread
From: Rich Felker @ 2012-11-21 0:37 UTC (permalink / raw)
To: musl
On Mon, Nov 19, 2012 at 09:51:16PM -0800, Isaac Dunham wrote:
> On Tue, 20 Nov 2012 01:18:05 +0000 (UTC)
> Roy <roytam@gmail.com> wrote:
>
> >
> > samba > 3.3 needs inet_makeaddr which is missing in musl.
>
> 1: This is a portability issue that should be tested in configure.
>
> 2. Here's a patch for musl, based on the NetBSD code.
I think it would be nice to avoid pulling in additional copyrights for
such trivial functions.. It's not that they're incompatible (license
seems okay), just that it makes for a lot more copyright status to
document and track. In my opinion, effort would be better spent just
reading the man pages and writing one- to five-line functions that do
the same thing.
Rich
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-11-21 0:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-11-20 1:18 missing inet_makeaddr Roy
2012-11-20 5:51 ` [PATCH] Re: [musl] " Isaac Dunham
2012-11-21 0:37 ` Rich Felker
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/musl/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).