Skip to content

fix win 11 error 0x80070001#3105

Closed
Islam-Darwish wants to merge 12 commits intoventoy:masterfrom
Islam-Darwish:master
Closed

fix win 11 error 0x80070001#3105
Islam-Darwish wants to merge 12 commits intoventoy:masterfrom
Islam-Darwish:master

Conversation

@Islam-Darwish
Copy link
Copy Markdown

@Islam-Darwish Islam-Darwish commented Feb 9, 2025

change efi partition attribute from 0xC000000000000001ULL to 0x9000000000000000ULL

@ventoy
Copy link
Copy Markdown
Owner

ventoy commented Feb 19, 2025

Hi,
How did you find that this attribute will cause 0x80070001 error ?

@Islam-Darwish
Copy link
Copy Markdown
Author

you can get it if you tried to install window 11 24h2 on vmware with secure boot, gpt

@ventoy
Copy link
Copy Markdown
Owner

ventoy commented Feb 19, 2025

Hi,
I can not reproduce this issue in my computer. Can you provide more details?

  1. Which VMWare version did you use?
  2. Which ISO file did you test? Where to download it (the download link)?
  3. You add your USB stick to VMware as a harddrive or as a removeable device?

If possible can you give me some screenshot or video?

Thanks!

@Islam-Darwish
Copy link
Copy Markdown
Author

Islam-Darwish commented Feb 19, 2025

VMware® Workstation 17 Pro
17.6.2 build-24409262

official windows any language
https://www.microsoft.com/en-us/software-download/windows11

tested by usb stick & usb card reader

add usb stick as removable device

you just need to configure system like this

Screenshot 2025-02-19 131143

Screenshot 2025-02-19 131202

Screenshot 2025-02-19 131229

Screenshot 2025-02-19 131237

Screenshot 2025-02-19 131253

Screenshot 2025-02-19 131258

@Islam-Darwish
Copy link
Copy Markdown
Author

Islam-Darwish commented Feb 19, 2025

this issue happens even without secure boot

ventoy added a commit that referenced this pull request Feb 19, 2025
@ventoy
Copy link
Copy Markdown
Owner

ventoy commented Feb 19, 2025

Please try this CI release to see whether it fix the issue.
Just update to this CI release.
(It will change the VTOYEFI partition attribute to 0x8000000000000000 during upgrade)

https://github.com/ventoy/Ventoy/actions/runs/13416687507
https://ventoy.net/en/doc_github_ci.html

@Islam-Darwish
Copy link
Copy Markdown
Author

Islam-Darwish commented Feb 19, 2025

ventoy-windows

vmware test:

NTFS -> GPT -> No Secure -> OK
NTFS -> GPT -> Secure Boot -> OK

Good Job.

#3029
No grudges , i am “Ultimate USB” developer 😅

Liquid-Syn added a commit to Liquid-Syn/Ventoy that referenced this pull request Jul 4, 2025
* Add support for FreeBSD 14.0

* Fix VTOY_LINUX_REMOUNT option does not work with latest linux kernel version. (ventoy#2661  ventoy#2674)

* update tool

* 1.0.97 release

* Fix the VTOY_LINUX_REMOUNT option on latest linux kernel

* Fix the boot issue for latest KAOS.

* Fix arch iso boot issue (ventoy#2825 ventoy#2824)

* bugfix

* update

* Fix VTOY_LINUX_REMOUNT feature in RHEL 9.4+ (ventoy#2827)

* Support ext4 fs with checksum seed feature.

* Fix VTOY_LINUX_REMOUNT option for latest CentOS 9 Stream. (ventoy#2827)

* Skip trash directory by default. (ventoy#2831)

* Updated Hungarian translation (ventoy#2836)

* Added Vietnamese language support for GRUB and updated Vietnamese translation of the installer (ventoy#2362)

* Updated Vietnamese language

Added new translation for `STR_4KN_UNSUPPORTED` and retouched in some places

* Added Vietnamese language support for GRUB menu

* Removed vi_VN from helplist and menulist

* Correct some spelling and grammar in BuildVentoyFromSource.txt (ventoy#2491)

* fix typo in diskio.h (ventoy#2497)

fucntion -> function

* The gpt pointer is not initialized, and when offset < 0, it may result in freeing unallocated memory. (ventoy#2499)

Signed-off-by: yuxiaojun <[email protected]>
Co-authored-by: yuxiaojun <[email protected]>

* Update languages.json (ventoy#2600)

minor update for indonesian language

* Update German translation (ventoy#2612)

* Update tr_TR.txt (ventoy#2618)

corrected grammar mistakes, fixed typos

* Adding root user check to ExtendPersistentImg (as in CreatePersistentImg) (ventoy#2633)

* Adding root user check & Enhance help info

* Update root user check

* support zvol devices (ventoy#2678)

their partitions are exposed as "/dev/zdXXpY", handle accordingly.

Signed-off-by: Fabian Grünbichler <[email protected]>

* Minor Update on README.md for Consistency (ventoy#2687)

* Update Italian help/menu (ventoy#2752)

* Update ar_AR.json (ventoy#2755)

* Update ar_AR.json

* Update ar_AR.json

* Japanese translation Fixed. (ventoy#2808)

* 1.0.98 release

* Fix the UEFI boot issue for some computer.

* Fix the install issue for /dev/mdX device. (ventoy#2846)

* Fix the issue that VTOY_LINUX_REMOUNT option does not take effect in latest openSUSE. (ventoy#2551)

* Japanese Fix typo. (ventoy#2839)

* Japanese translation Fixed.

* Fix typo

Fix Typo.

* Fixed Italian language (ventoy#2840)

* Fix French language (ventoy#2850)

Co-authored-by: tamdaz <[email protected]>

* Update Russian translation (ventoy#2857)

* 1.0.99

* Update to latest Shim (Fix verifying shim SBAT data failed) ventoy#2947

* docker compose up

* docker file

* support eweOS ISO (ventoy#3068)

* Fix the boot issue of SystemRescue 11.02+ (ventoy#2958)  (ventoy#3058)

* loader/linux: Ensure the newc pathname is NULL-terminated (ventoy#3082)

Per "man 5 cpio", the namesize in the cpio header includes the trailing
NUL byte of the pathname and the pathname is followed by NUL bytes, but
the current implementation ignores the trailing NUL byte when making
the newc header. Although make_header() tries to pad the pathname string,
the padding won't happen when strlen(name) + sizeof(struct newc_head)
is a multiple of 4, and the non-NULL-terminated pathname may lead to
unexpected results.

Assume that a file is created with 'echo -n aaaa > /boot/test12' and
loaded by grub2:

    linux /boot/vmlinuz
    initrd newc:test12:/boot/test12 /boot/initrd

The initrd command eventually invoked grub_initrd_load() and sent
't''e''s''t''1''2' to make_header() to generate the header:

00000070  30 37 30 37 30 31 33 30  31 43 41 30 44 45 30 30  |070701301CA0DE00|
00000080  30 30 38 31 41 34 30 30  30 30 30 33 45 38 30 30  |0081A4000003E800|
00000090  30 30 30 30 36 34 30 30  30 30 30 30 30 31 36 33  |0000640000000163|
000000a0  37 36 45 34 35 32 30 30  30 30 30 30 30 34 30 30  |76E4520000000400|
000000b0  30 30 30 30 30 38 30 30  30 30 30 30 31 33 30 30  |0000080000001300|
000000c0  30 30 30 30 30 30 30 30  30 30 30 30 30 30 30 30  |0000000000000000|
000000d0  30 30 30 30 30 36 30 30  30 30 30 30 30 30 74 65  |00000600000000te|
                                                                  ^namesize
000000e0  73 74 31 32 61 61 61 61  30 37 30 37 30 31 30 30  |st12aaaa07070100|
                   ^^ end of the pathname

Since strlen("test12") + sizeof(struct newc_head) is 116 = 29 * 4,
make_header() didn't pad the pathname, and the file content followed
"test12" immediately. This violates the cpio format and may trigger such
error during linux boot:

    Initramfs unpacking failed: ZSTD-compressed data is trunc

To avoid the potential problems, this commit counts the trailing NUL byte
in when calling make_header() and adjusts the initrd size accordingly.

Now the header becomes

00000070  30 37 30 37 30 31 33 30  31 43 41 30 44 45 30 30  |070701301CA0DE00|
00000080  30 30 38 31 41 34 30 30  30 30 30 33 45 38 30 30  |0081A4000003E800|
00000090  30 30 30 30 36 34 30 30  30 30 30 30 30 31 36 33  |0000640000000163|
000000a0  37 36 45 34 35 32 30 30  30 30 30 30 30 34 30 30  |76E4520000000400|
000000b0  30 30 30 30 30 38 30 30  30 30 30 30 31 33 30 30  |0000080000001300|
000000c0  30 30 30 30 30 30 30 30  30 30 30 30 30 30 30 30  |0000000000000000|
000000d0  30 30 30 30 30 37 30 30  30 30 30 30 30 30 74 65  |00000700000000te|
                                                                  ^namesize
000000e0  73 74 31 32 00 00 00 00  61 61 61 61 30 37 30 37  |st12....aaaa0707|
                      ^^ end of the pathname

Besides the trailing NUL byte, make_header() pads 3 more NUL bytes, and
the user can safely read the pathname without a further check.

To conform to the cpio format, the headers for "TRAILER!!!" are also
adjusted to include the trailing NUL byte, not ignore it.


Reviewed-by: Daniel Kiper <[email protected]>

Signed-off-by: Gary Lin <[email protected]>
Co-authored-by: Gary Lin <[email protected]>

* 1.1.00 release

* 1.1.01

* github ai version update

* Fix the "Unsupported vtoy type unknown" error when boot a VDI file created by VirtualBox 7.x

* 1.1.02 release

* Fix Windows 11 error 0x80070001. (ventoy#3010  ventoy#3029  ventoy#3105)

* Fix the issue that SLES/OpenSUSE can not boot after install. (ventoy#3125)

* Fix the boot issue for pfSense 2.7.x (ventoy#2775)

* Fix the boot issue for latest TinyCore Linux.

* Update Linux GUI type condition

* Update Ventoy2Disk.exe

* 1.1.03

* Fix the issue introduced in LinuxGUI version. (ventoy#3128)

* 1.1.04 release

* support devuan in Legacy BIOS mode.

* Fix the resolution issue when boot Windows/WinPE in UEFI mode.

* 1.1.05

* Support TrueNAS Scale (Linux) distro. (ventoy#3069 ventoy#3137)

* Don't force to use max resolution for VMware/VirtualBox. (ventoy#3140)

* Only for VirtualBox.

* Fix the boot issue for TrueNAS Scale. (ventoy#3069)

* Add BLOB list file and some missing build desc.

* Update BLOB_List.md

---------

Signed-off-by: yuxiaojun <[email protected]>
Signed-off-by: Fabian Grünbichler <[email protected]>
Signed-off-by: Gary Lin <[email protected]>
Co-authored-by: longpanda <[email protected]>
Co-authored-by: Balázs Úr <[email protected]>
Co-authored-by: Hoang Nguyen <[email protected]>
Co-authored-by: SunderB <[email protected]>
Co-authored-by: Ikko Eltociear Ashimine <[email protected]>
Co-authored-by: gcf <[email protected]>
Co-authored-by: yuxiaojun <[email protected]>
Co-authored-by: Ida Bagus <[email protected]>
Co-authored-by: WebSnke <[email protected]>
Co-authored-by: Batuhan <[email protected]>
Co-authored-by: Zhymabek Roman <[email protected]>
Co-authored-by: Fabian-Gruenbichler <[email protected]>
Co-authored-by: Samuel Johnson <[email protected]>
Co-authored-by: bovirus <[email protected]>
Co-authored-by: jermanuts <[email protected]>
Co-authored-by: Re*Index. (ot_inc) <[email protected]>
Co-authored-by: Zohir Tamda <[email protected]>
Co-authored-by: tamdaz <[email protected]>
Co-authored-by: Andrei Stepanov <[email protected]>
Co-authored-by: Liangbin Lian <[email protected]>
Co-authored-by: Gary Lin <[email protected]>
Co-authored-by: longpanda <[email protected]>
Co-authored-by: longpanda <[email protected]>
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.

2 participants