CPP-reviews
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--
added reviews from CPP
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
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