author | Christian Urban <christian dot urban at kcl dot ac dot uk> |
Fri, 06 Sep 2013 13:27:46 +0100 | |
changeset 14 | d43f46423298 |
parent 12 | fb962189e921 |
permissions | -rw-r--r-- |
12
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
1 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
2 |
Dear Christian |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
3 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
4 |
We regret to inform you that your submission to ESORICS 2013, "A Formal Model and Correctness Proof for an Access Control Policy Framework", has not been accepted. The conference was extremely competitive this year, with 43 papers out of 242 being accepted, an acceptance rate of around 18%%. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
5 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
6 |
The reviews for your paper are appended to this email. We hope you find them useful in preparing subsequent versions of your submission. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
7 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
8 |
Best wishes |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
9 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
10 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
11 |
Jason Crampton & Sushil Jajodia |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
12 |
Programme Co-chairs |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
13 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
14 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
15 |
----------------------- REVIEW 1 --------------------- |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
16 |
PAPER: 184 |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
17 |
TITLE: A Formal Model and Correctness Proof for an Access Control Policy Framework |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
18 |
AUTHORS: Chunhan Wu, Xingyuan Zhang and Christian Urban |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
19 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
20 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
21 |
----------- REVIEW ----------- |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
22 |
The paper presents a formalization of the (dynamic) model of the |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
23 |
Role-Compatibility (RC) model, in the interactive prover Isabelle, for |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
24 |
the authorization of core operations in an operating system. The |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
25 |
authors use the inductive approach, introduced by Paulson, for |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
26 |
cryptographic protocols. The formalization of the model takes into |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
27 |
account the dynamic behavior of the operation of an operating system. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
28 |
There are two groups of rules: admissible, formalizing the conditions |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
29 |
under which operating system actions can be executed, and granted, |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
30 |
formalizing the authorization requirements. The authors develop a |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
31 |
static check for taintable objects. It is claimed that the rules can |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
32 |
be easly adapted to the access control model of SELinux. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
33 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
34 |
The English of the paper is good. I have found few typos: |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
35 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
36 |
- page 4, line -8: deletion Of IPCs --> deletion of IPCs |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
37 |
- page 5, line 17: these function --> these functions |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
38 |
- page 6, line 15: the way how the --> the way the |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
39 |
- page 7, line 5: for the the role --> for the role |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
40 |
- page 9, line 10: For example staring --> For example starting |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
41 |
- page 10, line -6: unique process IDs --> unique process ID |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
42 |
- page 12, line -6: process in tainted --> process is tainted |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
43 |
- page 14, line -13: provisio --> proviso |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
44 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
45 |
The formalisation seems to be reasonable although I would have liked |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
46 |
an appendix to the paper in which the formalization is reported fully. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
47 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
48 |
The static check for taintable objects of the operating system seems |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
49 |
to be useful and its (limited) scope of applicability is honestly |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
50 |
discussed. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
51 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
52 |
The effort of taking into account te dynamic of the system, not only |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
53 |
the authorization policies, is intersting and important to develop |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
54 |
more precise security analysis techniques. However, the authors |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
55 |
should give a better idea of what kind of properties the proposed |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
56 |
check can verify. For example, can they perform information flow |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
57 |
analysis? |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
58 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
59 |
My main concern with the paper is the lack of examples on which the |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
60 |
various rules are shown and that the static check is not applied to a |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
61 |
(possibly realistic) scenario to show its usefulness. The advantages |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
62 |
of the proposed check are only argued abstractly. It is thus |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
63 |
difficult to understand the degree of automation with which the static |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
64 |
check can be carried out. I know that Isabelle offers a high degree |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
65 |
of automation to various proof attempts but the paper does not offer |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
66 |
hints about this crucial aspect. If the static check is not fully |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
67 |
automated, then the usefulness of the formal model is greatly |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
68 |
decreased. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
69 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
70 |
For this reason, I suggest to consider the paper as borderline. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
71 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
72 |
I have also the following comments on some aspects of the paper: |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
73 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
74 |
- in Section 3, the notion of state is not precisely characterized. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
75 |
One can derive that there is a given notion of initial state and |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
76 |
that the other state can be derived by applying to the initial state |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
77 |
a (finite) sequence of actions of the operating system, but I was |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
78 |
not able to find a place where this is explictly formalized. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
79 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
80 |
- at the end of page 5, the definition of new_proc seems to be |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
81 |
unnecessarely restrictive. The thing that needs to be formalized |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
82 |
here is the fact that the function returns a distinct identifier |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
83 |
w.r.t. the existing identifiers in a state. Taking the max of the |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
84 |
identifiers (naturals) and adding one is indeed a way to ensure this |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
85 |
but it is certanly not the only way. An alterantive, more general, |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
86 |
formalization is to assume the existence of an infinite set of |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
87 |
identifiers, taking out all the identifiers in the actual state, and |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
88 |
then pick any of the remaining identifiers. This admits as an |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
89 |
implementation the proposed definition of new_proc but also many |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
90 |
others. This seems to be important in the light of the remark at |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
91 |
page 14, immediately after the statement of Theorem 2, saying that |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
92 |
"we have no control over which process ID a process will be assigned |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
93 |
with." This does not seem to hold with the proposed definition of |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
94 |
new_proc. It does hold with the alternative definition above. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
95 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
96 |
- at the end of page 7, the notion of seeds are introduced but the |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
97 |
reader is left with wondering how these are identified. This is |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
98 |
unswered only very late in the paper. I suggest the authors to give |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
99 |
some hints about this issue already here and mention that it is |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
100 |
further discussed later in the paper. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
101 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
102 |
- at page 9, towards the end of Section 3, it is discussed that the |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
103 |
restriction to the (so-called) undeletable objects of the proposed |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
104 |
static check is needed to avoid to consider infinitely many objects |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
105 |
and that this is crucial to make the check effective. I am not sure |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
106 |
to have understood exactly what the authors really mean. One may |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
107 |
(automatically) reason about infinitely many objects by using |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
108 |
suitable constraints. It may well be the case that it is possible |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
109 |
to find (finitary) symbolic representations for states containing |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
110 |
finitely many objects (but whose exact number is unknown). This is |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
111 |
standard, e.g., in the infinite state model checking of parametric |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
112 |
systems. The authors may have a look at the following seminal paper |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
113 |
to have an idea of what can be done for the verification of systems |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
114 |
with infinite state spaces: |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
115 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
116 |
@article{AbdullaTCS, |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
117 |
author = {P. A. Abdulla and B. Jonsson}, |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
118 |
title = {Model Checking of Systems with Many Identical Timed Processes}, |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
119 |
journal = {Theoretical Computer Science}, |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
120 |
vol = {290 (1)}, |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
121 |
pages = {241--264}, |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
122 |
year = 2003, |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
123 |
} |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
124 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
125 |
I believe that this point should be clarified by the authors to put |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
126 |
their verification technique in the right relationship with others. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
127 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
128 |
A similar observation can be done for the following sentence at the |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
129 |
end of page 10: "otherwise exploring all possible "reachable" |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
130 |
objects can lead to the potential infinity like in the dynamic |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
131 |
model." Indeed, there can be several sources of infinity in a |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
132 |
problem that, as shown for example by the paper by Adbulla and |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
133 |
Jonsson paper above, can be tamed and an exhaustive reachability |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
134 |
analysis can be done. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
135 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
136 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
137 |
----------------------- REVIEW 2 --------------------- |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
138 |
PAPER: 184 |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
139 |
TITLE: A Formal Model and Correctness Proof for an Access Control Policy Framework |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
140 |
AUTHORS: Chunhan Wu, Xingyuan Zhang and Christian Urban |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
141 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
142 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
143 |
----------- REVIEW ----------- |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
144 |
This work presents a specification of the role-compatibility (RC) model in the |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
145 |
Isabelle theorem prover. RC is a role-based access control model created for use |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
146 |
in Rule Set Based Access Control (RSBAC), an access control kernel module that |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
147 |
provides implementations of mandatory, role-based, and access control-list |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
148 |
models of access control for Linux. The model is to be used to detect the spread |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
149 |
of "taintable" resources. The initial state of the system is assumed to contain |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
150 |
some "seeds" (tainted files), which may then spread through the system. The |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
151 |
static check, then, is whether a particular resource (e.g., the root of the |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
152 |
filesystem) is taintable. The tainted^s relation |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
153 |
(a static formulation of the taintable relation) is defined and proven to be |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
154 |
sound and complete with respect to the work's notion of taintable. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
155 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
156 |
The authors claim that their approach is more accurate than policy analysis, |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
157 |
since it takes into account additional details of the operating system, such as |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
158 |
the processes that are started. This allows their approach to avoid the |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
159 |
"over-approximations" of policy analysis (e.g., when a process assuming a |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
160 |
particular role could cause a violation, but such a process is never created). |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
161 |
These added details, however, seem at times to be based more on the authors' |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
162 |
intuition than any particular implementation. For example: |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
163 |
- A newly created process is assigned the ID: $\Max(current_ids) + 1$. Linux |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
164 |
(where RC is currently implemented) uses $next_id++ % MAX_ID$ [unless taken] |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
165 |
(i.e., it maintains a counter which is incremented when an ID is assigned, so |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
166 |
cloning and terminating repeatedly still increases the counter rather than |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
167 |
creating the same ID every time). |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
168 |
- The set of users is assumed to be fixed (not a requirement in RSBAC or Linux). |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
169 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
170 |
Thus, the model is more all-encompassing than some approaches to policy analysis |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
171 |
due to modeling these details at all, but it falls short of verifying any |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
172 |
particular implementation since these details are not justified. The work is |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
173 |
poorly positioned among related work, as the authors seem unaware of the huge |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
174 |
body of work on various techniques in access control static analysis [e.g., |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
175 |
1--3], especially work that also models the system at the process level [3]. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
176 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
177 |
The methods for specifying the model are not claimed to be novel, and are a |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
178 |
somewhat straightforward and unsurprising application of Paulson's inductive |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
179 |
method. The authors say their approach can be easily applied to other access |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
180 |
control models, but this seems more as a result of the approach being obvious |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
181 |
than of it being particularly reusable. Furthermore, the authors do not |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
182 |
demonstrate how a practical analysis would be carried out using this model; |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
183 |
instead, they briefly discuss the way they they imagine it being used and the |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
184 |
(uncited) reasons they believe the model's assumptions (only checking |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
185 |
undeletable files for taintedness, the existence of a set of known "seed" files |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
186 |
that are tainted in the initial state) are not too restrictive for it to be |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
187 |
usable in practice. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
188 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
189 |
The work's main claimed contribution is the formalization of RC in Isabelle. The |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
190 |
specification the authors present does not use novel techniques that can be |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
191 |
reapplied elsewhere and is not shown to have a practical advantage over existing |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
192 |
analysis techniques for role-based access control models. We question, |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
193 |
therefore, whether this formalization alone is a sufficient contribution to |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
194 |
warrant acceptance to a top venue such as ESORICS. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
195 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
196 |
[1] Mikhail I. Gofman, Ruiqi Luo, Ayla C. Solomon, Yingbin Zhang, Ping Yang, |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
197 |
Scott D. Stoller: RBAC-PAT: A Policy Analysis Tool for Role Based Access |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
198 |
Control. TACAS 2009:46--49. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
199 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
200 |
[2] Scott D. Stoller, Ping Yang, C. R. Ramakrishnan, Mikhail I. Gofman: |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
201 |
Efficient policy analysis for administrative role based access control. ACM |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
202 |
Conference on Computer and Communications Security 2007:445--455. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
203 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
204 |
[3] Arosha K. Bandara, Emil Lupu, Alessandra Russo: Using Event Calculus to |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
205 |
Formalise Policy Specification and Analysis. POLICY 2003. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
206 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
207 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
208 |
----------------------- REVIEW 3 --------------------- |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
209 |
PAPER: 184 |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
210 |
TITLE: A Formal Model and Correctness Proof for an Access Control Policy Framework |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
211 |
AUTHORS: Chunhan Wu, Xingyuan Zhang and Christian Urban |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
212 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
213 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
214 |
----------- REVIEW ----------- |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
215 |
This paper focuses on the previously published framework, Role-Compatibility Model (RC-model). Their static analysis discovers semantic inconsistencies of RC-model. In particular, they indicate InheritParentRole of RC-model. Their approach seems reasonable but this work is to determine the correctness of RC-model. Instead, it would be interesting if they analyze NIST RBAC model with their approach. Since RC-model itself is not formally designed, it is hard to judge how novel their analysis work is. In addition, they claim access control policy framework is formulated but it is hard to determine how their examples are related to role-based access control policies. The authors need to use generic RBAC-centric examples if their goal is to show their approach can be adapted for other RBAC models. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
216 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
217 |
Even though they briefly address their experiments in the conclusion, it is questionable how the formulated model can support real-world RBAC policies due to the lack of details on their experiment and proofs. Also, the references should be improved including logic-based reasoning of access control policies. Here are several papers that the authors need to consider: |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
218 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
219 |
Elisa Bertino, Barbara Catania, Elena Ferrari, and Paolo Perlasca. A logical framework for reasoning about access control models. ACM Trans. Inf. Syst. Secur, 6:71–127, 2003. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
220 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
221 |
J. Bryans. Reasoning about XACML policies using CSP. In Proceedings of the 2005 workshop on Secure web services, page 35. ACM, 2005. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
222 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
223 |
K. Fisler, S. Krishnamurthi, L.A. Meyerovich, and M.C. Tschantz. Verification and changeimpact analysis of access control policies. In Proceedings of the 27th international conference on Software engineering, pages 196–205. ACM New York, NY, USA, 2005. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
224 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
225 |
V. Kolovski, J. Hendler, and B. Parsia. Analyzing web access control policies. In Proceedings of the 16th international conference on World Wide Web, page 686. ACM, 2007. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
226 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
227 |
Joseph Y. Halpern and Vicky Weissman. Using first-order logic to reason about policies. ACM Trans. Inf. Syst. Secur., 11(4), 2008. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
228 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
229 |
Prathima Rao, Dan Lin, Elisa Bertino, Ninghui Li, Jorge Lobo: EXAM: An Environment for Access Control Policy Analysis and Management. POLICY 2008: 238-240 |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
230 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
231 |
Alessandro Armando, Enrico Giunchiglia, and Serena Elisa Ponta. Formal specification and automatic analysis of business processes under authorization constraints: an action-based approach. In Proceedings of the 6th International Conference on Trust, Privacy and Security in Digital Business (TrustBus’09), 2009. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
232 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
233 |
G. Ahn, Hongxin Hu, Joohyung Lee, and Yunsong Meng. Reasoning about XACML policy descriptions in answer set programming (preliminary report). In Proceedings of International Workshop on Nonmonotonic Reasoning (NMR), 2010. |
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
234 |
|
fb962189e921
added reviews from ESORICS
Christian Urban <christian dot urban at kcl dot ac dot uk>
parents:
diff
changeset
|
235 |
G. Ahn, Hongxin Hu, Joohyung Lee, and Yunsong Meng. Representing and reasoning about web access control policies. In Proc. 34th Annual IEEE Computer Software and Applications Conference (COMPSAC 2010), pages 137–146, 2010. |