Skip to content

Commit

Permalink
Merge pull request #36267 from nextcloud/bugfix/noid/dont-reuse-query…
Browse files Browse the repository at this point in the history
…-builder

Don't reuse query builder object with populated parameters in FederatedShareProvider
  • Loading branch information
nickvergessen authored Feb 6, 2023
2 parents eca7ab3 + f717219 commit fa1d50c
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions apps/federatedfilesharing/lib/FederatedShareProvider.php
Original file line number Diff line number Diff line change
Expand Up @@ -618,6 +618,7 @@ private function removeShareFromTableById($shareId) {
->andWhere($qb->expr()->neq('share_type', $qb->createNamedParameter(IShare::TYPE_CIRCLE)));
$qb->execute();

$qb = $this->dbConnection->getQueryBuilder();
$qb->delete('federated_reshares')
->where($qb->expr()->eq('share_id', $qb->createNamedParameter($shareId)));
$qb->execute();
Expand Down

0 comments on commit fa1d50c

Please sign in to comment.