[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20071018153043.GB3906@sequoia.sous-sol.org>
Date: Thu, 18 Oct 2007 08:30:43 -0700
From: Chris Wright <chrisw@...s-sol.org>
To: "Serge E. Hallyn" <serge@...lyn.com>
Cc: Chris Wright <chrisw@...s-sol.org>,
Andrew Morton <akpm@...ux-foundation.org>,
"Serge E. Hallyn" <serue@...ibm.com>, sds@...ho.nsa.gov,
linux-security-module@...r.kernel.org, morgan@...nel.org,
linux-kernel@...r.kernel.org, kaigai@...gai.gr.jp,
casey@...aufler-ca.com
Subject: Re: [RFC] [PATCH 2/2] capabilities: implement 64-bit capabilities
* Serge E. Hallyn (serge@...lyn.com) wrote:
> Quoting Chris Wright (chrisw@...s-sol.org):
> > * Serge E. Hallyn (serge@...lyn.com) wrote:
> > > I guess now that I've written this out, it seems pretty clear
> > > that capget64() and capget64() are the way to go. Any objections?
> >
> > How is capget64() different from capget() that supports 2 different
> > header->versions (I thought that was the whole point of the versioned,
> > rather opaque interface)? I don't object to a new syscall, but I don't
> > see why it's required to avoid breaking libcap.
>
> Hmm, I guess it *works*, it's just harder to explain the "inconsistent"
> behavior. Now instead of saying "capget() will fail under certain
> conditions while capget64() will always succeed", capget() will actually
> fail under certain conditions only if you send in a certain header.
>
> Again, once I've written it out, I guess it isn't *so* bad.
It's not really any different than issuing capget(0x19980330) (assuming
capget64 is different), and getting -EINVAL when the actual in-use
caps are > 32 bits wide. In either case the rules are the same --
old interface works fine as long as you don't have new caps involved.
Only advantage I see to using the extant interface is that the cap[sg]et
interface is already designed to be future-proof (albeit in an unusual
way compared with most other kernel syscalls).
thanks,
-chris
-
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/
Powered by blists - more mailing lists