From: Bruce Momjian on
Stephen Frost wrote:
> * Bruce Momjian (bruce(a)momjian.us) wrote:
> > I am not replying to many of these emails so I don't appear to be
> > brow-beating (forcing) the community into accepting this features. I
> > might be brow-beating the community, but I don't want to _appear_ to be
> > brow-beating. ;-)
>
> My apologies if I come across this way- I don't intend to... But I'm

You are fine. I was just saying that at a time I was one of the few
loud voices on this, and if this is going to happen, it will be because
we have a team that wants to do this, not because I am being loud. I
see the team forming nicely.

> also very enthusiastic about this. Also, it's become a much more
> personal issue for me due to this:
>
> http://csrc.nist.gov/news_events/documents/omb/draft-omb-fy2010-security-metrics.pdf
>
> OMB is now looking to include label-based security in their metrics.
> This directly impacts some of the PG-based systems I run.

Ah, very interesting, and good.

--
Bruce Momjian <bruce(a)momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

--
Sent via pgsql-hackers mailing list (pgsql-hackers(a)postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

From: Stephen Frost on
Bruce,

* Bruce Momjian (bruce(a)momjian.us) wrote:
> You are fine. I was just saying that at a time I was one of the few
> loud voices on this, and if this is going to happen, it will be because
> we have a team that wants to do this, not because I am being loud. I
> see the team forming nicely.

Not to rain down on the parade too much here, but I have to disagree
about a team forming nicely. That's, unfortunately, what it looks like
from the 10k-foot level. Indeed, it looks like we're making good
headway to get some kind of support into core from that level.

The reality is that we've barely started and really have still got
quite a ways to go and it would really be useful to bring in additional
resources on this. I wouldn't consider myself to be that "additional
resource" unless and until I can get funding for dedicated time (either
my own or someone else's). I've got a few action items that I'm
planning to resolve in the next few weeks, but I've been involved in
this for over a year now and it hasn't made much progress, overall, in
that time.

So, for anyone else who's interested in label-based security happening
for PostgreSQL (for whatever reason, masochisim perfectly acceptable),
please speak up and offer to help. We could use it.

Thanks,

Stephen
From: Robert Haas on
On Mon, Dec 14, 2009 at 10:21 PM, Stephen Frost <sfrost(a)snowman.net> wrote:
> Bruce,
>
> * Bruce Momjian (bruce(a)momjian.us) wrote:
>> You are fine.  I was just saying that at a time I was one of the few
>> loud voices on this, and if this is going to happen, it will be because
>> we have a team that wants to do this, not because I am being loud.  I
>> see the team forming nicely.
>
> Not to rain down on the parade too much here, but I have to disagree
> about a team forming nicely.  That's, unfortunately, what it looks like
> from the 10k-foot level.  Indeed, it looks like we're making good
> headway to get some kind of support into core from that level.
>
> The reality is that we've barely started and really have still got
> quite a ways to go and it would really be useful to bring in additional
> resources on this.  I wouldn't consider myself to be that "additional
> resource" unless and until I can get funding for dedicated time (either
> my own or someone else's).  I've got a few action items that I'm
> planning to resolve in the next few weeks, but I've been involved in
> this for over a year now and it hasn't made much progress, overall, in
> that time.

I completely agree. Many people have spent substantial time trying to
help KaiGai extract a committable patch from his work, and that effort
has not been successful. What I am concerned about is that by
continuing to spend time on KaiGai's work, we are wasting a lot of
community resources to no good end. It may be the case that even if
we had a patch that was technically excellent, the community would
decide that the amount of future maintenance that this feature would
require is not warranted by the number of users it would attract. Tom
is the only really vocal advocate that I'm aware of for that position,
but there may well be other people who feel similarly.

But these patches are, unfortunately, not technically excellent.
There have been multiple reviews of these patches that have produced
extensive laundry lists of items to be fixed. In the ordinary course
of events, that leads to one of two things happening: either the patch
author fixes most or all the problems and comes back with a patch that
shows marked improvement, or he or she gives up. This patch is unique
in my experience in that it has gone through - I believe - six
CommitFests now without either of those things happening. Not that
there hasn't been any improvement, but the ratio of reviewing-work to
improvement seems to be much higher than what is typical for us. Like
Stephen, I believe we need some additional resources who can improve
that ratio before we can really make a push to get this done.

....Robert

--
Sent via pgsql-hackers mailing list (pgsql-hackers(a)postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

From: KaiGai Kohei on
(2009/12/16 0:03), Robert Haas wrote:
> But these patches are, unfortunately, not technically excellent.
> There have been multiple reviews of these patches that have produced
> extensive laundry lists of items to be fixed. In the ordinary course
> of events, that leads to one of two things happening: either the patch
> author fixes most or all the problems and comes back with a patch that
> shows marked improvement, or he or she gives up. This patch is unique
> in my experience in that it has gone through - I believe - six
> CommitFests now without either of those things happening. Not that
> there hasn't been any improvement, but the ratio of reviewing-work to
> improvement seems to be much higher than what is typical for us. Like
> Stephen, I believe we need some additional resources who can improve
> that ratio before we can really make a push to get this done.

I had a talk with Stephen off list to make clear what I wondered.
It became apparent that I misunderstood the meaning of "cleanup first".
IIUC, he suggested to consolidate permission checks in several places
(such as createdb()) into same place to make more suitable for upcoming
framework, but the default PG checks are still inlined, not consolidated to
backend/security/*.

He also concerned our earlier approach has required higher hurdle to
join development, because it tried to do something useful feature although
a lot of features are separated, so past patch had to touch both of core
routines and selinux specific code.

So, I agreed with his opinion that we should restart from the pure cleanup
of the existing PG checks to make them more suitable for the upcoming security
framework. The scope of this effort stay in the pgsql world 100%. I don't
think it is an incorrect approach now.

In actually, I was suggested similar things at the begining of CF#3 from
Itagaki-san, but it was unclear whether we should go through the smaller
SE-PgSQL patch first or security framework first at that time.

I'll submit a small conceptual patch soon, as a draft.

Thanks,
--
OSS Platform Development Division, NEC
KaiGai Kohei <kaigai(a)ak.jp.nec.com>

--
Sent via pgsql-hackers mailing list (pgsql-hackers(a)postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers