AppDirsWithDifferentOwner.php 3.4 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104
  1. <?php
  2. declare(strict_types=1);
  3. /**
  4. * @copyright Copyright (c) 2023 Côme Chilliet <come.chilliet@nextcloud.com>
  5. *
  6. * @author Côme Chilliet <come.chilliet@nextcloud.com>
  7. *
  8. * @license GNU AGPL version 3 or any later version
  9. *
  10. * This program is free software: you can redistribute it and/or modify
  11. * it under the terms of the GNU Affero General Public License as
  12. * published by the Free Software Foundation, either version 3 of the
  13. * License, or (at your option) any later version.
  14. *
  15. * This program is distributed in the hope that it will be useful,
  16. * but WITHOUT ANY WARRANTY; without even the implied warranty of
  17. * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
  18. * GNU Affero General Public License for more details.
  19. *
  20. * You should have received a copy of the GNU Affero General Public License
  21. * along with this program. If not, see <http://www.gnu.org/licenses/>.
  22. *
  23. */
  24. namespace OCA\Settings\SetupChecks;
  25. use OCP\IL10N;
  26. use OCP\SetupCheck\ISetupCheck;
  27. use OCP\SetupCheck\SetupResult;
  28. class AppDirsWithDifferentOwner implements ISetupCheck {
  29. public function __construct(
  30. private IL10N $l10n,
  31. ) {
  32. }
  33. public function getName(): string {
  34. return $this->l10n->t('App directories owner');
  35. }
  36. public function getCategory(): string {
  37. return 'security';
  38. }
  39. /**
  40. * Iterates through the configured app roots and
  41. * tests if the subdirectories are owned by the same user than the current user.
  42. *
  43. * @return string[]
  44. */
  45. private function getAppDirsWithDifferentOwner(int $currentUser): array {
  46. $appDirsWithDifferentOwner = [[]];
  47. foreach (\OC::$APPSROOTS as $appRoot) {
  48. if ($appRoot['writable'] === true) {
  49. $appDirsWithDifferentOwner[] = $this->getAppDirsWithDifferentOwnerForAppRoot($currentUser, $appRoot);
  50. }
  51. }
  52. $appDirsWithDifferentOwner = array_merge(...$appDirsWithDifferentOwner);
  53. sort($appDirsWithDifferentOwner);
  54. return $appDirsWithDifferentOwner;
  55. }
  56. /**
  57. * Tests if the directories for one apps directory are writable by the current user.
  58. *
  59. * @param int $currentUser The current user
  60. * @param array $appRoot The app root config
  61. * @return string[] The none writable directory paths inside the app root
  62. */
  63. private function getAppDirsWithDifferentOwnerForAppRoot(int $currentUser, array $appRoot): array {
  64. $appDirsWithDifferentOwner = [];
  65. $appsPath = $appRoot['path'];
  66. $appsDir = new \DirectoryIterator($appRoot['path']);
  67. foreach ($appsDir as $fileInfo) {
  68. if ($fileInfo->isDir() && !$fileInfo->isDot()) {
  69. $absAppPath = $appsPath . DIRECTORY_SEPARATOR . $fileInfo->getFilename();
  70. $appDirUser = fileowner($absAppPath);
  71. if ($appDirUser !== $currentUser) {
  72. $appDirsWithDifferentOwner[] = $absAppPath;
  73. }
  74. }
  75. }
  76. return $appDirsWithDifferentOwner;
  77. }
  78. public function run(): SetupResult {
  79. $currentUser = posix_getuid();
  80. $currentUserInfos = posix_getpwuid($currentUser) ?: [];
  81. $appDirsWithDifferentOwner = $this->getAppDirsWithDifferentOwner($currentUser);
  82. if (count($appDirsWithDifferentOwner) > 0) {
  83. return SetupResult::warning(
  84. $this->l10n->t("Some app directories are owned by a different user than the web server one. This may be the case if apps have been installed manually. Check the permissions of the following app directories:\n%s", implode("\n", $appDirsWithDifferentOwner))
  85. );
  86. } else {
  87. return SetupResult::success($this->l10n->t('App directories have the correct owner "%s"', [$currentUserInfos['name'] ?? '']));
  88. }
  89. }
  90. }