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

[Security] Update electron to 24.8.5 and release against CVE-2023-5217 #2081

Closed
ShamrockLee opened this issue Oct 5, 2023 · 2 comments · Fixed by #2082
Closed

[Security] Update electron to 24.8.5 and release against CVE-2023-5217 #2081

ShamrockLee opened this issue Oct 5, 2023 · 2 comments · Fixed by #2082

Comments

@ShamrockLee
Copy link

CVE-2023-5217 is a heap buffer overflow in vp8 encoding in libvpx in Google Chrome prior to 117.0.5938.132 and libvpx 1.13.1 allowed a remote attacker to potentially exploit heap corruption via a crafted HTML page, according to GitHub Advisory Database.

Please consider updating the Electron dependency to 24.8.5 (or any of the patched version listed by the page linked) making a security release to keep this app safe.

Thank you for this great project!

@dusansimic
Copy link
Collaborator

Thanks for the effort to maintain the Nix package and notify us of this security flaw 😁. Some of these CVEs are not really that critical in case of Caprine since we don't render any custom HTML pages, let alone ones created by the user (any links are actually open separately in the default browser). We will continue to update Electron so we're up to date with all security fixes but I think it's good to know that they are not always that critical, it generally depends on the app 😄.

Besides this, I'm planning on refactoring the codebase to use a JS builder so we could use ES instead of CommonJS modules which most of our dependencies currently use but Electron does not support. That way we could more easily update all other dependencies too and have all the latest security fixes.

@dusansimic
Copy link
Collaborator

And also, in the last issue on nixpkgs repo I saw that you made the binary package instead of one built from source since you had some trouble building the app. I'm not that experiences with NixOS but if I could be of any help, don't hesitate to reach out! 😁 You can contact me via email on my GitHub profile or the Matrix account from my website.

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 a pull request may close this issue.

2 participants