ESORICS-Reviews
author Christian Urban <christian dot urban at kcl dot ac dot uk>
Sun, 23 Jun 2013 03:55:49 +0100
changeset 12 fb962189e921
permissions -rw-r--r--
added reviews from ESORICS
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
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.