diff options
| author | Aleksander Machniak <alec@alec.pl> | 2013-05-06 12:07:05 +0200 |
|---|---|---|
| committer | Aleksander Machniak <alec@alec.pl> | 2013-05-06 12:08:13 +0200 |
| commit | 78c270c9f259c33b703cfb26b96b8dbb062db21e (patch) | |
| tree | 201579445599534a703482ae7fd65be9011f397c /plugins | |
| parent | e4f288c3fe8cba1e271d346ec393c02db0793773 (diff) | |
Fix bugs caught by static analysis
Diffstat (limited to 'plugins')
| -rw-r--r-- | plugins/acl/acl.php | 5 | ||||
| -rw-r--r-- | plugins/password/drivers/sql.php | 2 |
2 files changed, 4 insertions, 3 deletions
diff --git a/plugins/acl/acl.php b/plugins/acl/acl.php index a4abe28a6..466185dcc 100644 --- a/plugins/acl/acl.php +++ b/plugins/acl/acl.php @@ -433,8 +433,9 @@ class acl extends rcube_plugin $acl = trim(get_input_value('_acl', RCUBE_INPUT_GPC)); $oldid = trim(get_input_value('_old', RCUBE_INPUT_GPC)); - $acl = array_intersect(str_split($acl), $this->rights_supported()); - $users = $oldid ? array($user) : explode(',', $user); + $acl = array_intersect(str_split($acl), $this->rights_supported()); + $users = $oldid ? array($user) : explode(',', $user); + $result = 0; foreach ($users as $user) { $user = trim($user); diff --git a/plugins/password/drivers/sql.php b/plugins/password/drivers/sql.php index b08833dbf..8c8dc87b5 100644 --- a/plugins/password/drivers/sql.php +++ b/plugins/password/drivers/sql.php @@ -183,7 +183,7 @@ class rcube_sql_password $res = $db->query($sql, $sql_vars); if (!$db->is_error()) { - if (strtolower(substr(trim($query),0,6))=='select') { + if (strtolower(substr(trim($sql),0,6)) == 'select') { if ($result = $db->fetch_array($res)) return PASSWORD_SUCCESS; } else { |