netdev - Re: [GIT PULL] [IPV6] COMPAT: Fix SSM applications on 64bit kernels.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sat, 26 Apr 2008 13:53:37 +0900 (JST)
From:	YOSHIFUJI Hideaki / 吉藤英明 
	<yoshfuji@...ux-ipv6.org>
To:	dlstevens@...ibm.com
Cc:	davem@...emloft.net, netdev@...r.kernel.org
Subject: Re: [GIT PULL] [IPV6] COMPAT: Fix SSM applications on 64bit
 kernels.

In article <OF326F3AF9.7322F5FB-ON88257437.00174874-88257437.00194F7B@...ibm.com> (at Fri, 25 Apr 2008 21:36:29 -0700), David Stevens <dlstevens@...ibm.com> says:

> > net/compat.c is not the right place.
> > Please put this in net/ipv4/compat.c or so.
> > Or at least, these must be guarded by CONFIG_INET.
> 
>         This code is for both v4 and v6, so I'd rather it not be in
> net/ipv4/compat.c. I didn't try without CONFIG_INET, -- will look
> at that.

We've been putting ipv4/ipv6 common things in net/ipv4/*.c,
so please put it in net/ipv4/compat.c.

> > > +struct compat_group_filter {
> > > +       __u32                            gf_interface;
> > > +       struct __kernel_sockaddr_storage gf_group
> > > +               __attribute__ ((aligned(4)));
> > > +       __u32                            gf_fmode;
> > > +       __u32                            gf_numsrc;
> > > +       struct __kernel_sockaddr_storage gf_slist[1]
> > > +               __attribute__ ((aligned(4)));
> > > +} __attribute__ ((packed));
> > > +
> > 
> > include/linux/in.n.
> 
>         I don't understand this comment. If you're suggesting I
> put these in an include file, I didn't because this is the only
> file that uses them and I don't want them to be visible to any
> other code. If you mean something else, please explain.

We put most (if not all) compat structures in headers.
Sparse definitions are not good.

--yoshfuji

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html