Skip to content

[v2, windows] Fix WebView2 minimum runtime version check - #1456

Merged
leaanthony merged 2 commits into
wailsapp:masterfrom
stffabi:feature/windows-fix-webview-minversionchecks
Jun 16, 2022
Merged

leaanthony merged 2 commits into
wailsapp:masterfrom
stffabi:feature/windows-fix-webview-minversionchecks

Conversation

@stffabi

@stffabi stffabi commented Jun 15, 2022

Copy link
Copy Markdown
Contributor

No description provided.

stffabi added 2 commits June 15, 2022 20:38
Otherwise 4294967295 will be returned instead of -1 and the
minimum version check can't detect an older version.
So an older version than the minimum might be used and will
result in AccessViolationExceptions.
The Webview2Loader.dll are already at version 1.0.992.28 for all platforms.
@stffabi
stffabi requested a review from leaanthony June 15, 2022 18:42
}

var result int
var result int32

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch

@leaanthony
leaanthony merged commit 884773a into wailsapp:master Jun 16, 2022
@stffabi
stffabi deleted the feature/windows-fix-webview-minversionchecks branch June 16, 2022 09:56
@stffabi

stffabi commented Jun 16, 2022

Copy link
Copy Markdown
Contributor Author

Thanks for merging

@leaanthony

Copy link
Copy Markdown
Member

Thanks for raising the PR! 👍👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants