From: "Gary V. Vaughan" <git@mlists.thewrittenword.com>
To: Martin Storsj? <martin@martin.st>
Cc: git@vger.kernel.org
Subject: Re: [PATCH 5/5] struct sockaddr_storage->ss_family is not portable
Date: Mon, 26 Apr 2010 16:55:22 +0000 [thread overview]
Message-ID: <20100426165522.GC28004@thor.il.thewrittenword.com> (raw)
In-Reply-To: <alpine.DEB.2.00.1004252204020.24585@cone.home.martin.st>
On Sun, Apr 25, 2010 at 10:05:16PM +0300, Martin Storsj? wrote:
> Hi Gary,
Hi Martin,
There's unfortunately a 1 month time slip between my having written
the messages and having figured out why they were being rejected
(actually 3 things that needed fixing before resends started arriving,
which is why it took so long)...
> On Sun, 25 Apr 2010, Gary V. Vaughan wrote:
>
> > On Fri, Mar 12, 2010 at 09:24:01AM +0200, Martin Storsj? wrote:
> > >
> > > Yes, but if the sockaddr struct can be arranged in different ways, the
> > > other ones (sockaddr_in, sockaddr_storage, sockaddr_in6) must also be
> > > defined coherently - you're always supposed to be able to cast an
> > > sockaddr_in (or any other of them) to a sockaddr and read the sa_family
> > > field. As far as I know, at least.
> >
> > Ah, good point. And now, having tested that on all our machines it
> > works perfectly, and is much more elegant!
> >
> > I'll resubmit presently.
>
> Actually, Brandon Casey already submitted a patch doing this, which is
> available in master by now, so this issue is all taken care of. :-)
...and I didn't notice that until I started rebasing the changesets
against a newer release.
Excellent that one issue is now resolved though, I'll repost my
outstanding patches presently.
Cheers,
--
Gary V. Vaughan (gary@thewrittenword.com)
next prev parent reply other threads:[~2010-04-26 16:55 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-11 16:37 [PATCH 5/5] struct sockaddr_storage->ss_family is not portable Gary V. Vaughan
2010-03-11 16:40 ` Martin Storsjö
2010-03-12 4:56 ` Gary V. Vaughan
2010-03-12 7:24 ` Martin Storsjö
2010-03-15 21:03 ` [PATCH] daemon.c: avoid accessing ss_family member of struct sockaddr_storage Brandon Casey
2010-03-15 21:29 ` Jeff King
2010-03-15 21:41 ` Martin Storsjö
2010-03-15 21:42 ` Brandon Casey
2010-04-25 8:37 ` Gary V. Vaughan
2010-03-15 21:37 ` Martin Storsjö
2010-03-15 22:10 ` [PATCH v2] " Brandon Casey
2010-03-16 7:52 ` Martin Storsjö
2010-04-25 8:36 ` [PATCH 5/5] struct sockaddr_storage->ss_family is not portable Gary V. Vaughan
2010-04-25 19:05 ` Martin Storsjö
2010-04-26 16:55 ` Gary V. Vaughan [this message]
2010-03-11 22:27 ` Jeff King
2010-03-11 23:57 ` Brandon Casey
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20100426165522.GC28004@thor.il.thewrittenword.com \
--to=git@mlists.thewrittenword.com \
--cc=git@vger.kernel.org \
--cc=martin@martin.st \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox