Skip to content

Conversation

@ge0rdi
Copy link
Member

@ge0rdi ge0rdi commented Aug 18, 2018

  • Remove warning about very old ClassicShell version
  • Remove obsolete/unused stuff
  • Try to terminate ClassicShell Start Menu (to make upgrade a bit easier)

Also fixes #72 along the way.

ge0rdi added 3 commits August 18, 2018 20:57
Latest version of ClassicShell was 4.3.1. There is no reason to display
any warning about very old versions (< 2.0.0).

Besides, this will remove dependency on version.dll which could be misused
for DLL hijacking attack.

Fixes Open-Shell#72
There is no need for password protected beta versions.
…e a bit easier)

During upgrade from ClassicShell we'll try to terminate
ClassicStartMenu.exe as well. So that installer won't complain about files
being in use.
@ge0rdi ge0rdi requested a review from XenHat August 18, 2018 20:40
@AppVeyorBot
Copy link

Build Open-Shell-Menu 4.4.131-xerdeswt completed (commit 3168cc6020 by @ge0rdi)

@ge0rdi ge0rdi merged commit a167291 into Open-Shell:master Aug 19, 2018
@ge0rdi ge0rdi deleted the setup-improvements branch August 19, 2018 06:56
@DAOWAce
Copy link

DAOWAce commented Aug 20, 2018

Not quite the right place to put this in, but just wanted to comment.

After my last post half a month ago, I decided to finally upgrade.

Backed up settings, uninstalled, rebooted, installed v.131 (explorer/start menu).

No issues of any kind. Successfully migrated old data (no import needed), immediately started working without an explorer restart (afaik).

Now to start testing to see if the issues I suffered with 4.3.1 are fixed.. and if it works with Win10 versions later than 1703 which I'm about to test upgrade to (1803).

@coddec
Copy link
Member

coddec commented Aug 20, 2018

@DAOWAce 👍 👍 😉

@kv-gh
Copy link

kv-gh commented Sep 23, 2018

Did this fix #76 ?

@XenHat
Copy link
Member

XenHat commented Sep 28, 2018

Edit: NVM

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.

Installer: Prevent DLL hijacking attack

6 participants