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] Squirrel.Windows-1.9.1 #36

Closed
wants to merge 2 commits into from

Conversation

nick-invision
Copy link

Squirrel.Windows-1.9.0 and older has a DLL hijacking vulnerability that was fixed here and published as 1.9.1. I understand that electron-builder-squirrel-windows is deprecated as of 1.9.0, but this security fix should at least be resolved in case people have no other reason to migrate off of squirrel.

Files copied directly from Squirrel.Windows.

This will also resolve issue #33

@Obi-Dann
Copy link

Obi-Dann commented Dec 6, 2020

Hi, I think it won't work because electron-builder is actually using a forked modified version of Squirrel.Windows #1
It looks like electron-builder's Squirrel.Windows is built from https://github.com/develar/Squirrel.Windows

So, in order to get it working, https://github.com/develar/Squirrel.Windows needs to be forked, the fix should be pulled from upstream and these binaries need to be rebuilt. Alternatively, electron-builder needs to be change to get it working with the "normal" non-hacked version of Squirrel.Windows. TBH, it really sucks that electron-builder uses a forked modified version of Squirrel.Windows without saying it anywhere specifically

@nick-invision
Copy link
Author

That's unfortunate. We forked this repo and electron builder and so that we could use the 1.9.1 release directly from squirrel.windows and haven't run into any issues yet. It would be nice to know why the squirrel fork was needed in the first place. I'll look into updating the squirrel fork this week.

@Obi-Dann
Copy link

Obi-Dann commented Dec 8, 2020

@nick-invision I have been trying to solve a similar problem, Squirrel.Windows 1.9.0 stopped generating deltas on large files. I ended up forking https://github.com/develar/Squirrel.Windows, setting up CD on github actions and pulling some of fixes from upstream. If you need some inspiration, here's the repo https://github.com/AurorNZ/Squirrel.Windows/tree/v1.x.

I am keen to contribute to make it some of the works official and, perhaps, help maintaining Squirrel.Windows for electron. @develar what do you think?
P.S. sorry that I am using a PR like a github issue

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.

None yet

2 participants