]> source.dussan.org Git - nextcloud-server.git/commitdiff
Remove @suppress SqlInjectionChecker 22883/head
authorMorris Jobke <hey@morrisjobke.de>
Wed, 16 Sep 2020 13:53:51 +0000 (15:53 +0200)
committerMorris Jobke <hey@morrisjobke.de>
Wed, 16 Sep 2020 13:53:56 +0000 (15:53 +0200)
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
20 files changed:
apps/dav/lib/CalDAV/CalDavBackend.php
apps/dav/lib/CardDAV/CardDavBackend.php
apps/files_external/lib/Service/DBConfigService.php
core/Command/Db/ConvertType.php
core/Migrations/Version14000Date20180404140050.php
lib/private/BackgroundJob/JobList.php
lib/private/Comments/Manager.php
lib/private/DB/Adapter.php
lib/private/DB/AdapterPgSql.php
lib/private/DB/Connection.php
lib/private/Files/Cache/Cache.php
lib/private/Files/Cache/Propagator.php
lib/private/Files/Config/UserMountCache.php
lib/private/Lock/DBLockingProvider.php
lib/private/Repair/CleanTags.php
lib/private/Repair/OldGroupMembershipShares.php
lib/private/Repair/RepairInvalidShares.php
lib/private/Security/Bruteforce/Throttler.php
lib/private/Setup/PostgreSQL.php
lib/public/AppFramework/Db/QBMapper.php

index 8f37cab384d39380538144de8240be5cc03a34ef..d0f88195e58a746bce327c8183444c30b2b249af 100644 (file)
@@ -742,7 +742,6 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
         * @param string $calendarUri
         * @param array $properties
         * @return int
-        * @suppress SqlInjectionChecker
         */
        public function createCalendar($principalUri, $calendarUri, array $properties) {
                $values = [
@@ -818,9 +817,6 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
                $supportedProperties = array_keys($this->propertyMap);
                $supportedProperties[] = '{' . Plugin::NS_CALDAV . '}schedule-calendar-transp';
 
-               /**
-                * @suppress SqlInjectionChecker
-                */
                $propPatch->handle($supportedProperties, function ($mutations) use ($calendarId) {
                        $newValues = [];
                        foreach ($mutations as $propertyName => $propertyValue) {
@@ -2148,9 +2144,6 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
                $supportedProperties = array_keys($this->subscriptionPropertyMap);
                $supportedProperties[] = '{http://calendarserver.org/ns/}source';
 
-               /**
-                * @suppress SqlInjectionChecker
-                */
                $propPatch->handle($supportedProperties, function ($mutations) use ($subscriptionId) {
                        $newValues = [];
 
index 28f48b61e9952593229d9c3fc5e30dc991f53ea7..14ade3f16e2188839282b81ea614ce823a6403b2 100644 (file)
@@ -374,9 +374,6 @@ class CardDavBackend implements BackendInterface, SyncSupport {
                        '{' . Plugin::NS_CARDDAV . '}addressbook-description',
                ];
 
-               /**
-                * @suppress SqlInjectionChecker
-                */
                $propPatch->handle($supportedProperties, function ($mutations) use ($addressBookId) {
                        $updates = [];
                        foreach ($mutations as $property => $newValue) {
index b1313f96e45c5b5bbbb8f77af8d02382cad53fb5..eaa5815cd30ee64bfd42ba9dc18aa90ac3a1d297 100644 (file)
@@ -153,7 +153,6 @@ class DBConfigService {
         * Get admin defined mounts
         *
         * @return array
-        * @suppress SqlInjectionChecker
         */
        public function getAdminMounts() {
                $builder = $this->connection->getQueryBuilder();
@@ -198,7 +197,6 @@ class DBConfigService {
         * @param int $type any of the self::APPLICABLE_TYPE_ constants
         * @param string|null $value user_id, group_id or null for global mounts
         * @return array
-        * @suppress SqlInjectionChecker
         */
        public function getAdminMountsFor($type, $value) {
                $builder = $this->connection->getQueryBuilder();
@@ -214,7 +212,6 @@ class DBConfigService {
         * @param int $type any of the self::APPLICABLE_TYPE_ constants
         * @param string[] $values user_ids or group_ids
         * @return array
-        * @suppress SqlInjectionChecker
         */
        public function getAdminMountsForMultiple($type, array $values) {
                $builder = $this->connection->getQueryBuilder();
@@ -238,7 +235,6 @@ class DBConfigService {
         * @param int $type any of the self::APPLICABLE_TYPE_ constants
         * @param string|null $value user_id, group_id or null for global mounts
         * @return array
-        * @suppress SqlInjectionChecker
         */
        public function getUserMountsFor($type, $value) {
                $builder = $this->connection->getQueryBuilder();
index 2c4b7d14de3817b78848c1c0aadd8297996f3bab..05bdef95bcb049f36e51885683becf48ab85b602 100644 (file)
@@ -295,7 +295,6 @@ class ConvertType extends Command implements CompletionAwareInterface {
         * @param Table $table
         * @param InputInterface $input
         * @param OutputInterface $output
-        * @suppress SqlInjectionChecker
         */
        protected function copyTable(Connection $fromDB, Connection $toDB, Table $table, InputInterface $input, OutputInterface $output) {
                if ($table->getName() === $toDB->getPrefix() . 'migrations') {
index 80312c162fab403ca75a9331b3938748141fd662..d9d7cf30d411fceb18b9114830523919cfe63344 100644 (file)
@@ -78,8 +78,6 @@ class Version14000Date20180404140050 extends SimpleMigrationStep {
         * @param IOutput $output
         * @param \Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
         * @param array $options
-        *
-        * @suppress SqlInjectionChecker
         */
        public function postSchemaChange(IOutput $output, \Closure $schemaClosure, array $options) {
                $qb = $this->connection->getQueryBuilder();
index d9d80374cfdd7d68ef5c298039bb4d64dcf9e158..37aecd61693ed0124c0e623eab15b981ae544211 100644 (file)
@@ -180,7 +180,6 @@ class JobList implements IJobList {
         * get the next job in the list
         *
         * @return IJob|null
-        * @suppress SqlInjectionChecker
         */
        public function getNext() {
                $query = $this->connection->getQueryBuilder();
@@ -300,7 +299,6 @@ class JobList implements IJobList {
         * Remove the reservation for a job
         *
         * @param IJob $job
-        * @suppress SqlInjectionChecker
         */
        public function unlockJob(IJob $job) {
                $query = $this->connection->getQueryBuilder();
index a0a670817f45dad10a60f9021a903d23c6491df5..1acfe79824c8163a8dbe12ba6be94190f655267f 100644 (file)
@@ -594,8 +594,6 @@ class Manager implements ICommentsManager {
         * @param int $folderId
         * @param IUser $user
         * @return array [$fileId => $unreadCount]
-        *
-        * @suppress SqlInjectionChecker
         */
        public function getNumberOfUnreadCommentsForFolder($folderId, IUser $user) {
                $qb = $this->dbConn->getQueryBuilder();
@@ -935,7 +933,6 @@ class Manager implements ICommentsManager {
         * @param \DateTime $dateTime
         * @param IUser $user
         * @since 9.0.0
-        * @suppress SqlInjectionChecker
         */
        public function setReadMark($objectType, $objectId, \DateTime $dateTime, IUser $user) {
                $this->checkRoleParameters('Object', $objectType, $objectId);
index f27a90dd690758a2735933e64962ad433c85e793..562fea25f5cd3e4529d81482574fd482be35068f 100644 (file)
@@ -130,9 +130,6 @@ class Adapter {
                }
        }
 
-       /**
-        * @suppress SqlInjectionChecker
-        */
        public function insertIgnoreConflict(string $table,array $values) : int {
                try {
                        $builder = $this->conn->getQueryBuilder();
index 7f7f51507804e110929dfdfd988093f3f651cd0f..77f0b6b77225f8ef4e859f0f8bfd888f03f669d1 100644 (file)
@@ -41,9 +41,6 @@ class AdapterPgSql extends Adapter {
                return $statement;
        }
 
-       /**
-        * @suppress SqlInjectionChecker
-        */
        public function insertIgnoreConflict(string $table,array $values) : int {
                if ($this->isPre9_5CompatMode() === true) {
                        return parent::insertIgnoreConflict($table, $values);
index 3b24703d4341616dc789c826bfc451fe295a98e2..c5766dcd7e07e4cecb69513c8d99fec39b657ff8 100644 (file)
@@ -279,7 +279,6 @@ class Connection extends ReconnectWrapper implements IDBConnection {
         * @return int number of new rows
         * @throws \Doctrine\DBAL\DBALException
         * @throws PreConditionNotMetException
-        * @suppress SqlInjectionChecker
         */
        public function setValues($table, array $keys, array $values, array $updatePreconditionValues = []) {
                try {
index 77289e674b3e9fd590b9fb69a56fcaf91ff5bf77..8a173f351970af2e253a49340a72740651c19ff7 100644 (file)
@@ -254,8 +254,6 @@ class Cache implements ICache {
         *
         * @return int file id
         * @throws \RuntimeException
-        *
-        * @suppress SqlInjectionChecker
         */
        public function insert($file, array $data) {
                // normalize file
@@ -613,7 +611,6 @@ class Cache implements ICache {
         * @param string $targetPath
         * @throws \OC\DatabaseException
         * @throws \Exception if the given storages have an invalid id
-        * @suppress SqlInjectionChecker
         */
        public function moveFromCache(ICache $sourceCache, $sourcePath, $targetPath) {
                if ($sourceCache instanceof Cache) {
index 92fa6436548fd5d70ffacafb93ae58a9ce869666..c9200d33b1178765fba90963e2c9f8224213878d 100644 (file)
@@ -63,7 +63,6 @@ class Propagator implements IPropagator {
         * @param string $internalPath
         * @param int $time
         * @param int $sizeDifference number of bytes the file has grown
-        * @suppress SqlInjectionChecker
         */
        public function propagateChange($internalPath, $time, $sizeDifference = 0) {
                // Do not propogate changes in ignored paths
@@ -156,7 +155,6 @@ class Propagator implements IPropagator {
 
        /**
         * Commit the active propagation batch
-        * @suppress SqlInjectionChecker
         */
        public function commitBatch() {
                if (!$this->inBatch) {
index 0dc3e5d45292409ba577e73a09aab4e6029295ef..6fa9cd96bfe3e52c8ad88d2f9d7f8cb8158250ca 100644 (file)
@@ -375,7 +375,6 @@ class UserMountCache implements IUserMountCache {
        /**
         * @param array $users
         * @return array
-        * @suppress SqlInjectionChecker
         */
        public function getUsedSpaceForUsers(array $users) {
                $builder = $this->connection->getQueryBuilder();
index f48bf57028f9b2145a7ba8f1e9d52a5e29241a6e..2ad7c162f36fe03ce1282b582d99d111a5e14245 100644 (file)
@@ -216,8 +216,6 @@ class DBLockingProvider extends AbstractLockingProvider {
        /**
         * @param string $path
         * @param int $type self::LOCK_SHARED or self::LOCK_EXCLUSIVE
-        *
-        * @suppress SqlInjectionChecker
         */
        public function releaseLock(string $path, int $type) {
                $this->markRelease($path, $type);
@@ -288,8 +286,6 @@ class DBLockingProvider extends AbstractLockingProvider {
 
        /**
         * release all lock acquired by this instance which were marked using the mark* methods
-        *
-        * @suppress SqlInjectionChecker
         */
        public function releaseAll() {
                parent::releaseAll();
index 959eb50e5a7679526281e2520d0ea5122744862c..f414ef09efc1757e58d175e2264414a8c3f48f9b 100644 (file)
@@ -171,7 +171,6 @@ class CleanTags implements IRepairStep {
         * @param string $sourceId
         * @param string $sourceNullColumn      If this column is null in the source table,
         *                                                              the entry is deleted in the $deleteTable
-        * @suppress SqlInjectionChecker
         */
        protected function deleteOrphanEntries(IOutput $output, $repairInfo, $deleteTable, $deleteId, $sourceTable, $sourceId, $sourceNullColumn) {
                $qb = $this->connection->getQueryBuilder();
index 2b2fd9cd470527f12e176bee861dc9c74ba0d07f..38254188221ad15e27849106187dc6980c0fec1b 100644 (file)
@@ -66,7 +66,6 @@ class OldGroupMembershipShares implements IRepairStep {
         * Must throw exception on error.
         *
         * @throws \Exception in case of failure
-        * @suppress SqlInjectionChecker
         */
        public function run(IOutput $output) {
                $deletedEntries = 0;
index 27defdeda013251aee11e7fa88676bcf9213480d..1fc98a1d466a75c47c73d643c1e3b28eb95ea5ac 100644 (file)
@@ -56,7 +56,6 @@ class RepairInvalidShares implements IRepairStep {
 
        /**
         * Adjust file share permissions
-        * @suppress SqlInjectionChecker
         */
        private function adjustFileSharePermissions(IOutput $out) {
                $mask = \OCP\Constants::PERMISSION_READ | \OCP\Constants::PERMISSION_UPDATE | \OCP\Constants::PERMISSION_SHARE;
index 61e18088e9456f4eac26e5f2788e67c913334451..01c0e02ef8f669ffad7fb2118e76b295f3a5bcc3 100644 (file)
@@ -115,7 +115,6 @@ class Throttler {
         * @param string $action
         * @param string $ip
         * @param array $metadata Optional metadata logged to the database
-        * @suppress SqlInjectionChecker
         */
        public function registerAttempt(string $action,
                                                                        string $ip,
index 17ae991a1de226ad380f4e676078e34d2142ec20..3d7a0b2a4b13d30dffe0a2bd82c68b4bb30e7f49 100644 (file)
@@ -39,7 +39,6 @@ class PostgreSQL extends AbstractDatabase {
        /**
         * @param string $username
         * @throws \OC\DatabaseSetupException
-        * @suppress SqlInjectionChecker
         */
        public function setupDatabase($username) {
                try {
index ebbe92e7875f7c73a4e1844645e88f21377e2af6..f6009c5de876e66ccd66e389ef9bd99de2434e99 100644 (file)
@@ -106,7 +106,6 @@ abstract class QBMapper {
         * @param Entity $entity the entity that should be created
         * @return Entity the saved entity with the set id
         * @since 14.0.0
-        * @suppress SqlInjectionChecker
         */
        public function insert(Entity $entity): Entity {
                // get updated fields to save, fields have to be set using a setter to
@@ -145,7 +144,6 @@ abstract class QBMapper {
         * @return Entity the saved entity with the (new) id
         * @throws \InvalidArgumentException if entity has no id
         * @since 15.0.0
-        * @suppress SqlInjectionChecker
         */
        public function insertOrUpdate(Entity $entity): Entity {
                try {
@@ -161,7 +159,6 @@ abstract class QBMapper {
         * @param Entity $entity the entity that should be created
         * @return Entity the saved entity with the set id
         * @since 14.0.0
-        * @suppress SqlInjectionChecker
         */
        public function update(Entity $entity): Entity {
                // if entity wasn't changed it makes no sense to run a db query