1
0
mirror of https://github.com/valinet/ExplorerPatcher.git synced 2024-11-14 19:17:35 +01:00
Commit Graph

669 Commits

Author SHA1 Message Date
Valentin Radu
0344a5e156 Version: 22621.1413.54.5 2023-03-21 00:00:56 +02:00
Valentin Radu
6bc2ea5d2b All: Fix crash when attempting to hook function on older OS builds
`RtlQueryFeatureConfiguration` is not available on old Windows 10
builds, like 17763 (LTSC 2019).
Related issue: https://github.com/valinet/ExplorerPatcher/discussions/1142
2023-03-20 23:58:32 +02:00
Valentin Radu
8e9403cfb5 Version: 22621.1413.54.4 2023-03-20 20:49:44 +02:00
Valentin Radu
5649a83739 Start11: Fixed a bug that prevented the menu from working when the setting "Disable Recommended section" is used and the display scaling is 125%
Related issue: https://github.com/valinet/ExplorerPatcher/issues/1118
2023-03-20 20:48:34 +02:00
Valentin Radu
4e55feefc3 Version: 22621.1413.54.3 2023-03-19 05:28:43 +02:00
Valentin Radu
27a8fd9a6b Start11: Better enforcement for disabling the "Recommended" section 2023-03-19 05:27:36 +02:00
Valentin Radu
0de81fdc68 Resources: Updated copyright info 2023-03-19 05:27:08 +02:00
Valentin Radu
a5e5287954 Weather: Fixed a bug that prevented the widget from displaying correctly 2023-03-18 22:31:17 +02:00
Valentin Radu
8b5443d59b Version: 22621.1413.54.1 2023-03-18 16:02:40 +02:00
Valentin Radu
2a1aad2d03 Weather: Fixed widget icons when using Microsoft icon pack
This basically undoes commit a8c7fba and properly fixes the issue:
it seems Google moved the images in a separate child `img` tag, under
what I was previously looking for. Furthermore, the src is not a URL
anymore, but directly the image encoded in base64. Fortunately, we
can match the last characters against images obtained from their 48px
and 64px endpoints. More details here:
https://github.com/valinet/ExplorerPatcher/discussions/755#discussioncomment-5353030

It seems Google has introduced a bug, where they do not change the
main image when changing to another day in the widget. Hopefully they
will fix it in the future.
2023-03-18 16:00:39 +02:00
Valentin Radu
1738b45866 Setup: explorer will restart using the token it was running under before starting application maintenance
The patch has been adapted to employ the old behavior when setup is
elevated using the same credentials, while using the updated code with
`CreateProcessWithTokenW` when otherwise.

Original description (via email):

"I have two accounts on my Windows machine: A normal one that is a
standard user (not admin) which I use as my main account where I have
ExplorerPatcher installed and configured, and an Admin account which is
 a Windows administrator account.

During installation and update the installer restarts itself and
requests admin privileges. For this I have to provide the password to
the Admin account. The installer then runs as that Admin user, stops
the explorer process, installs ExplorerPatcher and then tries to start
the explorer again. But the explorer never starts, which leaves me with
 an empty screen and a session without an explorer. I can then start a
Taskmanager via Ctrl + Shift + Esc and manually start the explorer, but
 this is annoying and maybe even frightening for a nontechnical user.

The reason why the explorer is not started again is that it is started
as the wrong user. It is started as the Admin user, which isn't logged
in so the explorer quits immediately.

The fix is to remember the user that the explorer was running under and
then start the new explorer for that user. I have tested these changes
in a Windows 11 virtual machine, by installing and uninstalling for a
standard user, as well as installing and uninstalling for an
administrator user."

Original patch: https://github.com/Abestanis/ExplorerPatcher/tree/fix_explorer_restart
Credit @Abestanis
2023-03-18 05:28:13 +02:00
Valentin Radu
d7e5b7d3c9 All: Implemented a mechanism to stop repeated crashes
When Windows is updated, it may fundamentally change the layout and
functionality of `explorer`'s internal data structure. Since
ExplorerPatcher intensively patches those in order to deliver its
functionality, it may lead to `explorer` crashing when it performs
these modifications while being unaware (outdated) of the latest OS
changes. The worst scenario is when the crashes happen when `explorer`
starts up (for example, when the user logs in), and could cause the
user to experience an endless loop of `explorer` crashes, and leave
him/her unable to practically use the computer.

In order to mitigate this scenario, when `explorer` starts up as the
shell, ExplorerPatcher increments a counter stored in the registry.
After a predefined timeout (by default, 10 seconds), ExplorerPatcher
will reset the counter to 0. If `explorer` crashes during this period,
the counter will not be reset. When `explorer` restarts, the cycle
repeats. If the counter reaches a predefined value (by default, 3),
instead of starting up again normally (and very probably crashing
again), ExplorerPatcher will display a message window informing the
user about what is happening, offering a few suggestions on how to
proceed next and disable its entire functionality until the next
File Explorer restart, in order to give the user a chance to perform
maintenance on the machine.
2023-03-18 05:23:12 +02:00
Valentin Radu
0ad140c47f Taskbar10: Disable tablet optimized taskbar feature
This fixes Task View and Win-Tab, Alt-Tab breaking after pressing
Win-Tab, flyouts alignment, notification center alignment, Windows key
shortcuts on OS builds 22621.1413+.

Thanks @CthRio for the heads up.
2023-03-18 00:45:13 +02:00
Valentin Radu
23a4190018 All: Infrastructure for reporting which OS features are enabled 2023-03-18 00:43:13 +02:00
Valentin Radu
4f3dab5a5c Version: 22621.1344.53.1 2023-03-01 21:29:35 +02:00
Valentin Radu
f9d702ebbf Taskbar10: Fixed a bug that crashed explorer on OS build 22621.1344
It seems Microsoft updated some stuff in `windowsudk.shellcommon.dll`
and has introduced a new interface (`ITaskbarSettings6`), from which
they call the `GetEffectiveSearchMode` method.

This is not relevant for the Windows 10 taskbar, but the code gets
called there anyway as well, so it had to be patched to support this
new interface.

This fix should address the issue and have ExplorerPatcher still work
on newer 22621-based OS builds, like 1344 ("Moment 2" update).
2023-03-01 21:19:33 +02:00
Valentin Radu
cc0af464c3 Weather: Fixed a bug that displayed the widget area using a different background color
It seems Google changed the widget and explicitly set a background
color, so this patch takes that into account and removes it.
Previously, the widget did not have a background, rather inheriting it
from the `body` element.
2023-03-01 20:41:09 +02:00
Valentin Radu
c083327e2f Weather: Fixed a bug that might throw a script error when certain elements are not ready 2023-03-01 20:39:05 +02:00
Valentin Radu
a8c7fbadaa Weather: Fixed a bug that could prevent the widget from properly loading
It seems that either the web page, either something in Microsoft's
WebView2 implementation changed so that when
`ICoreWebView2::NavigationCompleted` is fired, the elements of interest
on the page are not ready, which causes all this mess, as you can tell.

The solution for now was to delay the execution of my scripts, which
seemed to have gotten rid of the problem for now. I don't particularly
like the solution, I'd of course want something more robust, but I
guess these are the pitfalls when you do not control the entire
ecosystem...
2023-03-01 20:38:35 +02:00
Valentin Radu
ca8ce137d8 ep_extra: Implemented a loadable module for Windows 7's Alt-Tab
As a way to showcase the `ep_extra` loader I wrote in my
previous commit, I have developed this module which enables the use of
Windows 7's Alt-Tab. Mind you, the implementation might not be
complete and 100% ironed out, but rather a functional preview on what
it can look like.

Grab a copy of `AltTab.dll` from Windows 7 (I have tested against
version 6.1.7600.16385) and place it in `C:\Windows`. Then, reload
`explorer` with the `ep_extra` loader and this module in `C:\Windows`
and when you "alt-tab", you should see the Windows 7 window switcher.
This is not a reproduction - it executes the original code from
Microsoft's DLL, only slightly patched to work properly in with the
updated APIs in the newer Windows versions.
2023-03-01 20:35:31 +02:00
Valentin Radu
1f4b586f03 ep_extra: Implemented an ep_extra-based loader
ExplorerPatcher includes a mechanism which allows the user to load
a single DLL, named `ep_extra.dll` and placed in `C:\Windows` when
`explorer.exe` loads. This was long requested by some users who wanted
to perform their own code/initializations/hooks when `explorer` started.

In the mean time, I thought `ep_extra.dll` would work nicely as a
loader for any number of modules. I could've included this functionality
in the main ExplorerPatcher code, but I decided to take it up as a
challenge instead and offer a robust implementation without changing
ExplorerPatcher's main code.

Thus, when using this as `ep_extra.dll`, it will also load DLLs that
match the `ep_extra_*.dll` pattern. These DLLs must export a function
called `setup` which the loader will execute on the thread that is
loading the DLLs. Although not currently checked, return 0 from this
function if your initialization succeeded, or some error code when it
fails.

What's up with the assembly and shell codes and weird threads that I
create? Well, I realized I kind of did a mistake when coding
ExplorerPatcher, in that I should have loaded and executed `ep_extra`
on a seprate thread, not in `explorer`'s main thread. But since I said
I'd do this challenge without changing EP, this was my solution
towards having this `ep_extra` loader do its work, load the other DLLs,
if any, and then unload itself from memory safely and (almost) cleanly
without disturbing the main application right after it does its job.
This way, you can update it seamlessly when `explorer` is running,
which is much more convenient than having to kill `explorer`, replace
the DLL, and then manually reload `explorer`. I don't know if this is
the best way, but it is the way I thought about when realizing that I
cannot call `FreeLibrary` simply, since the "next line" (which would
have been a "return") is well outside of valid memory at that point.
`FreeLibraryAndExitThread` also can't be used since I do not want to
exit `explorer`'s main thread which the loader's function is called on.
2023-03-01 20:27:44 +02:00
Valentin Radu
7c4567ac79 Version: 22621.819.52.2 2022-11-17 16:09:16 +02:00
Valentin Radu
9f9d43e103 Start11: Fix "Disable recommended section" to work at 125% display scaling 2022-11-17 16:05:23 +02:00
Valentin Radu
02cb6e900c Version: 22621.819.52.1 2022-11-17 03:18:41 +02:00
Valentin Radu
451db3c5b6 Taskbar11: Option to use the stock taskbar context menu 2022-11-17 03:13:27 +02:00
Valentin Radu
53fad19901 Start: Better way to determine the monitor on which the Start menu will open 2022-11-17 02:52:31 +02:00
Valentin Radu
4212e357b7 Start11: Center menu on screen also works when taskbar is not at the bottom 2022-11-17 02:51:32 +02:00
Valentin Radu
d262c41850 Version: 22621.608.51.6 2022-11-17 01:32:06 +02:00
Valentin Radu
d7a038564b All: Protect against crashes caused by failure to patch the IAT
Impact: A recent bug report on the Mozilla Firefox issue tracker
(https://bugzilla.mozilla.org/show_bug.cgi?id=1798707) identifies a
crash in the Firefox browser caused by an invalid memory access
performed by ExplorerPatcher (https://crash-stats.mozilla.org/signature/?signature=explorerpatcher.amd64.dll%20|%20%3Cunknown%20in%20Windows.UI.FileExplorer.dll%3E%20|%20explorerpatcher.amd64.dll%20|%20RtlpFindEntry%20|%20RtlpAllocateHeap%20|%20RtlpAllocateHeapInternal%20|%20explorerpatcher.amd64.dll%20|%20RtlDosApplyFileIsolationRedirection_Ustr%20|%20LdrpApplyFileNameRed...&date=%3E%3D2022-11-02T20%3A44%3A00.000Z&date=%3C2022-11-16T20%3A44%3A00.000Z).
This might happen only when the "Register as shell extension: option
is used, and ExplorerPatcher is injected in other processes. Testing
was unable to reproduce the issue, but looking on the crash logs it
was determined that it likely happens in "VnPatchDelayIAT", where
the memory is patched regardless of whether the protection level
actually succeeded changing or not. The call is suspected to fail
when certain antivirus solutions are used, although a clear test case
with this scenario could not be determined.

Also, code review determined that a race condition might happen in both
"VnPatchIAT" and "VnPatchDelayIAT", where some other thread might
unload the module while the code works with it, attempting to patch the
requested function.

Description: The issue has been addressed by improved checks and
ensuring the module is not unloaded while the functions work with it.
The program only attempts to patch the memory if the previous
protection change call actually succeeded. Additionally, the module
reference count is increased prior to working with it when attempting
the patch, in order to prevent other threads from successfully
unloading it. The proposed changes should harden the code against
unexpected behavior and should address the crashes experienced when
the code runs in other processes, including Firefox.
2022-11-17 01:28:31 +02:00
Valentin Radu
6190fd2278 Version: 22621.608.51.5 2022-11-15 15:37:46 +02:00
Valentin Radu
2572a802db Start11: Respect "Layout" settings on 22621-based builds
Impact: The actual layout setting from Windows Settings -
Personalization - Start is ignored and the "Default" layout setting is
always used when displaying the Windows 11 Start menu on 22621-based
builds when the setting to disable the "Recommended" section is NOT
used in ExplorerPatcher Properties.

Description: The issue has been addressed by remembering the current/
previous setting and using that when setting the height of the pinned
area when the setting to disable the "Recommended" section is NOT
used in ExplorerPatcher Properties, as opposed to the previous behavior
where a hardcoded value was being used.
2022-11-15 15:35:33 +02:00
Valentin Radu
5048a4f76f Version: 22621.608.51.4 2022-11-13 00:00:54 +02:00
Valentin Radu
a6a88b1b87 Taskbar11: Fixed a bug that could crash explorer.exe when right clicking certain system tray icons on 22621-based builds
Impact: Failure to check against a NULL value and dereferencing it
leads to a crash in `explorer.exe` with fault offset 0xfc69 on
22621-based OS builds. This happens when right clicking certain system
tray icons, like "Epic Games Launcher" when using the Windows 11
taskbar.

Description: The issue has been addressed with improved checks: a check
against NULL values is performed before attempting to work with the
data the variables might point to.
2022-11-12 23:56:19 +02:00
Valentin Radu
3717aefdaa Version: 22621.608.51.3 2022-10-12 18:30:45 +03:00
Valentin Radu
5cef3b12c3 GUI: Expose sws scroll wheel behavior option 2022-10-12 18:30:04 +03:00
Valentin Radu
9d64a8c3a5 sws: Fixed a bug that could prevent correct reload of settings when entries were deleted from the registry 2022-10-12 18:29:44 +03:00
Valentin Radu
f285371de0 Version: 22621.608.51.2 2022-10-12 03:04:59 +03:00
Valentin Radu
d4cae8106b All: Fix taskbar cascade and tile windows options 2022-10-12 03:01:28 +03:00
Valentin Radu
3fe00cb138 sws: Support for changing selection in window list using the mouse wheel 2022-10-12 02:51:59 +03:00
Valentin Radu
e4e3c61ade Version: 22621.608.51.1 2022-10-03 00:19:27 +03:00
Valentin Radu
0833f513f8 General: Fix network and battery flyouts on OS build 22621 2022-10-02 23:52:37 +03:00
Valentin Radu
556aeb5aae General: Fixed a bug that could prevent the battery or network flyouts from showing 2022-10-02 23:29:59 +03:00
Valentin Radu
33d1518977 General: Support EP as version 22621
The final Windows 11 22H2 that is shipped to users is OS build 22621.

Unfortunately, I updated EP some time ago to 22622. This creates a
discrepancy between the OS build most people have and the version of
EP that they might have installed. Furthermore, I currently test EP
against OS build 22621. To make things more clear, the next version of
EP will have the major build number set to 22621.

In order to have EP use version 22621 from now on and not prevent
updating from EP 22622... to EP 22621..., I devised this workaround
where the EP version is kept to 22622 as far as updates are concerned,
but the internal build number of EP really is set to 22621.

In the future, if we want to bump the EP version to 22622 again, a
regular change in `version.h` will have the setup patcher insert a `!`
as the first character in the hash part of the version string, which
tells the updater to display that version as a real 22622 build and not
artificially decrement it to 22621.

The only slight inconvenience is that, when the next update arrives,
people will be told they upgrade to EP 22622... still, but they will
receive an EP 22621... build. But at least the updater will work and
detect the new version. From there on, subsequent versions will be
shown as 22621 in the notification if the build number for that version
really is 22621.
2022-10-02 23:03:16 +03:00
Valentin Radu
4cfc2cea4b Updates: Check remote hash against locally computed hash 2022-10-02 22:49:43 +03:00
Valentin Radu
50f3a709c0 Weather: Fixed wrong left padding of the widget
Google seems to have updated the web page and the widget is now 14px
more to the right, so the code had to account for this.
2022-09-28 14:36:27 +03:00
Valentin Radu
fe071230ab Version: 22622.450.50.3 2022-09-11 13:28:12 +03:00
Valentin Radu
35046cd91a ep_dwm: Fixed use-after-free bug; fixed incorrect check for running dwm instances 2022-09-11 13:27:59 +03:00
Valentin Radu
5de2eb0044 Version: 22622.450.50.2 2022-08-07 21:29:08 +03:00
Valentin Radu
d55f3c73c9 GUI: Support disabling File Explorer modern search bar in 32-bit applications 2022-08-07 20:44:55 +03:00
Valentin Radu
8fd7c46db0 All: Substitute RegDeleteKeyW with RegDeleteTreeW
This is because the keys should be deleted regardless of whether they
contain subkeys or not.
2022-08-07 20:40:36 +03:00