author | Christian Urban <christian dot urban at kcl dot ac dot uk> |
Fri, 06 Sep 2013 13:27:46 +0100 | |
changeset 14 | d43f46423298 |
child 15 | baa2970a9687 |
permissions | -rw-r--r-- |
14
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
1 |
From VM Fri Sep 6 13:27:30 2013 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
2 |
X-VM-v5-Data: ([nil nil nil nil t nil nil nil nil] |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
3 |
["8896" "Monday" "26" "August" "2013" "16:57:55" "+0200" "CPP 2013" "cpp2013@easychair.org" nil "98" "CPP 2013 notification for paper 42" "^From:" nil nil "8" nil nil nil nil nil nil nil nil nil nil] |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
4 |
nil) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
5 |
Return-Path: <cpp2013@easychair.org> |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
6 |
Delivered-To: unknown |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
7 |
Received: from mailbroy.in.tum.de (131.159.46.88:995) by |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
8 |
Christians-MacBook-Air-2.local with POP3-SSL; 26 Aug 2013 16:26:47 -0000 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
9 |
X-Original-To: urbanc@mailbroy.informatik.tu-muenchen.de |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
10 |
Delivered-To: urbanc@mailbroy.informatik.tu-muenchen.de |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
11 |
Received: from localhost (scan46 [10.46.146.16]) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
12 |
by mail46.informatik.tu-muenchen.de (Postfix) with ESMTP id 9A063885ECCEA |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
13 |
for <urbanc@mailbroy.informatik.tu-muenchen.de>; Mon, 26 Aug 2013 16:58:08 +0200 (CEST) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
14 |
X-Virus-Scanned: amavisd-new at informatik.tu-muenchen.de |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
15 |
X-Spam-MTA-Flag: NO |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
16 |
X-Spam-MTA-Status: No, score=0.0 required=5.0 tests=none autolearn=unavailable |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
17 |
version=3.3.2 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
18 |
X-Spam-MTA-Level: |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
19 |
X-Spam-MTA-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on scan46 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
20 |
Received: from mail46.informatik.tu-muenchen.de ([10.46.146.15]) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
21 |
by localhost (mail46.informatik.tu-muenchen.de [10.46.146.16]) (amavisd-new, port 10024) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
22 |
with ESMTP id i--Dvm4t7vwc |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
23 |
for <urbanc@mailbroy.informatik.tu-muenchen.de>; |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
24 |
Mon, 26 Aug 2013 16:58:03 +0200 (CEST) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
25 |
Received: from mailrelay1.informatik.tu-muenchen.de (mailrelay1.informatik.tu-muenchen.de [131.159.254.14]) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
26 |
by mail46.informatik.tu-muenchen.de (Postfix) with ESMTP id 119618133D93D |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
27 |
for <urbanc@mailbroy.informatik.tu-muenchen.de>; Mon, 26 Aug 2013 16:58:03 +0200 (CEST) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
28 |
Received: by mailrelay1.informatik.tu-muenchen.de (Postfix) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
29 |
id E3FAA27E4D; Mon, 26 Aug 2013 16:58:02 +0200 (CEST) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
30 |
Delivered-To: christian.urban@informatik.tu-muenchen.de |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
31 |
Received: from mailrelay1.informatik.tu-muenchen.de (localhost [127.0.0.1]) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
32 |
by mailrelay1.informatik.tu-muenchen.de (Postfix) with ESMTP id CD21927E49 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
33 |
for <urbanc@in.tum.de>; Mon, 26 Aug 2013 16:58:02 +0200 (CEST) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
34 |
Received: from maildmz2.informatik.tu-muenchen.de (maildmz2.informatik.tu-muenchen.de [131.159.0.15]) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
35 |
by mailrelay1.informatik.tu-muenchen.de (Postfix) with ESMTP id C41B127E48 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
36 |
for <urbanc@in.tum.de>; Mon, 26 Aug 2013 16:58:02 +0200 (CEST) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
37 |
Received: by maildmz2.informatik.tu-muenchen.de (Postfix, from userid 1506) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
38 |
id BF13F4C7BD; Mon, 26 Aug 2013 16:58:02 +0200 (CEST) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
39 |
X-Spam-Checker-Version: SpamAssassin 3.2.5-tuminfo_1 (2008-06-10) on maildmz2 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
40 |
X-Spam-Level: |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
41 |
X-Spam-Status: No, score=-2.6 required=7.0 tests=AWL,BAYES_00 autolearn=ham |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
42 |
version=3.2.5-tuminfo_1 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
43 |
Received: from maildmz2.informatik.tu-muenchen.de (localhost [127.0.0.1]) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
44 |
by maildmz2.informatik.tu-muenchen.de (Postfix) with ESMTP id 07430491BD |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
45 |
for <urbanc@in.tum.de>; Mon, 26 Aug 2013 16:57:54 +0200 (CEST) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
46 |
Received: from easychair.org (easychair.org [80.241.208.147]) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
47 |
by maildmz2.informatik.tu-muenchen.de (Postfix) with ESMTP id E7286491BC |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
48 |
for <urbanc@in.tum.de>; Mon, 26 Aug 2013 16:57:53 +0200 (CEST) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
49 |
Received: from localhost.localdomain (m1647.giga-dns.com [80.241.208.147]) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
50 |
by easychair.org (8.14.5/8.14.5) with ESMTP id r7QEvt6V020904 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
51 |
for <urbanc@in.tum.de>; Mon, 26 Aug 2013 16:57:55 +0200 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
52 |
Message-Id: <201308261457.r7QEvt6V020904@easychair.org> |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
53 |
Content-Disposition: inline |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
54 |
Content-Transfer-Encoding: 8bit |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
55 |
Content-Type: text/plain; charset="UTF-8" |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
56 |
MIME-Version: 1.0 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
57 |
X-Mailer: MIME::Lite 3.029 (F2.82; T1.38; A2.12; B3.13; Q3.13) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
58 |
From: CPP 2013 <cpp2013@easychair.org> |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
59 |
To: Christian Urban <urbanc@in.tum.de> |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
60 |
Subject: CPP 2013 notification for paper 42 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
61 |
Date: Mon, 26 Aug 2013 16:57:55 +0200 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
62 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
63 |
Dear Christian Urban, |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
64 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
65 |
We are happy to inform you that you paper 42, |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
66 |
A Formal Model and Correctness Proof for an Access Control Policy Framework |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
67 |
has been accepted for presentation at the third edition of the |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
68 |
International Conference on Certified Programs and Proofs. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
69 |
Enclosed are the reviews for your paper. Please pay close attention to |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
70 |
the comments of the reviewers in preparing the final version of your paper, |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
71 |
which will be due on Monday September 16, 2013. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
72 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
73 |
Kind regards, |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
74 |
Georges Gonthier |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
75 |
Michael Norrish |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
76 |
CPP 13 program chairs |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
77 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
78 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
79 |
----------------------- REVIEW 1 --------------------- |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
80 |
PAPER: 42 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
81 |
TITLE: A Formal Model and Correctness Proof for an Access Control Policy Framework |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
82 |
AUTHORS: Chunhan Wu, Xingyuan Zhang and Christian Urban |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
83 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
84 |
OVERALL EVALUATION: 0 (borderline paper) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
85 |
REVIEWER'S CONFIDENCE: 4 (high) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
86 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
87 |
----------- REVIEW ----------- |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
88 |
This paper presents a formalisation of the Role-Compatibility access control model (RC-Model), which is similar to the Domain-Type Enforcement (DTE) model of SELinux, in Isabelle/HOL. The model is formalised with respect to an abstract operating system model with operations such as CreateFile, CloneProcess, WriteFile etc. Execution is modelled as sequences / traces of such events. Meaning is given to RC-Model policies by restricting the allowable traces of events to those permitted by the policy. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
89 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
90 |
Policies are further given meaning in terms of a notion of "taintedness" introduced by the authors. An initial set of /seeds/ are initially tainted; taint spreads out through the system (for instance from a process p to a file f when p writes to f etc.) as execution proceeds. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
91 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
92 |
Since the possible state space (sequences of allowed events) is infinite, the authors then devise a static, finite check to determine whether a particular entity can become tainted. This check effectively builds an abstraction (tainted^s) of the system in which all newly-created objects are aggregated together (which makes it finite), but records precise information regarding the taintedness of initial objects (which makes it precise for those objects). |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
93 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
94 |
The authors prove that their static check is both sound and complete (for undeletable objects). |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
95 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
96 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
97 |
The development appears technically sound. However, the ideas are fairly well understood from e.g. Paulson's work on the inductive model of security protocols, and from work devising finite, safe abstractions of access control systems, where the idea of aggregation for newly-created objects to produce finite, safe abstractions of systems is well applied. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
98 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
99 |
I would also argue that some of the claims in the motivation could be made more precise or backed-up with better evidence. For instance, the claim in the opening sentence -- which OSes exactly use RBAC? SELinux uses DTE which seems simpler than the RC-Model, but the authors never cite the DTE papers when mentioning SELinux. I assume that the similarity between DTE and RC-Model is the basis for the claim that the RC-Model is used to secure "Apache servers". This should be made clear. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
100 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
101 |
In general, I would suggest citing the SELinux papers, especially on the Domain-Type Enforcement model -- you certainly have space. Broader treatment of related work (e.g. on finite abstractions of dynamic access control systems by aggregation -- see e.g. Fred Spiessens' work on SCOLLAR) could also be accommodated in page limit I think. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
102 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
103 |
One other point is that the "tainted" notion seems to be presented mostly in terms of tracking the effects of a brek-in (see last paragraph of Section 4). However, it seems like it could equally well be used to reason about confidentiality in the absence of covert channels. This might deserve some discussion. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
104 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
105 |
Overall the work is solid, but isn't huge on novelty. Hence, my score of Borderline. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
106 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
107 |
Minor comments: |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
108 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
109 |
- Drop the word "randomly" in quotes from page 5. new process IDs need only be unique, no? |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
110 |
- Explain more explicitly that states are sequences of events; that paths are stored in reverse-order. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
111 |
- consider putting the "tained" relation into its own subsection, as this is new material over and above the straight RC-Model, right? |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
112 |
- On second-to-last line of page 10, drop "anchor of" |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
113 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
114 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
115 |
----------------------- REVIEW 2 --------------------- |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
116 |
PAPER: 42 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
117 |
TITLE: A Formal Model and Correctness Proof for an Access Control Policy Framework |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
118 |
AUTHORS: Chunhan Wu, Xingyuan Zhang and Christian Urban |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
119 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
120 |
OVERALL EVALUATION: 2 (accept) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
121 |
REVIEWER'S CONFIDENCE: 4 (high) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
122 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
123 |
----------- REVIEW ----------- |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
124 |
This paper presents a formalization and correctness proof for the an access control policy framework called the Role-Compatibility Model. The paper introduces the dynamic semantics of a system controlled by an RC-model policy. It then develops a static check which is capable of determining all possible effects of a given RC-model policy on an initial system configuration. Finally, the paper proves the correctness of this check with respect to the dynamic semantics. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
125 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
126 |
In part, the paper is a case study of formalizing a recently introduced semi-formal system. However, the authors go beyond this by introducing a dynamic semantics which allows them to establish the correctness of the model they formalize. The contributions are interesting and relevant. I recommend the paper be accepted. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
127 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
128 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
129 |
----------------------- REVIEW 3 --------------------- |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
130 |
PAPER: 42 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
131 |
TITLE: A Formal Model and Correctness Proof for an Access Control Policy Framework |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
132 |
AUTHORS: Chunhan Wu, Xingyuan Zhang and Christian Urban |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
133 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
134 |
OVERALL EVALUATION: -1 (weak reject) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
135 |
REVIEWER'S CONFIDENCE: 4 (high) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
136 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
137 |
----------- REVIEW ----------- |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
138 |
The paper formalizes the Role-Compatibility (RC) model of [4,5]. It defines its dynamic semantics, a notion of "taintability" (roughly the ability to influence a resource), and a more abstract semantics that is sound and complete for tainting. Both semantics consider only static, declarative policies, rather than e.g. actual executables and file contents. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
139 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
140 |
I am a bit surprised by the choice of RC as a starting point, inasmuch as there is a vast literature on RBAC, including more widely deployed systems, and papers using SMTs to check whether a policy excludes bad traces. Also, tainting is an important property, but many others are meant to be enforced by RBAC policies. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
141 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
142 |
The paper is well written, but its theorem is not very surprising, as the three main new definitions are closely aligned. Arguably, policies being complete is too much to hope for with real RBAC systems. Also, there is not much discussion of the formalization effort itself. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
143 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
144 |
Related work: "Gran: Model Checking Grsecurity RBAC Policies" at CSF 2012 explores similar issues with policies (but does not provide formal proofs); in particular it defines a property similar to tainting, develops an abstraction for dynamic taintability, and report issues with role inheritance rules. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
145 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
146 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
147 |
----------------------- REVIEW 4 --------------------- |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
148 |
PAPER: 42 |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
149 |
TITLE: A Formal Model and Correctness Proof for an Access Control Policy Framework |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
150 |
AUTHORS: Chunhan Wu, Xingyuan Zhang and Christian Urban |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
151 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
152 |
OVERALL EVALUATION: 2 (accept) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
153 |
REVIEWER'S CONFIDENCE: 3 (medium) |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
154 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
155 |
----------- REVIEW ----------- |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
156 |
This is an appealing formalisation paper about an area with which I am not very familiar. The background explanations are clear though, so I felt I was getting a good picture of what had been done, and its significance. In essence, the paper describes an access-control formalism due to Ott called the Role-Compatibility Model, develops an "obvious" mechanisation of that model, and then describes a nice decision procedure (original to this paper?) for checking the possible effects of corruption or tainting. If one's security policy has been well-designed, then the d.p. will tell one that various forms of compromise can or can't affect permanent/important parts of the system. So, for example, a compromised user account may well spread "compromise" or "taint" over a large proportion of the system (everything that user can touch), but should not be able to affect permanent files/resources owned by root (the contents of the /usr/bin directory, say). |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
157 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
158 |
I hesitate to rank the paper more strongly because of my own lack of confidence, but this seems like a strong contribution. The formalisation of the RC model is elegantly done, but is perhaps fairly obvious given Paulson's well-known approach of characterising all possible system traces with an inductive relation. However, adding to that contribution by inventing a finite abstraction of a system's dynamic behaviour, thereby creating a verified decision procedure for access control policy correctness, seems like something that certainly deserves to be published at CPP. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
159 |
|
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
160 |
My only request for change would be for the authors to make it clearer whether or not their d.p. is original to them, and the degree of contribution they think it represents to the security community. |
d43f46423298
added reviews from CPP
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
161 |