Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: cherry-pick cf64617c1cc5 from chromium #34052

Merged
merged 3 commits into from May 5, 2022

Conversation

ppontes
Copy link
Member

@ppontes ppontes commented May 3, 2022

FSA: Pass File ownership to worker for async FSARFD file operations

We cannot access the backing file as a member of the
FileSystemAccessRegularFileDelegate since WrapCrossThreadPersistent
does NOT cancel the task posted to the worker pool if delegate is
destroyed. Passing ownership to the worker task ensures the file must
be alive when used. After the operation, ownership of the file is
passed back to the delegate.

This pattern was already used for the SetLength operation on old Macs.

Bug: 1299743
Change-Id: Ie00c09e8f77dc353f280af726a68ed6c572b750b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3498864
Reviewed-by: Marijn Kruisselbrink mek@chromium.org
Commit-Queue: Austin Sullivan asully@chromium.org
Cr-Commit-Position: refs/heads/main@{#980167}

Notes: Backported fix for CVE-2022-1485.

@ppontes ppontes requested review from a team as code owners May 3, 2022 16:42
@ppontes ppontes added 18-x-y backport-check-skip Skip trop's backport validity checking semver/patch backwards-compatible bug fixes labels May 3, 2022
@electron-cation electron-cation bot added new-pr 🌱 PR opened in the last 24 hours and removed new-pr 🌱 PR opened in the last 24 hours labels May 3, 2022
@zcbenz zcbenz merged commit fabdfb2 into 18-x-y May 5, 2022
@zcbenz zcbenz deleted the cherry-pick/18-x-y/chromium/cf64617c1cc5 branch May 5, 2022 00:33
@release-clerk
Copy link

release-clerk bot commented May 5, 2022

Release Notes Persisted

Backported fix for CVE-2022-1485.

vikunja-bot pushed a commit to go-vikunja/desktop that referenced this pull request May 11, 2022
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [electron](https://github.com/electron/electron) | devDependencies | patch | [`18.2.0` -> `18.2.2`](https://renovatebot.com/diffs/npm/electron/18.2.0/18.2.2) |

---

### Release Notes

<details>
<summary>electron/electron</summary>

### [`v18.2.2`](https://github.com/electron/electron/releases/v18.2.2)

[Compare Source](electron/electron@v18.2.0...v18.2.2)

### Release Notes for v18.2.2

#### Fixes

-   Fixed a crash on Windows when opening apps in multiple, separate user sessions. [#&#8203;34161](electron/electron#34161) <span style="font-size:small;">(Also in [19](https://github.com/electron/electron/pull/34160))</span>
-   Fixed an OSR crash happening when input select items were rendered. [#&#8203;34092](electron/electron#34092) <span style="font-size:small;">(Also in [17](electron/electron#34091), [19](https://github.com/electron/electron/pull/34093))</span>
-   Fixed an issue where calling `SetLoginItemSettings()` could potentially cause network volumes to be incorrectly mounted. [#&#8203;34106](electron/electron#34106) <span style="font-size:small;">(Also in [17](electron/electron#34107), [19](https://github.com/electron/electron/pull/34108))</span>
-   Fixed crash on startup due to missing gtk symbol on older distros. [#&#8203;34150](electron/electron#34150) <span style="font-size:small;">(Also in [19](https://github.com/electron/electron/pull/34151))</span>

#### Other Changes

-   Backported fix for CVE-2022-1485. [#&#8203;34052](electron/electron#34052)
-   Backported fix for CVE-2022-1497. [#&#8203;34072](electron/electron#34072)

</details>

---

### Configuration

📅 **Schedule**: At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).

Reviewed-on: https://kolaente.dev/vikunja/desktop/pulls/93
Co-authored-by: renovate <renovatebot@kolaente.de>
Co-committed-by: renovate <renovatebot@kolaente.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
18-x-y backport-check-skip Skip trop's backport validity checking security 🔒 semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants