[NTOS] Fix MiFindInitializationCode (#751)
authorTimo Kreuzer <timo.kreuzer@reactos.org>
Tue, 21 Aug 2018 08:35:57 +0000 (10:35 +0200)
committerGitHub <noreply@github.com>
Tue, 21 Aug 2018 08:35:57 +0000 (10:35 +0200)
commitd5181e44dd10615fe6c3515e38ac11ff01b51ec6
tree27fe5293dc58b4aef171a448b6ffc569b494a258
parentb5aa79a66df7ce3ea6771b9582f68ef1ab5cdd2d
[NTOS] Fix MiFindInitializationCode (#751)

Short: The code was suffering from an off-by-one bug (inconsistency between inclusive end exclusive end address), which could lead to freeing one page above the initialization code. This led to freeing part of the kernel import section on x64. Now it is consistently using the aligned/exclusive end address.

Long:
* Initialization sections are freed both for the boot loaded images as well as for drivers that are loaded later. Obviously the second mechanism needs to be able to run at any time, so it is not initialization code itself. For some reason someone decided though, it would be a smart idea to implement the code twice, once for the boot loaded images, once for drivers and concluding that the former was initialization code itself and had to be freed.
* Since freeing the code that frees the initialization sections, while it is doing that is not possible, it uses a "smart trick", initially skipping that range, returning its start and end to the caller and have the caller free it afterwards.
* The code was using the end address in an inconsistent way, partly aligning it to the start of the following section, sometimes pointing to the last byte that should be freed. The function that freed each chunk was assuming the latter (i.e. that the end was included in the range) and thus freed the page that contained the end address. The end address for the range that was returned to the caller was aligned to the start of the next section, and the caller used it to free the range including the following page. On x64 this was the start of the import section of ntoskrnl. How that worked on x86 I don't even want to know.
ntoskrnl/mm/ARM3/sysldr.c