All of lore.kernel.org
 help / color / mirror / Atom feed
From: nicky726@gmail.com (Nicky726)
To: refpolicy@oss.tresys.com
Subject: [refpolicy]  Policy for Konqueror and KDE v7
Date: Sun, 21 Feb 2010 13:14:12 +0100	[thread overview]
Message-ID: <201002211314.13958.Nicky726@gmail.com> (raw)
In-Reply-To: <1265984479.4386.17.camel@gorn.columbia.tresys.com>

Hello,

Dne P? 12. ?nora 2010 15:21:19 jste napsal(a): 
> On Mon, 2010-02-01 at 14:53 +0100, Nicky726 wrote:
> > Hello,
> > 
> > implemented konqueror_stream_connect and interfaces for access to
> > koneuror temp files etc. to my policy, as Dominick Grift suggested.
> 
> Does not compile for me:
> 
> policy/modules/roles/staff.te":78:ERROR 'duplicate declaration of
> type/attribute' at token ';' on line 2431678:
> 	type staff_xproperty_t, xproperty_type;
> #line 78

Could it be this line in konqueror_role_template?
	xserver_user_x_domain_template($1, konqueror_t, konqueror_tmp_t)

If so, am I supposed to create konqueror_role withouth this call just for 
staff?
I don't get this error when compiling modules under Fedora, so I don't exactly 
know, what is wrong.

> Other things to fix:
> 
> * fix the whitespace (eg trailing whitespace)
> * move kde_manage_home_dirs() to be after kde_search_home() in kde.if
> * make kde_home_filetrans() have the object class as a 3rd parameter
> * remove "shared" from the kde type names (kde_shared_home_t ->
> kde_home_t)
> * it seems that konqueror_role_template() can be trimmed down by moving
> common rules into knoqueror.te.
> * in konqueror_role_template() the konqueror_dbus_chat() should go in
> the optional with the explicit dbus usage.
> * files_search_rw() does not exist.
> * "interfaces from LAYER directory" comments are unnecessary.

Sould be fixed in attached patch.

With regards,
Ondrej Vadinsky

-- 
Don't it always seem to go
That you don't know what you've got
Till it's gone

(Joni Mitchell)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 2010-02-21-Policy-for-Konqueror-and-KDE-v7.patch
Type: text/x-patch
Size: 15337 bytes
Desc: not available
Url : http://oss.tresys.com/pipermail/refpolicy/attachments/20100221/ec11a022/attachment.bin 

  reply	other threads:[~2010-02-21 12:14 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-01 13:53 [refpolicy] Policy for Konqueror and KDE v6 Nicky726
2010-02-12 14:21 ` Christopher J. PeBenito
2010-02-21 12:14   ` Nicky726 [this message]
2010-02-22 13:56     ` [refpolicy] Policy for Konqueror and KDE v7 Christopher J. PeBenito

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=201002211314.13958.Nicky726@gmail.com \
    --to=nicky726@gmail.com \
    --cc=refpolicy@oss.tresys.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.