linux-kernel - Re: [PATCH 3/3] generic sys

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:	Wed, 6 Jan 2010 19:59:05 +0000
From:	Russell King - ARM Linux <linux@....linux.org.uk>
To:	Arnd Bergmann <arnd@...db.de>
Cc:	Christoph Hellwig <hch@....de>, akpm@...ux-foundation.org,
	linux-kernel@...r.kernel.org, linux-arch@...r.kernel.org,
	starvik@...s.com, jesper.nilsson@...s.com, dhowells@...hat.com,
	ysato@...rs.sourceforge.jp, takata@...ux-m32r.org,
	geert@...ux-m68k.org, zippel@...ux-m68k.org, gerg@...inux.org,
	ralf@...ux-mips.org, benh@...nel.crashing.org,
	schwidefsky@...ibm.com, heiko.carstens@...ibm.com,
	lethal@...ux-sh.org, davem@...emloft.net, jdike@...toit.com,
	tglx@...utronix.de, mingo@...hat.com, hpa@...or.com,
	viro@...iv.linux.org.uk
Subject: Re: [PATCH 3/3] generic sys_ipc wrapper

On Wed, Jan 06, 2010 at 08:52:11PM +0100, Arnd Bergmann wrote:
> > +		switch (version) {

Open of first

> > +		default: {

Open of second

> > +			ulong raddr;
> > +			ret = do_shmat(first, (char __user *)ptr,
> > +				       second, &raddr);
> > +			if (ret)
> > +				return ret;
> > +			return put_user(raddr, (ulong __user *) third);
> > +		}

close of second

> > +
> > +#if defined(__i386__) || defined(__frv__) || defined(__mips__) ||\
> > +    defined(__mn10300__)
> > +		case 1:
> > +			/* iBCS2 emulator entry point */
> > +			if (!segment_eq(get_fs(), get_ds()))
> > +				return -EINVAL;
> > +			/*
> > +			 * The "(ulong *) third" is valid _only_ because of
> > +			 * the kernel segment thing.
> > +			 */
> > +			return do_shmat(first, (char __user *) ptr, second,
> > +					(unsigned long *)third);
> > +#endif
> > +#if defined(__arm__) || defined(__sparc__)
> > +		case 1:
> > +			/* Of course, we don't support iBCS2! */
> > +			return -EINVAL;
> > +#endif
> > +		}

close of first.

> This would not build on any of the architecture mentioned, because of
> incorrect placement of curly braces, but as Al mentioned it should just
> go away entirely.

They look fine to me.

In any case, all three patches get my ack.

Acked-by: Russell King <rmk+kernel@....linux.org.uk>

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/