897529907975ecc88aa8a2fca8dda140602ddf11
[ssproject1617.git] / report / v4_access.tex
1
2 \noindent
3 The \CMS{} has the following access control mechanisms:
4
5 \begin{itemize}
6 \item A login mechanism, where logged in users are allowed to access the backend, and anonymous users are not.
7 \item For logged in users, a role system with three specified roles: admin, editor and regular user.
8 \item Different users (cap.~diff?)
9 \end{itemize}
10
11 \noindent
12 Some typical objects of access control, in this case, are:
13
14 \begin{itemize}
15 \item Database records
16 \item Editing capabilities
17 \item Role privileges
18 \item Folders, files, info over these data
19 \end{itemize}
20
21 Our check reveales that the access control mechanisms are basically only a stub, and haven't been developed to their usually implied meaning, thus flattening the access control to the single ascept of being logged in or not. Hence, the main remaining security consideration deal with whether this login mechanism protects `backend' objects from anonymous users.
22
23 These are the results of our check:
24
25 \begin{enumerate}[label={V4.\arabic*}]
26
27 % Access controls:
28 % - principle of least privilege?
29 % - inter-role/context safety
30 % - fail safely?
31 % - presentation -- server side
32 % - cannot be manipulated
33 % - onion layers
34 % - safe/working auth in place
35
36 \item
37 \fail{}
38 Verify that the principle of least privilege exists {-} users
39 should only be able to access functions, data files, URLs,
40 controllers, services, and other resources, for which they
41 possess specific authorization. This implies protection
42 against spoofing and elevation of privilege.
43
44 \begin{result}
45 The login system that is in place seems to work, and at least theoretically only allows registered users to access the backend.
46
47 There is a role system, but it seems only to be a stub, for the role of a user has no effect whatsoever on their capabilities.
48 \end{result}
49
50 \addtocounter{enumi}{2}
51 \item
52 \fail{}
53 Verify that access to sensitive records is protected, such
54 that only authorized objects or data is accessible to each
55 user (for example, protect against users tampering with a
56 parameter to see or alter another user's account).
57
58 \begin{result}
59 There is no different access context for distinct users. In particular, they
60 are allowed to access and edit each others' account info, including
61 password. Taken one way, this could be said to be by design and thus OK\. But in any reasonable design concept allowing for distinct user accounts, this is clearly not the desired setup.
62 \end{result}
63
64 \item
65 \fail{}
66 Verify that directory browsing is disabled unless
67 deliberately desired. Additionally, applications should not
68 allow discovery or disclosure of file or directory metadata,
69 such as \code{Thumbs.db}, \code{.DS\_Store}, \code{.git} or \code{.svn} folders.
70
71 \begin{result}
72 \begin{itemize}[leftmargin=*]
73 \item \code{.gitignore} accessible, as well as any other dot-preceded file (except \code{.htaccess} itself by default Apache rules), as well as files such as \code{Thumbs.db} and \code{.DS\_Store}.
74 \item Directory contents were listed in my simple setup. A global apache setting may disable by default, but the \code{.htaccess} file doesn't explicitly disable (with \code{Options -Indexes}), so that the \CMS{}'s codebase basically enables the listing by default.
75 \end{itemize}
76 \end{result}
77
78 \addtocounter{enumi}{2}
79 \item
80 \pass{}
81 Verify that access controls fail securely.
82
83 \begin{result}
84 The only remaining access control mechanism is that of logging in. It fails securely, in the sense that no info about the correctness of the username/password pair are leaked (response time analysis not included); the single error message reads: ``Incorrect details.''
85 \end{result}
86
87 \item
88 \fail{}
89 Verify that the same access control rules implied by the
90 presentation layer are enforced on the server side.
91
92 \begin{result}
93 Fail, because the role and distinct user systems are stubs.
94 \end{result}
95
96 \item
97 \fail{}
98 Verify that all user and data attributes and policy
99 information used by access controls cannot be
100 manipulated by end users unless specifically authorized.
101
102 \begin{result}
103 This item is the main remaining security concern, as the login form allows SQL
104 injections that are capable to alter any information stored in the database.
105 This is described in more detail in item V2.6 on page~\pageref{auth:6}.
106 \end{result}
107
108 \notapplicable{
109 \item
110 Verify that there is a centralized mechanism (including
111 libraries that call external authorization services) for
112 protecting access to each type of protected resource.
113 }
114
115 \item
116 \fail{}
117 Verify that all access control decisions can be logged and
118 all failed decisions are logged.
119
120 \begin{result}
121 No such decision logging present. There is only a minor amount of logging, and this is related to not finding content (pages and articles).
122 \end{result}
123
124 \item
125 \fail{}
126 Verify that the application or framework uses strong
127 random anti-\CSRF{} tokens or has another transaction
128 protection mechanism.
129
130 \begin{result}
131 There is no transation protection mechanism at all.
132 \end{result}
133
134 \item
135 \fail{}
136 Verify the system can protect against aggregate or
137 continuous access of secured functions, resources, or
138 data. For example, consider the use of a resource
139 governor to limit the number of edits per hour or to
140 prevent the entire database from being scraped by an
141 individual user.
142
143 \begin{result}
144 No such prevention present.
145 \end{result}
146
147 \item
148 \fail{}
149 Verify the application has additional authorization (such
150 as step up or adaptive authentication) for lower value
151 systems, and / or segregation of duties for high value
152 applications to enforce anti-fraud controls as per the risk
153 of application and past fraud.
154
155 \begin{result}
156 No such mechanisms.
157 \end{result}
158
159 \item
160 \fail{}
161 Verify that the application correctly enforces context-
162 sensitive authorisation so as to not allow unauthorised
163 manipulation by means of parameter tampering.
164
165 \begin{result}
166 No such mechanisms.
167 \end{result}
168
169 \end{enumerate}