Skip to content
Snippets Groups Projects
Commit e374df4b authored by Jaime Perez Crespo's avatar Jaime Perez Crespo
Browse files

Minor formatting issues.

parent 9c3d26ab
No related branches found
No related tags found
No related merge requests found
......@@ -68,7 +68,9 @@ class Auth
$as = new \SimpleSAML_Auth_Simple('admin');
$as->login();
} else {
throw new \SimpleSAML_Error_Exception('Cannot find "admin" auth source, and admin privileges are required.');
throw new \SimpleSAML_Error_Exception(
'Cannot find "admin" auth source, and admin privileges are required.'
);
}
}
}
\ No newline at end of file
......@@ -71,21 +71,28 @@ class System
{
$globalConfig = \SimpleSAML_Configuration::getInstance();
$tempDir = rtrim($globalConfig->getString('tempdir', sys_get_temp_dir().DIRECTORY_SEPARATOR.'simplesaml'),
DIRECTORY_SEPARATOR);
$tempDir = rtrim(
$globalConfig->getString(
'tempdir',
sys_get_temp_dir().DIRECTORY_SEPARATOR.'simplesaml'
),
DIRECTORY_SEPARATOR
);
if (!is_dir($tempDir)) {
if (!mkdir($tempDir, 0700, true)) {
$error = error_get_last();
throw new \SimpleSAML_Error_Exception('Error creating temporary directory "'.$tempDir.
'": '.$error['message']);
throw new \SimpleSAML_Error_Exception(
'Error creating temporary directory "'.$tempDir.'": '.$error['message']
);
}
} elseif (function_exists('posix_getuid')) {
// check that the owner of the temp directory is the current user
$stat = lstat($tempDir);
if ($stat['uid'] !== posix_getuid()) {
throw new \SimpleSAML_Error_Exception('Temporary directory "'.$tempDir.
'" does not belong to the current user.');
throw new \SimpleSAML_Error_Exception(
'Temporary directory "'.$tempDir.'" does not belong to the current user.'
);
}
}
......@@ -174,24 +181,27 @@ class System
$res = @file_put_contents($tmpFile, $data);
if ($res === false) {
$error = error_get_last();
throw new \SimpleSAML_Error_Exception('Error saving file "'.$tmpFile.
'": '.$error['message']);
throw new \SimpleSAML_Error_Exception(
'Error saving file "'.$tmpFile.'": '.$error['message']
);
}
if (self::getOS() !== self::WINDOWS) {
if (!chmod($tmpFile, $mode)) {
unlink($tmpFile);
$error = error_get_last();
throw new \SimpleSAML_Error_Exception('Error changing file mode of "'.$tmpFile.
'": '.$error['message']);
throw new \SimpleSAML_Error_Exception(
'Error changing file mode of "'.$tmpFile.'": '.$error['message']
);
}
}
if (!rename($tmpFile, $filename)) {
unlink($tmpFile);
$error = error_get_last();
throw new \SimpleSAML_Error_Exception('Error moving "'.$tmpFile.'" to "'.
$filename.'": '.$error['message']);
throw new \SimpleSAML_Error_Exception(
'Error moving "'.$tmpFile.'" to "'.$filename.'": '.$error['message']
);
}
}
}
......@@ -230,7 +230,7 @@ class XML
try {
$doc = \SAML2_DOMDocumentFactory::fromString($xml);
} catch(\Exception $e) {
} catch (\Exception $e) {
throw new \DOMException('Error parsing XML string.');
}
......@@ -405,7 +405,7 @@ class XML
try {
$dom = \SAML2_DOMDocumentFactory::fromString($xml);
$res = true;
} catch(Exception $e) {
} catch (Exception $e) {
$res = false;
}
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment