Ros-diffs

ros-diffs@reactos.org

81390 discussions

[reactos] 01/01: [MSPAINT] Update Romanian (ro-RO) translation (#5564)

by Andrei Miloiu

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=9293388c6178461779b3f… commit 9293388c6178461779b3fddbbe524a147744a69d Author: Andrei Miloiu <miloiuandrei(a)gmail.com> AuthorDate: Fri Aug 18 13:04:31 2023 +0300 Commit: GitHub <noreply(a)github.com> CommitDate: Fri Aug 18 13:04:31 2023 +0300 [MSPAINT] Update Romanian (ro-RO) translation (#5564) --- base/applications/mspaint/lang/ro-RO.rc | 30 +++++++++++++++--------------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/base/applications/mspaint/lang/ro-RO.rc b/base/applications/mspaint/lang/ro-RO.rc index 516d15bc335..134acf54bc1 100644 --- a/base/applications/mspaint/lang/ro-RO.rc +++ b/base/applications/mspaint/lang/ro-RO.rc @@ -4,7 +4,7 @@ * PURPOSE: Romanian resource file * TRANSLATORS: Copyright 2009 Petru Dimitriu <petrimetri(a)gmail.com> * Copyright 2011-2018 Ștefan Fulea <stefan.fulea(a)mail.com> - * Copyright 2022 Andrei Miloiu <miloiuandrei(a)gmail.com> + * Copyright 2022-2023 Andrei Miloiu <miloiuandrei(a)gmail.com> */ LANGUAGE LANG_ROMANIAN, SUBLANG_NEUTRAL @@ -105,18 +105,18 @@ ID_POPUPMENU MENU BEGIN POPUP "Popup #0" BEGIN - MENUITEM "C&ut\tCtrl+X", IDM_EDITCUT - MENUITEM "Copy\tCtrl+C", IDM_EDITCOPY - MENUITEM "&Paste\tCtrl+V", IDM_EDITPASTE - MENUITEM "De&lete selection\tDel", IDM_EDITDELETESELECTION - MENUITEM "Select &All\tCtrl+A", IDM_EDITSELECTALL + MENUITEM "Dec&upează\tCtrl+X", IDM_EDITCUT + MENUITEM "Copia&ză\tCtrl+C", IDM_EDITCOPY + MENUITEM "&Lipește\tCtrl+V", IDM_EDITPASTE + MENUITEM "Șt&erge selecția\tDel", IDM_EDITDELETESELECTION + MENUITEM "Selectea&ză tot\tCtrl+A", IDM_EDITSELECTALL MENUITEM SEPARATOR - MENUITEM "C&opy to...", IDM_EDITCOPYTO - MENUITEM "Paste &from...", IDM_EDITPASTEFROM + MENUITEM "C&opiere în...", IDM_EDITCOPYTO + MENUITEM "Lipire &din...", IDM_EDITPASTEFROM MENUITEM SEPARATOR - MENUITEM "Rotate/&Mirror...\tCtrl+R", IDM_IMAGEROTATEMIRROR - MENUITEM "&Stretch/Skew...\tCtrl+W", IDM_IMAGESTRETCHSKEW - MENUITEM "&Invert colors\tCtrl+I", IDM_IMAGEINVERTCOLORS + MENUITEM "&Rotire/Oglindire...\tCtrl+R", IDM_IMAGEROTATEMIRROR + MENUITEM "Întindere/În&clinare...\tCtrl+W", IDM_IMAGESTRETCHSKEW + MENUITEM "&Inversează culorile\tCtrl+I", IDM_IMAGEINVERTCOLORS END END @@ -214,7 +214,7 @@ BEGIN END IDD_FONTS DIALOG 0, 0, 225, 25 -CAPTION "Fonts" +CAPTION "Fonturi" STYLE DS_CENTER | DS_MODALFRAME | WS_POPUPWINDOW | WS_CAPTION EXSTYLE WS_EX_TOOLWINDOW FONT 8, "MS Shell Dlg" @@ -263,7 +263,7 @@ BEGIN IDS_VERTICAL "Vertical" IDS_PRINTRES "%d x %d pixeli/cm" IDS_CANTPASTE "Nu a putut fi lipit din clipboard. Formatul de date este fie incorect, fie nesuportat." - IDS_SAVEERROR "Failed to save the bitmap to file:\n\n%s" - IDS_CANTSENDMAIL "Failed to send a mail." - IDS_LOSECOLOR "The color information will be lost in this operation. Are you sure to proceed?" + IDS_SAVEERROR "Eșec în păstrarea imaginii bitmap („hartă de biți”) în fișierul:\n\n%s" + IDS_CANTSENDMAIL "Eșec în trimiterea unei scrisori." + IDS_LOSECOLOR "În această operațiune informațiile legate de culoare vor fi pierdute. Sigur veți continua?" END

1 year, 10 months

[reactos] 01/01: [EXPLORER] Update Romanian (ro-RO) translation (#5558)

by Andrei Miloiu

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=e7112320424bb40d65dcd… commit e7112320424bb40d65dcd5ff6fba88ddcee2de19 Author: Andrei Miloiu <miloiuandrei(a)gmail.com> AuthorDate: Fri Aug 18 13:04:03 2023 +0300 Commit: GitHub <noreply(a)github.com> CommitDate: Fri Aug 18 13:04:03 2023 +0300 [EXPLORER] Update Romanian (ro-RO) translation (#5558) --- base/shell/explorer/lang/ro-RO.rc | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/base/shell/explorer/lang/ro-RO.rc b/base/shell/explorer/lang/ro-RO.rc index 7ad4423780e..c2d0f00b432 100644 --- a/base/shell/explorer/lang/ro-RO.rc +++ b/base/shell/explorer/lang/ro-RO.rc @@ -3,7 +3,7 @@ * LICENSE: LGPL-2.1-or-later (https://spdx.org/licenses/LGPL-2.1-or-later) * PURPOSE: Romanian resource file * TRANSLATORS: Copyright 2011-2018 Ștefan Fulea <stefan.fulea(a)mail.com> - * Copyright 2022 Andrei Miloiu <miloiuandrei(a)gmail.com> + * Copyright 2022-2023 Andrei Miloiu <miloiuandrei(a)gmail.com> */ LANGUAGE LANG_ROMANIAN, SUBLANG_NEUTRAL @@ -20,7 +20,7 @@ BEGIN MENUITEM "Aranjări &orizontale", ID_SHELL_CMD_TILE_WND_H MENUITEM "Aranjări &verticale", ID_SHELL_CMD_TILE_WND_V MENUITEM "Afișează &biroul", ID_SHELL_CMD_SHOW_DESKTOP - MENUITEM "Des&face", ID_SHELL_CMD_UNDO_ACTION + MENUITEM "Desfa&ce", ID_SHELL_CMD_UNDO_ACTION MENUITEM SEPARATOR MENUITEM "&Gestionar de activități", ID_SHELL_CMD_OPEN_TASKMGR MENUITEM SEPARATOR @@ -60,7 +60,7 @@ BEGIN MENUITEM "E&xecutare…", IDM_RUN, MFT_STRING, MFS_ENABLED MENUITEM "", 450, MFT_SEPARATOR, MFS_ENABLED MENUITEM "Sinc&ronizează", IDM_SYNCHRONIZE, MFT_STRING, MFS_ENABLED - MENUITEM "De&autentificare %s…", IDM_LOGOFF, MFT_STRING, MFS_ENABLED + MENUITEM "Dea&utentificare %s…", IDM_LOGOFF, MFT_STRING, MFS_ENABLED MENUITEM "De&conectare…", IDM_DISCONNECT, MFT_STRING, MFS_ENABLED MENUITEM "D&etașare calculator", IDM_UNDOCKCOMPUTER, MFT_STRING, MFS_ENABLED MENUITEM "Î&nchidere calculator…", IDM_SHUTDOWN, MFT_STRING, MFS_ENABLED @@ -75,11 +75,11 @@ BEGIN GROUPBOX "Aspectul barei de activități", IDC_STATIC, 6, 6, 240, 120 CONTROL "", IDC_TASKBARPROP_TASKBARBITMAP, "Static", SS_BITMAP | SS_SUNKEN | SS_CENTERIMAGE, 12, 18, 229, 24 AUTOCHECKBOX "Fi&xează bara de activități", IDC_TASKBARPROP_LOCK, 13, 45, 200, 10 - AUTOCHECKBOX "&Auto-ascunde bara de activități", IDC_TASKBARPROP_HIDE, 13, 58, 200, 10 - AUTOCHECKBOX "&Menține bara de activități deasupra celorlalte ferestre", IDC_TASKBARPROP_ONTOP, 13, 71, 200, 10 + AUTOCHECKBOX "Auto-ascun&de bara de activități", IDC_TASKBARPROP_HIDE, 13, 58, 200, 10 + AUTOCHECKBOX "Menține &bara de activități deasupra celorlalte ferestre", IDC_TASKBARPROP_ONTOP, 13, 71, 200, 10 AUTOCHECKBOX "&Grupează butoanele similare din bara de activități", IDC_TASKBARPROP_GROUP, 13, 84, 200, 10 AUTOCHECKBOX "Afișează bara „&Lansări rapide”", IDC_TASKBARPROP_SHOWQL, 13, 97, 200, 10 - AUTOCHECKBOX "U&se small icons", IDC_TASKBARPROP_SMALLICONS, 13, 110, 200, 10 + AUTOCHECKBOX "Folos&ește pictograme mici", IDC_TASKBARPROP_SMALLICONS, 13, 110, 240, 10 END IDD_TASKBARPROP_STARTMENU DIALOGEX 0, 0, 252, 218 @@ -106,11 +106,11 @@ BEGIN LTEXT "Puteți menține ordonată zona de notificare prin ascunderea pictogramelor neaccesate recent.", IDC_STATIC, 13, 48, 223, 16 AUTOCHECKBOX "Asc&unde pictogramele neaccesate", IDC_TASKBARPROP_HIDEICONS, 13, 70, 125, 10 PUSHBUTTON "&Particularizare…", IDC_TASKBARPROP_ICONCUST, 188, 70, 50, 14 - GROUPBOX "System icons", IDC_STATIC, 6, 99, 240, 56 - LTEXT "Select which system icons to always show.", IDC_STATIC, 13, 113, 223, 8 + GROUPBOX "Pictograme de sistem", IDC_STATIC, 6, 99, 240, 56 + LTEXT "Selectați ce pictograme de sistem urmează să fie afișate mereu.", IDC_STATIC, 13, 113, 240, 8 AUTOCHECKBOX "Afișează &ceas", IDC_TASKBARPROP_CLOCK, 13, 127, 80, 10 AUTOCHECKBOX "Afișează s&ecunde", IDC_TASKBARPROP_SECONDS, 120, 127, 80, 10, WS_DISABLED - AUTOCHECKBOX "&Desktop", IDC_TASKBARPROP_DESKTOP, 13, 140, 80, 10 + AUTOCHECKBOX "&Birou", IDC_TASKBARPROP_DESKTOP, 13, 140, 80, 10 END IDD_NOTIFICATIONS_CUSTOMIZE DIALOGEX 0, 0, 232, 240 @@ -164,7 +164,7 @@ END IDD_MODERNSTART_ADVANCED DIALOGEX 0, 0, 251, 198, 0 STYLE DS_SETFONT | WS_CAPTION | WS_CHILD | WS_DISABLED -CAPTION "Advanced" +CAPTION "Avansat" FONT 8, "MS Shell Dlg" BEGIN GROUPBOX "Opțiuni ale meniului „Pornire” ", IDC_STATIC, 7, 7, 238, 42

1 year, 10 months

[reactos] 01/01: [FREELDR] Convert ntfs.c to UTF-8 without BOM

by Serge Gautherie

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=aa2d2139be17a99f9e9c2… commit aa2d2139be17a99f9e9c2387934b4a43e75aa79f Author: Serge Gautherie <reactos-git_serge_171003(a)gautherie.fr> AuthorDate: Fri Aug 18 00:51:36 2023 +0200 Commit: Serge Gautherie <reactos-git_serge_171003(a)gautherie.fr> CommitDate: Fri Aug 18 00:51:36 2023 +0200 [FREELDR] Convert ntfs.c to UTF-8 without BOM Addendum to f0d73e0 (and ab3bd82). --- boot/freeldr/freeldr/lib/fs/ntfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/boot/freeldr/freeldr/lib/fs/ntfs.c b/boot/freeldr/freeldr/lib/fs/ntfs.c index 9361f526932..fb4db082cd8 100644 --- a/boot/freeldr/freeldr/lib/fs/ntfs.c +++ b/boot/freeldr/freeldr/lib/fs/ntfs.c @@ -1,7 +1,7 @@ /* * FreeLoader NTFS support * Copyright (C) 2004 Filip Navara <xnavara(a)volny.cz> - * Copyright (C) 2009-2010 Herv� Poussineau + * Copyright (C) 2009-2010 Hervé Poussineau * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by

1 year, 10 months

[reactos] 01/01: [MSAFD] Fix non-blocking sockets support for recv() (#5575) CORE-14486

by Julio Carchi

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=442f5dfab50da48700a20… commit 442f5dfab50da48700a20d4bdc6917e48105ad86 Author: Julio Carchi <juliocarchi(a)yahoo.com> AuthorDate: Thu Aug 17 16:13:13 2023 -0500 Commit: GitHub <noreply(a)github.com> CommitDate: Thu Aug 17 23:13:13 2023 +0200 [MSAFD] Fix non-blocking sockets support for recv() (#5575) CORE-14486 Currently ReactOS' winsock2 implementation lacks of non-blocking sockets support for recv() apicall, this causes that applications that make use of this feature can lead to unexpected behaviors, one of them is Nginx web server, which uses non-blocking sockets when serving pages through Https protocol. CORE-14486 It also brings us significantly closer in master head to running Firefox 52, Mypal 29.3.0 and New Moon 28 browser, where the latter allows to connect to mattermost from within ReactOS. In master head an additional reg file is needed to stop us from exporting specific NT6+ APIs, but in older releases all that should work out of the box with this brilliant patch. Co-authored-by: Julio Carchi Ruiz <julcar(a)informaticos.com> Co-authored-by: Stanislav Motylkov <x86corez(a)gmail.com> --- dll/win32/msafd/misc/dllmain.c | 32 ++++++++++++++++++-------------- dll/win32/msafd/misc/sndrcv.c | 7 +++++++ 2 files changed, 25 insertions(+), 14 deletions(-) diff --git a/dll/win32/msafd/misc/dllmain.c b/dll/win32/msafd/misc/dllmain.c index a4cded75c00..79623a142c0 100644 --- a/dll/win32/msafd/misc/dllmain.c +++ b/dll/win32/msafd/misc/dllmain.c @@ -1495,22 +1495,25 @@ WSPAccept( ListenReceiveData = (PAFD_RECEIVED_ACCEPT_DATA)ReceiveBuffer; /* If this is non-blocking, make sure there's something for us to accept */ - FD_ZERO(&ReadSet); - FD_SET(Socket->Handle, &ReadSet); - Timeout.tv_sec=0; - Timeout.tv_usec=0; - - if (WSPSelect(0, &ReadSet, NULL, NULL, &Timeout, lpErrno) == SOCKET_ERROR) + if (Socket->SharedData->NonBlocking) { - NtClose(SockEvent); - return SOCKET_ERROR; - } + FD_ZERO(&ReadSet); + FD_SET(Socket->Handle, &ReadSet); + Timeout.tv_sec=0; + Timeout.tv_usec=0; - if (ReadSet.fd_array[0] != Socket->Handle) - { - NtClose(SockEvent); - if (lpErrno) *lpErrno = WSAEWOULDBLOCK; - return SOCKET_ERROR; + if (WSPSelect(0, &ReadSet, NULL, NULL, &Timeout, lpErrno) == SOCKET_ERROR) + { + NtClose(SockEvent); + return SOCKET_ERROR; + } + + if (ReadSet.fd_array[0] != Socket->Handle) + { + NtClose(SockEvent); + if (lpErrno) *lpErrno = WSAEWOULDBLOCK; + return SOCKET_ERROR; + } } /* Send IOCTL */ @@ -1782,6 +1785,7 @@ WSPAccept( AcceptSocketInfo->SharedData->State = SocketConnected; AcceptSocketInfo->SharedData->ConnectTime = GetCurrentTimeInSeconds(); + AcceptSocketInfo->SharedData->NonBlocking = Socket->SharedData->NonBlocking; /* Return Address in SOCKADDR FORMAT */ if( SocketAddress ) diff --git a/dll/win32/msafd/misc/sndrcv.c b/dll/win32/msafd/misc/sndrcv.c index 07fe8b1f1c0..06b0afb1d51 100644 --- a/dll/win32/msafd/misc/sndrcv.c +++ b/dll/win32/msafd/misc/sndrcv.c @@ -291,6 +291,13 @@ WSPRecv(SOCKET Handle, NULL, 0); + /* Non-blocking sockets must wait until data is available */ + if (Status == STATUS_PENDING && Socket->SharedData->NonBlocking) + { + if (lpErrno) *lpErrno = WSAEWOULDBLOCK; + return SOCKET_ERROR; + } + /* Wait for completion of not overlapped */ if (Status == STATUS_PENDING && lpOverlapped == NULL) {

1 year, 10 months

[reactos] 01/01: [FREELDR] Convert source files to UTF-8 without BOM

by Hermès Bélusca-Maïto

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=ab3bd82928b29376a57c7… commit ab3bd82928b29376a57c70bfa76c51c737f6bcf0 Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Thu Aug 17 15:36:54 2023 +0200 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Thu Aug 17 15:36:54 2023 +0200 [FREELDR] Convert source files to UTF-8 without BOM --- boot/freeldr/freeldr/arch/i386/hwdisk.c | 2 +- boot/freeldr/freeldr/arch/i386/pc/pchw.c | 2 +- boot/freeldr/freeldr/arch/realmode/pxe.inc | 2 +- boot/freeldr/freeldr/disk/scsiport.c | 2 +- boot/freeldr/freeldr/include/arcemul.h | 2 +- boot/freeldr/freeldr/include/fs/pxe.h | 2 +- boot/freeldr/freeldr/include/ntoskrnl.h | 2 +- boot/freeldr/freeldr/lib/fs/fat.c | 2 +- boot/freeldr/freeldr/lib/fs/iso.c | 2 +- boot/freeldr/freeldr/lib/fs/pxe.c | 2 +- boot/freeldr/freeldr/lib/inifile/ini_init.c | 2 +- 11 files changed, 11 insertions(+), 11 deletions(-) diff --git a/boot/freeldr/freeldr/arch/i386/hwdisk.c b/boot/freeldr/freeldr/arch/i386/hwdisk.c index cf23c60ac21..0a7ef8600ae 100644 --- a/boot/freeldr/freeldr/arch/i386/hwdisk.c +++ b/boot/freeldr/freeldr/arch/i386/hwdisk.c @@ -2,7 +2,7 @@ * FreeLoader * * Copyright (C) 2003, 2004 Eric Kohl - * Copyright (C) 2009 Herv� Poussineau + * Copyright (C) 2009 Hervé Poussineau * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by diff --git a/boot/freeldr/freeldr/arch/i386/pc/pchw.c b/boot/freeldr/freeldr/arch/i386/pc/pchw.c index 1dfa46bd120..4e8249dd99f 100644 --- a/boot/freeldr/freeldr/arch/i386/pc/pchw.c +++ b/boot/freeldr/freeldr/arch/i386/pc/pchw.c @@ -2,7 +2,7 @@ * FreeLoader * * Copyright (C) 2003, 2004 Eric Kohl - * Copyright (C) 2009 Herv� Poussineau + * Copyright (C) 2009 Hervé Poussineau * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by diff --git a/boot/freeldr/freeldr/arch/realmode/pxe.inc b/boot/freeldr/freeldr/arch/realmode/pxe.inc index cb56439c980..523c40c6a6a 100644 --- a/boot/freeldr/freeldr/arch/realmode/pxe.inc +++ b/boot/freeldr/freeldr/arch/realmode/pxe.inc @@ -1,6 +1,6 @@ /* * FreeLoader - * Copyright (C) 2011 Herv� Poussineau + * Copyright (C) 2011 Hervé Poussineau * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by diff --git a/boot/freeldr/freeldr/disk/scsiport.c b/boot/freeldr/freeldr/disk/scsiport.c index 63d81c3dab8..633facc496c 100644 --- a/boot/freeldr/freeldr/disk/scsiport.c +++ b/boot/freeldr/freeldr/disk/scsiport.c @@ -3,7 +3,7 @@ * LICENSE: GPL - See COPYING in the top level directory * FILE: boot/freeldr/freeldr/disk/scsiport.c * PURPOSE: Interface for SCSI Emulation - * PROGRAMMERS: Herv� Poussineau <hpoussin(a)reactos.org> + * PROGRAMMERS: Hervé Poussineau <hpoussin(a)reactos.org> */ /* INCLUDES *******************************************************************/ diff --git a/boot/freeldr/freeldr/include/arcemul.h b/boot/freeldr/freeldr/include/arcemul.h index 437fecea28f..2c01ed71a3a 100644 --- a/boot/freeldr/freeldr/include/arcemul.h +++ b/boot/freeldr/freeldr/include/arcemul.h @@ -1,6 +1,6 @@ /* * FreeLoader - * Copyright (C) 2009 Herv� Poussineau <hpoussin(a)reactos.org> + * Copyright (C) 2009 Hervé Poussineau <hpoussin(a)reactos.org> * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by diff --git a/boot/freeldr/freeldr/include/fs/pxe.h b/boot/freeldr/freeldr/include/fs/pxe.h index 5d2cc2abc42..9ac80055505 100644 --- a/boot/freeldr/freeldr/include/fs/pxe.h +++ b/boot/freeldr/freeldr/include/fs/pxe.h @@ -1,6 +1,6 @@ /* * FreeLoader PXE support - * Copyright (C) 2011 Herv� Poussineau + * Copyright (C) 2011 Hervé Poussineau * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by diff --git a/boot/freeldr/freeldr/include/ntoskrnl.h b/boot/freeldr/freeldr/include/ntoskrnl.h index c88c5e55eba..393913d2be5 100644 --- a/boot/freeldr/freeldr/include/ntoskrnl.h +++ b/boot/freeldr/freeldr/include/ntoskrnl.h @@ -3,7 +3,7 @@ * LICENSE: GPL - See COPYING in the top level directory * FILE: boot/freeldr/freeldr/include/ntoskrnl.h * PURPOSE: NTOS glue routines for the MINIHAL library - * PROGRAMMERS: Herv� Poussineau <hpoussin(a)reactos.org> + * PROGRAMMERS: Hervé Poussineau <hpoussin(a)reactos.org> */ #include <ntdef.h> diff --git a/boot/freeldr/freeldr/lib/fs/fat.c b/boot/freeldr/freeldr/lib/fs/fat.c index a7495f17770..1b5ad239e1f 100644 --- a/boot/freeldr/freeldr/lib/fs/fat.c +++ b/boot/freeldr/freeldr/lib/fs/fat.c @@ -3,7 +3,7 @@ * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) * PURPOSE: FAT filesystem driver for FreeLoader * COPYRIGHT: Copyright 1998-2003 Brian Palmer (brianp(a)sginet.com) - * Copyright 2009 Herv� Poussineau + * Copyright 2009 Hervé Poussineau * Copyright 2019 Victor Perevertkin (victor.perevertkin(a)reactos.org) */ diff --git a/boot/freeldr/freeldr/lib/fs/iso.c b/boot/freeldr/freeldr/lib/fs/iso.c index ba7c57f38d4..54222b2a160 100644 --- a/boot/freeldr/freeldr/lib/fs/iso.c +++ b/boot/freeldr/freeldr/lib/fs/iso.c @@ -1,7 +1,7 @@ /* * FreeLoader * Copyright (C) 1998-2003 Brian Palmer <brianp(a)sginet.com> - * Copyright (C) 2009 Herv� Poussineau <hpoussin(a)reactos.org> + * Copyright (C) 2009 Hervé Poussineau <hpoussin(a)reactos.org> * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by diff --git a/boot/freeldr/freeldr/lib/fs/pxe.c b/boot/freeldr/freeldr/lib/fs/pxe.c index 87d06b75e2c..66145a3d328 100644 --- a/boot/freeldr/freeldr/lib/fs/pxe.c +++ b/boot/freeldr/freeldr/lib/fs/pxe.c @@ -1,6 +1,6 @@ /* * FreeLoader - * Copyright (C) 2011 Herv� Poussineau + * Copyright (C) 2011 Hervé Poussineau * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by diff --git a/boot/freeldr/freeldr/lib/inifile/ini_init.c b/boot/freeldr/freeldr/lib/inifile/ini_init.c index b63a578085a..9992b0b7151 100644 --- a/boot/freeldr/freeldr/lib/inifile/ini_init.c +++ b/boot/freeldr/freeldr/lib/inifile/ini_init.c @@ -1,6 +1,6 @@ /* * FreeLoader - * Copyright (C) 2009 Herv� Poussineau <hpoussin(a)reactos.org> + * Copyright (C) 2009 Hervé Poussineau <hpoussin(a)reactos.org> * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by

1 year, 10 months

[reactos] 01/01: [BOOTSECT][FREELDR] Support booting from NTFS partitions (#3416)

by Sylvain Deverre

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=f0d73e0f7fa97b17e636c… commit f0d73e0f7fa97b17e636c76653172ff2122f434f Author: Sylvain Deverre <deverre.sylv(a)gmail.com> AuthorDate: Thu Aug 17 15:07:59 2023 +0200 Commit: GitHub <noreply(a)github.com> CommitDate: Thu Aug 17 15:07:59 2023 +0200 [BOOTSECT][FREELDR] Support booting from NTFS partitions (#3416) Implement NTFS boot sector that loads FreeLdr from a NTFS partition. CORE-17474 - Able to find/parse root directory; - Handle fixups in FILE MFT record; - Implement directory tree search; - Implement loading found file from disk; - Handle fixups in INDX records; - Fail if compressed or sparse; - Attempt to support 64-bit disks. Some TO-DOs for later: - Handle "weird" NTFS partition with uncommon sector sizes / sectors per cluster / sectors per index record; - Better implementation for file loading; - 64-bit LCN support. --- boot/freeldr/bootsect/CMakeLists.txt | 2 + boot/freeldr/bootsect/ntfs.S | 893 +++++++++++++++++++++++++++++++++++ boot/freeldr/freeldr/lib/fs/ntfs.c | 2 +- 3 files changed, 896 insertions(+), 1 deletion(-) diff --git a/boot/freeldr/bootsect/CMakeLists.txt b/boot/freeldr/bootsect/CMakeLists.txt index d0c527779d5..a46c9066c6a 100644 --- a/boot/freeldr/bootsect/CMakeLists.txt +++ b/boot/freeldr/bootsect/CMakeLists.txt @@ -8,6 +8,7 @@ if(ARCH STREQUAL "i386" OR ARCH STREQUAL "amd64") CreateBootSectorTarget(fat ${CMAKE_CURRENT_SOURCE_DIR}/fat.S ${CMAKE_CURRENT_BINARY_DIR}/fat.bin 7c00) CreateBootSectorTarget(fat32 ${CMAKE_CURRENT_SOURCE_DIR}/fat32.S ${CMAKE_CURRENT_BINARY_DIR}/fat32.bin 7c00) + CreateBootSectorTarget(ntfsvbr ${CMAKE_CURRENT_SOURCE_DIR}/ntfs.S ${CMAKE_CURRENT_BINARY_DIR}/ntfs.bin 7c00) CreateBootSectorTarget(btrfsvbr ${CMAKE_CURRENT_SOURCE_DIR}/btrfs.S ${CMAKE_CURRENT_BINARY_DIR}/btrfs.bin 7c00) if(SARCH STREQUAL "pc98") @@ -34,6 +35,7 @@ if(ARCH STREQUAL "i386" OR ARCH STREQUAL "amd64") add_cd_file(TARGET btrfsvbr DESTINATION loader NO_CAB FILE ${CMAKE_CURRENT_BINARY_DIR}/btrfs.bin FOR bootcd regtest) add_cd_file(TARGET fat DESTINATION loader NO_CAB FILE ${CMAKE_CURRENT_BINARY_DIR}/fat.bin FOR bootcd regtest) add_cd_file(TARGET fat32 DESTINATION loader NO_CAB FILE ${CMAKE_CURRENT_BINARY_DIR}/fat32.bin FOR bootcd regtest) + add_cd_file(TARGET ntfsvbr DESTINATION loader NO_CAB FILE ${CMAKE_CURRENT_BINARY_DIR}/ntfs.bin FOR bootcd regtest) add_cd_file(TARGET isoboot DESTINATION loader NO_CAB NOT_IN_HYBRIDCD FILE ${CMAKE_CURRENT_BINARY_DIR}/isoboot.bin FOR all hybridcd) add_cd_file(TARGET isobtrt DESTINATION loader NO_CAB NOT_IN_HYBRIDCD FILE ${CMAKE_CURRENT_BINARY_DIR}/isobtrt.bin FOR bootcd regtest) elseif(ARCH STREQUAL "arm") diff --git a/boot/freeldr/bootsect/ntfs.S b/boot/freeldr/bootsect/ntfs.S new file mode 100644 index 00000000000..85081df464a --- /dev/null +++ b/boot/freeldr/bootsect/ntfs.S @@ -0,0 +1,893 @@ +/* + * PROJECT: ReactOS Bootsector + * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) + * FILE: boot/freeldr/bootsect/fat32.S + * PURPOSE: Implementing boot sector for NTFS + * COPYRIGHT: Copyright 2020 Sylvain Deverre (deverre.sylv(a)gmail.com) + */ + +/* INCLUDES ******************************************************************/ + +#include <asm.inc> +#include <freeldr/include/arch/pc/x86common.h> + +.code16 + +//ORG HEX(7c00) + +start: + jmp short main + nop +OEMName: + .ASCII "NTFS " // NTFS signature +BytesPerSector: + .word 512 +SectsPerCluster: + .byte 8 +ReservedSectors: + .word 0 // Unused by NTFS +NumberOfFats: + .byte 0 // Unused by NTFS +MaxRootEntries: + .word 0 // Unused by NTFS +TotalSectors: + .word 0 // Unused by NTFS +MediaDescriptor: + .byte HEX(0f8) +SectorsPerFat: + .word 0 // Unused by NTFS +SectorsPerTrack: + .word 0 // Should contain disk geometry +NumberOfHeads: + .word 0 // Should contain disk geometry +HiddenSectors: + .long 0 // Filled by format program +TotalSectorsBig: + .long 0 // Unused by NTFS +// NTFS inserted info +BootDrive: + .byte HEX(80) +CurrentHead: + .byte 0 +BootSignature: + .byte HEX(80) +Unused: + .byte 0 +VolumeSectorCount: + .quad 0 // Must be patched by format program ! +MftLocation: + .quad 0 // Must be patched by format program ! +MftMirrorLocation: + .quad 0 // Must be patched by format program ! +ClustersPerMftRecord: + .long 0 +ClustersPerIndexRecord: + .long 0 +VolumeSerialNumber: + .quad 0 +Checksum: + .long 0 + +main: + xor ax,ax // Setup segment registers + mov ds,ax // Make DS correct + mov es,ax // Make ES correct + mov ss,ax // Make SS correct + mov sp, HEX(7c00) + mov bp, sp // Setup a stack + + cmp byte ptr [BootDrive], HEX(0ff) // If they have specified a boot drive then use it + jne GetDriveParameters + + mov byte ptr [BootDrive], dl // Save the boot drive + +GetDriveParameters: + mov ah, 8 + mov dl, byte ptr [BootDrive] // Get boot drive in dl + int HEX(13) // Request drive parameters from the bios + jnc CalcDriveSize // If the call succeeded then calculate the drive size + + // If we get here then the call to the BIOS failed + // so just set CHS equal to the maximum addressable + // size + mov cx, HEX(0ffff) + mov dh, cl + +CalcDriveSize: + // Now that we have the drive geometry + // lets calculate the drive size + mov bl, ch // Put the low 8-bits of the cylinder count into BL + mov bh, cl // Put the high 2-bits in BH + shr bh, 6 // Shift them into position, now BX contains the cylinder count + and cl, HEX(3f) // Mask off cylinder bits from sector count + // CL now contains sectors per track and DH contains head count + movzx eax, dh // Move the heads into EAX + movzx ebx, bx // Move the cylinders into EBX + movzx ecx, cl // Move the sectors per track into ECX + inc eax // Make it one based because the bios returns it zero based + inc ebx // Make the cylinder count one based also + mul ecx // Multiply heads with the sectors per track, result in edx:eax + mul ebx // Multiply the cylinders with (heads * sectors) [stored in edx:eax already] + + // We now have the total number of sectors as reported + // by the bios in eax, so store it in our variable + mov dword ptr ds:[BiosCHSDriveSize], eax + +LoadExtraBootCode: + // First we have to load our extra boot code at + // next sector into memory at [0000:7e00h] + mov eax, HEX(1) + xor edx, edx + mov cx, 4 + xor bx, bx + mov es, bx // Read sector to [0000:7e00h] + mov bx, HEX(7e00) + call ReadSectors + jmp StartSearch + + +// Reads logical sectors into [ES:BX] +// EDX:EAX has logical sector number to read +// CX has number of sectors to read +ReadSectors: + push es + add eax, dword ptr [HiddenSectors] // Add offset from the disk beginning + test edx, edx + jnz ReadSectorsLBA + cmp eax, dword ptr ds:[BiosCHSDriveSize] // Check if they are reading a sector outside CHS range + jae ReadSectorsLBA // Yes - go to the LBA routine + // If at all possible we want to use LBA routines because + // They are optimized to read more than 1 sector per read + + pushad // Save logical sector number & sector count + +CheckInt13hExtensions: // Now check if this computer supports extended reads + mov ah, HEX(41) // AH = 41h + mov bx, HEX(55aa) // BX = 55AAh + mov dl, byte ptr [BootDrive] // DL = drive (80h-FFh) + int HEX(13) // IBM/MS INT 13 Extensions - INSTALLATION CHECK + jc ReadSectorsCHS // CF set on error (extensions not supported) + cmp bx, HEX(0aa55) // BX = AA55h if installed + jne ReadSectorsCHS + test cl,1 // CX = API subset support bitmap + jz ReadSectorsCHS // Bit 0, extended disk access functions (AH=42h-44h,47h,48h) supported + + popad // Restore sector count & logical sector number + +ReadSectorsLBA: + pushad // Save logical sector number & sector count + + cmp cx, 64 // Since the LBA calls only support 0x7F sectors at a time we will limit ourselves to 64 + jbe ReadSectorsSetupDiskAddressPacket // If we are reading less than 65 sectors then just do the read + mov cx, 64 // Otherwise read only 64 sectors on this loop iteration + +ReadSectorsSetupDiskAddressPacket: + mov word ptr ds:[LBASectorsRead],cx + push edx + push eax // Put 64-bit logical block address on stack + push es // Put transfer segment on stack + push bx // Put transfer offset on stack + push cx // Set transfer count + push 16 // Set size of packet to 10h + mov si, sp // Setup disk address packet on stack + + mov dl, byte ptr [BootDrive] // Drive number + mov ah, HEX(42) // Int 13h, AH = 42h - Extended Read + int HEX(13) // Call BIOS + jc PrintDiskError // If the read failed then abort + + add sp, 16 // Remove disk address packet from stack + + popad // Restore sector count & logical sector number + + push bx + mov ebx, dword ptr ds:[LBASectorsRead] + add eax, ebx // Increment sector to read + adc edx, 0 + shl ebx, 5 + mov dx, es + add dx, bx // Setup read buffer for next sector + mov es, dx + pop bx + + sub cx, word ptr ds:[LBASectorsRead] + jnz ReadSectorsLBA // Read next sector + + pop es + ret + +LBASectorsRead: + .long 0 + + +// Reads logical sectors into [ES:BX] +// EAX has logical sector number to read +// CX has number of sectors to read +ReadSectorsCHS: + popad // Get logical sector number & sector count off stack + +ReadSectorsCHSLoop: + pushad + xor edx, edx + movzx ecx, word ptr [SectorsPerTrack] + div ecx // Divide logical by SectorsPerTrack + inc dl // Sectors numbering starts at 1 not 0 + mov cl, dl // Sector in CL + mov edx, eax + shr edx, 16 + div word ptr [NumberOfHeads] // Divide logical by number of heads + mov dh, dl // Head in DH + mov dl, byte ptr [BootDrive] // Drive number in DL + mov ch, al // Cylinder in CX + ror ah, 2 // Low 8 bits of cylinder in CH, high 2 bits + // in CL shifted to bits 6 & 7 + or cl, ah // Or with sector number + mov ax, HEX(0201) + int HEX(13) // DISK - READ SECTORS INTO MEMORY + // AL = number of sectors to read, CH = track, CL = sector + // DH = head, DL = drive, ES:BX -> buffer to fill + // Return: CF set on error, AH = status (see AH=01h), AL = number of sectors read + + jc PrintDiskError // If the read failed then abort + + popad + + inc eax // Increment Sector to Read + + mov dx, es + add dx, 32 // Increment read buffer for next sector + mov es, dx + + loop ReadSectorsCHSLoop // Read next sector + + pop es + ret + +// Displays a disk error message +// And reboots +PrintDiskError: + mov si, offset msgDiskError // Bad boot disk message + call PutChars // Display it + + jmp Reboot + +// Displays a file system error message +// And reboots +PrintFileSystemError: + mov si, offset msgFileSystemError // FreeLdr not found message + call PutChars // Display it + +Reboot: + mov si, offset msgAnyKey // Press any key message + call PutChars // Display it + xor ax, ax + int HEX(16) // Wait for a keypress + int HEX(19) // Reboot + +PutChars: + lodsb + or al, al + jz short Done + mov ah, HEX(0e) + mov bx, 7 + int HEX(10) + jmp short PutChars +Done: + ret + +msgDiskError: + .ascii "Disk error", CR, LF, NUL +msgFileSystemError: + .ascii "File system error", CR, LF, NUL +msgAnyKey: + .ascii "Press any key to restart", CR, LF, NUL + +SectsPerMFT: + .word 0 + +MFTStartSector: + .quad 0 + +BiosCHSDriveSize: + .long 0 + +.org 509 // Pad to 509 bytes +BootPartition: + .byte 1 + +BootFlagSignature: + .word HEX(0aa55) // BootSector signature + +// End of Bootsector +// Next sector starts as sector 2, since boot sector is +// as a file under NTFS ($Boot, which has inode number 7) +// and takes the two first clusters of the volume (which means +// 16 sectors, checked on Windows and mkfs.ntfs from ntfsutils) + +#define ATTRIBUTE_DATA HEX(80) +#define ATTRIBUTE_INDEX_ROOT HEX(90) +#define ATTRIBUTE_INDEX_ALLOCATION HEX(A0) +#define FILE_MAGIC HEX(454c4946) +#define INDX_MAGIC HEX(58444e49) +#define NTFS_FILE_ROOT 5 +StartSearch: + // Compute MFT start sector + mov eax, dword ptr [MftLocation] + mov cl, byte ptr [SectsPerCluster] + movzx ecx, cx + mul ecx + mov dword ptr [MFTStartSector], eax + + // Compute size of MFT entry in sectors + xor ax, ax + mov al, byte ptr [ClustersPerMftRecord] + test al, al + js NegativeOffset + mov cx, word ptr [SectsPerCluster] + mul cx + mov word ptr [SectsPerMFT], cx + jmp SearchNext + NegativeOffset: + // ClustersPerMftRecord is negative, so we need to perform 1 << (-ClustersPerMftRecord) + // to get the number of bytes needed for a MFT entry + not al + inc al + mov cl, al + xor ax, ax + inc ax + shl ax, cl + + // But here want to store sectors, so we need to divide by BytesPerSector + xor dx, dx + mov cx, word ptr [BytesPerSector] + div cx + mov word ptr [SectsPerMFT], ax + + SearchNext: + mov bp, sp + sub sp, HEX(10) + + // Read Root Directory MFT into [2000:0] + mov ax, HEX(2000) + mov es, ax + xor bx, bx + xor edx, edx + mov eax, NTFS_FILE_ROOT + call ReadInode + + // Finds freeldr.sys into index root's B-Tree + // and return its MFT index + xor ax, ax + call ExploreIndexRoot + + // Read the MFT entry of freeldr.sys into [A00:0] + push eax + mov ax, HEX(A00) + mov es, ax + pop eax + call ReadInode + + xor ax, ax + mov ebx, HEX(30) + call FindAttributeHdr + mov si, ax + // Move to the attribute data + add si, word ptr es:[si + HEX(14)] + + // We don't support compressed, sparse or encrypted Freeldr + test dword ptr es:[si + HEX(38)], HEX(4a00) + jnz CompressedFreeldr + + // Compute size in clusters + mov eax, dword ptr es:[si + HEX(28)] + mov cl, byte ptr [SectsPerCluster] + xor edx, edx + div ecx + mov cx, word ptr [BytesPerSector] + movzx ecx, cx + xor edx, edx + div ecx + mov edx, eax + + push ax + mov si, offset msgLoading + call PutChars + pop ax + + xor ax, ax + mov ebx, HEX(80) + call FindAttributeHdr + mov si, ax + add ax, word ptr es:[si + HEX(20)] + xor ecx, ecx + xor bx, bx + push FREELDR_BASE / 16 + pop fs + FreeLdrLoad: + pushad + call ReadNonResidentAttribute + popad + mov bx, fs + add bx, HEX(100) + mov fs, bx + xor bx, bx + inc ecx + cmp ecx, edx + jbe FreeLdrLoad + + mov dl, byte ptr [BootDrive] + mov dh, byte ptr [BootPartition] + ljmp16 0, FREELDR_BASE + +// Error message if Freeldr is compressed, encrypted or sparse +CompressedFreeldr: + mov si, offset msgFreeldrCompressed + call PutChars + jmp Reboot + +// Finds Freeldr.sys into the directory tree and returns the +// inode index +// INPUT: +// - ES:[AX] address of the MFT record +// OUTPUT: +// - EDX:EAX MFT number of the found file +ExploreIndexRoot: + #define fileRecordBase 2 + #define fileRecord 4 + #define indexRootData 6 + #define allocationRunList 8 + + push bp + mov bp, sp + push bx + push si + push di + sub sp, HEX(10) + mov word ptr [bp - fileRecordBase], es + mov word ptr [bp - fileRecord], ax + + mov ebx, ATTRIBUTE_INDEX_ROOT + call FindAttributeHdr + test ax, ax + jz PrintFileSystemError // fail if no INDEX_ROOT + + mov si, ax + cmp byte ptr es:[si + 8], HEX(0) + jnz PrintFileSystemError // fail if attribute is non-resident + + add si, word ptr es:[si + HEX(14)] + cmp dword ptr es:[si], HEX(30) + jnz PrintFileSystemError // fail if FILE_NAME attribute isn't indexed + + mov word ptr [bp - indexRootData], si + mov ax, word ptr [bp - fileRecord] + + test byte ptr es:[si + HEX(0c)], 1 + jz ExploreNext // Skip index allocation lookup if we don't have children + + mov ebx, ATTRIBUTE_INDEX_ALLOCATION + call FindAttributeHdr + test ax, ax + jz PrintFileSystemError // No INDEX_ALLOCATION found + + mov si, ax + cmp byte ptr es:[si + 8], 1 + jnz PrintFileSystemError // Fail if attribute is resident (shouldn't be) + + add si, word ptr es:[si + HEX(20)] + mov word ptr [bp - allocationRunList], si // save run list + + ExploreNext: + mov si, word ptr [bp - indexRootData] + lea si, [si + 32] // get the first INDEX_ENTRY + + // Main search loop. We browse the B-Tree which contains directory entries + // SI contains the current index entry. + NodeCheck: + test word ptr es:[si + HEX(0C)], 2 + jnz NodeCheckLastNode + + mov cl, byte ptr es:[si + HEX(50)] + movzx cx, cl + lea si, [si + HEX(52)] + mov di, offset FreeLdr + call CompareWideStrInsensitive + lea si, [si - HEX(52)] + test ax, ax + jz RootIndexFound + test word ptr es:[si + HEX(0C)], 1 + jz ContinueSearch + + test ax, HEX(f000) + jnz LookupChildNode // if result < 0 then explore child node + + ContinueSearch: + add si, word ptr es:[si + 8] + jmp NodeCheck + + RootIndexFound: + mov eax, dword ptr es:[si] // We found root entry, return with its MFT number + mov dx, word ptr es:[si + 4] // Return high part into edx. + // We take only the first word, the high word contains the + // sequence number + movzx edx, dx + jmp ExitIndexTree + + NodeCheckLastNode: + test word ptr es:[si + HEX(0C)], 1 + jz PrintFreeldrError + + LookupChildNode: + // Take the right LCN at the end of the index record + add si, word ptr es:[si + 8] + mov ecx, dword ptr es:[si - 8] + + // Read the fixed up LCN + mov bx, word ptr [bp - allocationRunList] + mov ax, word ptr [bp - fileRecordBase] + mov es, ax + mov ax, HEX(9000) + mov fs, ax + mov ax, bx + xor bx, bx + call ReadINDX + + // Go again to the loop but with the child node list + mov ax, HEX(9000) + mov es, ax + mov si, 0 + add si, es:[si + HEX(18)] // Go to the first node + lea si, [si + HEX(18)] + jmp NodeCheck + + ExitIndexTree: + pop di + pop si + mov sp, bp + pop bp + ret + +// 64-bit multiplication +// EDX:EAX operand +// ECX operator +Multiply64: + push bp + mov bp, sp + sub sp, 12 + // Save the high part of the multiplication + mov dword ptr [bp - 4], edx + + // Multiply the low part and save the result + mul ecx + mov dword ptr[bp - 8], eax + mov dword ptr[bp - 12], edx + + // Multiply the high part and add the carry + mov eax, dword ptr [bp - 4] + mul ecx + add eax, dword ptr [bp - 12] + + // Format correctly the number + mov edx, eax + mov eax, dword ptr [bp - 8] + mov sp, bp + pop bp + ret + +// Compare case-insensitive strings +// [ES:SI] - the source file name +// [DS:DI] - the destination file name +// CX - compare length +CompareWideStrInsensitive: + push bx + push si + push di + movzx cx, cl + CmpLoop: + mov ax, word ptr es:[si] + cmp ax, 'a' + jl NoUpper + cmp ax, 'z' + jg NoUpper + sub ax, HEX(20) + NoUpper: + mov bx, word ptr ds:[di] + sub bx, ax + test bx, bx + jnz CompareFail + add si, 2 + add di, 2 + dec cx + jnz CmpLoop + CompareFail: + mov ax, bx + pop di + pop si + pop bx + ret + +// Reads a NTFS cluster +// INPUT: +// - EDX:EAX : cluster number to read +// OUTPUT: +// - ES:BX : address to read +ReadCluster: + push ecx + mov cl, byte ptr [SectsPerCluster] // Convert clusters to sectors + movzx ecx, cx + call Multiply64 + call ReadSectors + pop ecx + ret + +// Reads a MFT entry +// INPUT: +// - EDX:EAX : MFT inode +// OUTPUT: +// - ES:[BX] : address to read +ReadInode: + push ecx + push si + push di + + push edx + mov cx, word ptr [SectsPerMFT] // Get the correct number of sectors for the FILE entry + movzx ecx, cx + mul ecx + movzx eax, ax + add eax, dword ptr [MFTStartSector] // Add it to the start of the MFT + mov cx, word ptr [SectsPerMFT] + movzx ecx, cx + pop edx + call ReadSectors + + cmp dword ptr es:[bx], FILE_MAGIC // Ensure we get a valid FILE record + jnz PrintFileSystemError + + call ApplyFixups + + pop di + pop si + pop ecx + ret + +#define UpdateSequenceOffset 4 +#define UpdateSequenceLen 6 +// Apply fixups to INDX and FILE records +// INPUT: +// - ES:[BX] - pointer to the record to fixup +ApplyFixups: + push si + push di + mov si, bx + add si, word ptr es:[bx + UpdateSequenceOffset] + xor cx, cx + inc cx + FixupLoop: + cmp cx, word ptr es:[bx + UpdateSequenceLen] + jz EndFixupLoop + mov si, bx + add si, word ptr es:[bx + UpdateSequenceOffset] + mov ax, word ptr es:[si] // Get first fixup value + + mov di, cx + shl di, 9 + add di, bx + sub di, 2 + cmp ax, word ptr es:[di] // Check fixup value + jnz PrintFileSystemError // Fixup is corrupted, so print error + inc cx + add si, cx + mov ax, word ptr es:[si] // Apply fixup + mov word ptr es:[di], ax + + jmp FixupLoop + EndFixupLoop: + + pop di + pop si + ret + +// Reads a non-resident attribute +// INPUT: +// - ES:[AX] : Address of the data runs +// - ECX : LCN to read +// OUTPUT: +// - FS:[BX] : Address to write to +ReadNonResidentAttribute: + #define currentLCN 4 + #define offsetWrite 8 + #define startReadLCN HEX(0C) + push bp + mov bp, sp + sub sp, HEX(10) + push edx + push si + mov dword ptr [bp - currentLCN], 0 // Store the current LCN + mov word ptr [bp - offsetWrite], bx + mov dword ptr [bp - startReadLCN], ecx + mov si, ax + xor edx, edx + RunLoop: + mov al, byte ptr es:[si] + test al, al + jz NotFound + call UnpackRun + add dword ptr [bp - currentLCN], eax + cmp dword ptr [bp - startReadLCN], ecx + jb FoundRun + sub dword ptr [bp - startReadLCN], ecx // Decrement the cluster + jmp RunLoop + FoundRun: + mov ebx, dword ptr [bp - currentLCN] + mov ecx, dword ptr [bp - startReadLCN] + add ebx, ecx + + push es + mov ax, fs + mov es, ax + mov eax, ebx + mov bx, word ptr [bp - offsetWrite] + call ReadCluster + pop es + jmp RunSearchEnd + NotFound: + xor ax, ax + RunSearchEnd: + pop si + pop edx + mov sp, bp + pop bp + ret + +// Decodes a run in the runlist +// INPUT: +// - ES:[SI] : address of the run +// OUTPUT: +// - EAX : Unpacked LCN +// - ECX : Unpacked run length (in sectors) +// - SI : Next run in the run list +UnpackRun: + push bp + mov bp, sp + sub sp, HEX(10) + push ebx + + // Unpack run header + mov bl, byte ptr es:[si] + inc si + + mov bh, bl + shr bh, 4 + and bl, 7 + mov byte ptr [bp-2], bh // BH contains the LCN length + mov byte ptr [bp-1], bl // BL contains the number of cluster length + + mov al, bl + call UnpackLen + mov dword ptr [bp - 8], ebx + + mov al, byte ptr [bp-2] + call UnpackLen + mov cl, byte ptr es:[si-1] // Fixup sign if last byte is > 255 + test cl, HEX(80) + jz NoSign + not eax + add ebx, eax + + NoSign: + mov eax, ebx + mov ecx, dword ptr [bp - 8] + pop ebx + mov sp, bp + pop bp + ret + + +// Auxiliary function that unpacks n bytes in the memory +// INPUT: +// - AL : size to unpack (max 4 bytes) +// OUTPUT: +// - EAX : the mask used to unpack (for negative number fixup) +// - EBX : the unpacked number +// - SI : Next byte to read +UnpackLen: + push cx + movzx ax, al + + // Read the whole DWORD and then compute a mask to remove + // unneeded bytes to get correct size + xor ebx, ebx + mov ebx, dword ptr es:[si] + add si, ax + + cmp al, 4 + jnz UnpackLen_not4 + xor eax, eax + dec eax + jmp UnpackLen_mask + +UnpackLen_not4: + mov cl, al // Compute mask (2^(8*len) - 1) + shl cl, 3 + xor eax, eax + inc eax + shl eax, cl + dec eax + +UnpackLen_mask: + and ebx, eax // Apply mask + pop cx + ret + +// Reads an INDX sector and applies fixups +// INPUT: +// - ES:[AX] : Address of the data runs +// - ECX : LCN to read +// OUTPUT: +// - FS:[BX] : Address to write to +ReadINDX: + push es + push bx + call ReadNonResidentAttribute + test ax, ax + jz PrintFileSystemError + cmp dword ptr fs:[0], INDX_MAGIC + jnz PrintFileSystemError // jump if not valid + pop bx + + mov ax, fs + mov es, ax + call ApplyFixups + pop es + ret + +// Finds an attribute header into the MFT +// INPUT: +// - ES:[AX] : pointer to the MFT entry +// - EBX : type to find +// OUTPUT: +// - ES:[AX] : Pointer to the attribute header in the MFT entry +FindAttributeHdr: + push cx + push si + push edx + mov si, ax + mov cx, word ptr es:[si+HEX(14)] // Get offset attribute + add si, cx + FindAttributeHdrLoop: + mov edx, dword ptr es:[si] // Get attribute type + cmp edx, ebx + jz AttrFound + cmp edx, HEX(ffffffff) + jz AttrNotFound + add cx, word ptr es:[si + 4] // Add size of the attribute + add si, word ptr es:[si + 4] + jmp FindAttributeHdrLoop + + AttrNotFound: + // Attribute not found, reset the offset + xor cx, cx + AttrFound: + mov ax, cx + pop edx + pop si + pop cx + ret + +PrintFreeldrError: + mov si, offset msgFreeldr + call PutChars + jmp Reboot + +FreeLdr: + .word 'F', 'R', 'E', 'E', 'L', 'D', 'R', '.', 'S', 'Y', 'S' +msgFreeldr: + .ascii "FreeLdr not found, cannot boot", CR, LF, NUL +msgLoading: + .ascii "Loading FreeLoader...", CR, LF, NUL +msgFreeldrCompressed: + .ascii "freeldr.sys is a sparse, compressed or encrypted file, cannot boot", CR, LF, NUL +.endcode16 + +END diff --git a/boot/freeldr/freeldr/lib/fs/ntfs.c b/boot/freeldr/freeldr/lib/fs/ntfs.c index c7e41c23ef8..9361f526932 100644 --- a/boot/freeldr/freeldr/lib/fs/ntfs.c +++ b/boot/freeldr/freeldr/lib/fs/ntfs.c @@ -1,7 +1,7 @@ /* * FreeLoader NTFS support * Copyright (C) 2004 Filip Navara <xnavara(a)volny.cz> - * Copyright (C) 2009-2010 Herv� Poussineau + * Copyright (C) 2009-2010 Herv� Poussineau * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by

1 year, 10 months

[reactos] 01/01: [CMD][TESTS] Fix typo in "height" word (#5580)

by Mustafa Ateş Uzun

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=7d2f6b65b3bfe2d8597ca… commit 7d2f6b65b3bfe2d8597ca8d499aeb4f831911654 Author: Mustafa Ateş Uzun <mustafauzun0(a)gmail.com> AuthorDate: Thu Aug 17 14:30:26 2023 +0300 Commit: GitHub <noreply(a)github.com> CommitDate: Thu Aug 17 14:30:26 2023 +0300 [CMD][TESTS] Fix typo in "height" word (#5580) --- base/shell/cmd/lang/cs-CZ.rc | 4 ++-- base/shell/cmd/lang/el-GR.rc | 4 ++-- base/shell/cmd/lang/es-ES.rc | 2 +- base/shell/cmd/lang/id-ID.rc | 4 ++-- base/shell/cmd/lang/pl-PL.rc | 4 ++-- base/shell/cmd/lang/sk-SK.rc | 4 ++-- base/shell/cmd/lang/sq-AL.rc | 4 ++-- base/shell/cmd/lang/uk-UA.rc | 4 ++-- modules/rostests/tests/combotst/combotst.c | 4 ++-- modules/rostests/tests/edit/edittest.c | 4 ++-- 10 files changed, 19 insertions(+), 19 deletions(-) diff --git a/base/shell/cmd/lang/cs-CZ.rc b/base/shell/cmd/lang/cs-CZ.rc index 043cbcfca8a..2005df0a6c5 100644 --- a/base/shell/cmd/lang/cs-CZ.rc +++ b/base/shell/cmd/lang/cs-CZ.rc @@ -453,14 +453,14 @@ Updates are available at: https://reactos.org" STRING_VOL_HELP3 " Volume Serial Number is %04X-%04X\n" STRING_VOL_HELP4 "Displays the disk volume label and serial number, if they exist.\n\nVOL [drive:]" STRING_WINDOW_HELP1 "change console window aspect\n\n\ -WINDOW [/POS[=]left,top,width,heigth]\n\ +WINDOW [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['title']\n\n\ /POS specify window placement and dimensions\n\ MIN minimize the window\n\ MAX maximize the window\n\ RESTORE restore the window" STRING_WINDOW_HELP2 "change console window aspect\n\n\ -ACTIVATE 'window' [/POS[=]left,top,width,heigth]\n\ +ACTIVATE 'window' [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['title']\n\n\ window tile of window on which perform actions\n\ /POS specify window placement and dimensions\n\ diff --git a/base/shell/cmd/lang/el-GR.rc b/base/shell/cmd/lang/el-GR.rc index fc811d25dc6..a5fbf33c3fc 100644 --- a/base/shell/cmd/lang/el-GR.rc +++ b/base/shell/cmd/lang/el-GR.rc @@ -452,14 +452,14 @@ Updates are available at: https://reactos.org" STRING_VOL_HELP3 " Volume Serial Number is %04X-%04X\n" STRING_VOL_HELP4 "Displays the disk volume label and serial number, if they exist.\n\nVOL [drive:]" STRING_WINDOW_HELP1 "change console window aspect\n\n\ -WINDOW [/POS[=]left,top,width,heigth]\n\ +WINDOW [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['title']\n\n\ /POS specify window placement and dimensions\n\ MIN minimize the window\n\ MAX maximize the window\n\ RESTORE restore the window" STRING_WINDOW_HELP2 "change console window aspect\n\n\ -ACTIVATE 'window' [/POS[=]left,top,width,heigth]\n\ +ACTIVATE 'window' [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['title']\n\n\ window tile of window on which perform actions\n\ /POS specify window placement and dimensions\n\ diff --git a/base/shell/cmd/lang/es-ES.rc b/base/shell/cmd/lang/es-ES.rc index 3bae0a9794e..8d10735703d 100644 --- a/base/shell/cmd/lang/es-ES.rc +++ b/base/shell/cmd/lang/es-ES.rc @@ -463,7 +463,7 @@ MIN minimiza la ventana\n\ MAX maximiza la ventana\n\ RESTORE restaura la ventana" STRING_WINDOW_HELP2 "Cambia el aspecto de la ventana de la consola.\n\n\ -ACTIVATE 'window' [/POS[=]left,top,width,heigth]\n\ +ACTIVATE 'window' [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['title']\n\n\ window Título de la ventana en la que realizar las acciónes\n\ /POS Especifica la posición y dimensión de la ventana\n\ diff --git a/base/shell/cmd/lang/id-ID.rc b/base/shell/cmd/lang/id-ID.rc index 7623e6208af..2d129e3fadd 100644 --- a/base/shell/cmd/lang/id-ID.rc +++ b/base/shell/cmd/lang/id-ID.rc @@ -447,14 +447,14 @@ Pemutakiran tersedia di: https://reactos.org" STRING_VOL_HELP3 " Nomor Seri Volume adalah %04X-%04X\n" STRING_VOL_HELP4 "Menampilkan label dan nomor seri volume disk, jika ada.\n\nVOL [drive:]" STRING_WINDOW_HELP1 "mengubah aspek jendela konsol\n\n\ -WINDOW [/POS[=]left,top,width,heigth]\n\ +WINDOW [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['title']\n\n\ /POS menetapkan penempatan dan dimensi jendela\n\ MIN mengecilkan jendela\n\ MAX memaksimalkan jendela\n\ RESTORE mengembalikan jendela" STRING_WINDOW_HELP2 "mengubah aspek jendela konsol\n\n\ -ACTIVATE 'window' [/POS[=]left,top,width,heigth]\n\ +ACTIVATE 'window' [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['title']\n\n\ window tile jendela yang melakukan aksi\n\ /POS menetapkan penempatan dan dimensi jendela\n\ diff --git a/base/shell/cmd/lang/pl-PL.rc b/base/shell/cmd/lang/pl-PL.rc index 20361769681..4bbb420d202 100644 --- a/base/shell/cmd/lang/pl-PL.rc +++ b/base/shell/cmd/lang/pl-PL.rc @@ -453,14 +453,14 @@ Uaktualnienia dostępne na stronie: https://reactos.org" STRING_VOL_HELP3 " Numer seryjny woluminu to: %04X-%04X\n" STRING_VOL_HELP4 "Wyświetla etykietę woluminu w danym napędzie (o ile istnieje) i jego numer seryjny.\n\nVOL [napęd:]" STRING_WINDOW_HELP1 "Zmienia wymiar i pozycję danego okna konsoli\n\n\ -WINDOW [/POS[=]left,top,width,heigth]\n\ +WINDOW [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['tytuł']\n\n\ /POS określa położenie i rozmiary okna\n\ MIN minimalizuje okno\n\ MAX maksymalizuje okno\n\ RESTORE odtwarza standardowe ustawienia okna" STRING_WINDOW_HELP2 "Zmienia wymiar i pozycję określonego okna konsoli\n\n\ -ACTIVATE 'tytuł' [/POS[=]left,top,width,heigth]\n\ +ACTIVATE 'tytuł' [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['nowy_tytuł']\n\n\ window tytuł okna, na którym zostaną przeprowadzone zmiany\n\ /POS określa położenie i rozmiary okna\n\ diff --git a/base/shell/cmd/lang/sk-SK.rc b/base/shell/cmd/lang/sk-SK.rc index eccaa057f3b..75955a903db 100644 --- a/base/shell/cmd/lang/sk-SK.rc +++ b/base/shell/cmd/lang/sk-SK.rc @@ -454,14 +454,14 @@ Aktualizácie sú dostupné na: https://reactos.org" STRING_VOL_HELP3 " Volume Serial Number is %04X-%04X\n" STRING_VOL_HELP4 "Displays the disk volume label and serial number, if they exist.\n\nVOL [drive:]" STRING_WINDOW_HELP1 "change console window aspect\n\n\ -WINDOW [/POS[=]left,top,width,heigth]\n\ +WINDOW [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['title']\n\n\ /POS specify window placement and dimensions\n\ MIN minimalizovať okno\n\ MAX maximalizujte okno\n\ RESTORE obnoviť okno" STRING_WINDOW_HELP2 "change console window aspect\n\n\ -ACTIVATE 'window' [/POS[=]left,top,width,heigth]\n\ +ACTIVATE 'window' [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['title']\n\n\ window tile of window on which perform actions\n\ /POS specify window placement and dimensions\n\ diff --git a/base/shell/cmd/lang/sq-AL.rc b/base/shell/cmd/lang/sq-AL.rc index 6149ab9da4a..65a852dac54 100644 --- a/base/shell/cmd/lang/sq-AL.rc +++ b/base/shell/cmd/lang/sq-AL.rc @@ -452,14 +452,14 @@ Updates janë në dispozicion në: https://reactos.org" STRING_VOL_HELP3 " Numri Serial i Volumit është %04X-%04X\n" STRING_VOL_HELP4 "Shfaq etiketa e volumeve të diskut, nëse ekzistojne.\n\nVOL [drive:]" STRING_WINDOW_HELP1 "ndrysho aspektin e dritares se konsolit\n\n\ -WINDOW [/POS[=]left,top,width,heigth]\n\ +WINDOW [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['titulli']\n\n\ /POS specifikojë vendosje e dritares dhe dimensionet\n\ MIN minimizo dritaren\n\ MAX rrit dritaren\n\ RESTORE riktheni dritaren" STRING_WINDOW_HELP2 "ndrysho aspektin e dritares se konsolit\n\n\ -ACTIVATE 'window' [/POS[=]left,top,width,heigth]\n\ +ACTIVATE 'window' [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['title']\n\n\ window dritare ku mund te kryesh veprime\n\ /POS specifikojë vendosje e dritares dhe dimensionet\n\ diff --git a/base/shell/cmd/lang/uk-UA.rc b/base/shell/cmd/lang/uk-UA.rc index 62eb7db3de0..ece8cafbed2 100644 --- a/base/shell/cmd/lang/uk-UA.rc +++ b/base/shell/cmd/lang/uk-UA.rc @@ -458,14 +458,14 @@ VER [/C][/R][/W]\n\n\ STRING_VOL_HELP3 " Серiйний номер тому: %04X-%04X\n" STRING_VOL_HELP4 "Виводить мiтку тому та серiйний номер, якщо вони iснують.\n\nVOL [диск:]" STRING_WINDOW_HELP1 "змiнює вигляд вiкна консолi\n\n\ -WINDOW [/POS[=]left,top,width,heigth]\n\ +WINDOW [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['заголовок']\n\n\ /POS визначає розміри та розмiщення вiкна\n\ MIN згортає вiкно\n\ MAX розгортає вiкно\n\ RESTORE вiдновлює вiкно" STRING_WINDOW_HELP2 "змiнює вигляд вiкна консолi\n\n\ -ACTIVATE 'window' [/POS[=]left,top,width,heigth]\n\ +ACTIVATE 'window' [/POS[=]left,top,width,height]\n\ [MIN|MAX|RESTORE] ['title']\n\n\ window заголовок вiкна, вигляд якого треба змiнити\n\ /POS визначає розміри та розмiщення вiкна\n\ diff --git a/modules/rostests/tests/combotst/combotst.c b/modules/rostests/tests/combotst/combotst.c index aeed18d159a..592eeb716eb 100644 --- a/modules/rostests/tests/combotst/combotst.c +++ b/modules/rostests/tests/combotst/combotst.c @@ -388,8 +388,8 @@ WndProc ( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam ) Buttons[i++] = CreatePushButton("Width +",70,100,WS_VISIBLE); Buttons[i++] = CreatePushButton("Width -",70,101,WS_VISIBLE); - Buttons[i++] = CreatePushButton("Heigth +",70,102,WS_VISIBLE); - Buttons[i++] = CreatePushButton("Heigth -",70,103,WS_VISIBLE); + Buttons[i++] = CreatePushButton("Height +",70,102,WS_VISIBLE); + Buttons[i++] = CreatePushButton("Height -",70,103,WS_VISIBLE); Buttons[i++] = CreatePushButton("CreateWindowA",CHECKBUTWIDTH,CREATEWINDOW_ID,WS_VISIBLE); Buttons[i++] = CreatePushButton("CreateWindowExA",CHECKBUTWIDTH,CREATEWINDOWEX_ID,WS_VISIBLE); diff --git a/modules/rostests/tests/edit/edittest.c b/modules/rostests/tests/edit/edittest.c index 75a2e676fdd..086578fdab7 100644 --- a/modules/rostests/tests/edit/edittest.c +++ b/modules/rostests/tests/edit/edittest.c @@ -383,8 +383,8 @@ WndProc ( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam ) Buttons[i++] = CreatePushButton("Width +",70,100,WS_VISIBLE); Buttons[i++] = CreatePushButton("Width -",70,101,WS_VISIBLE); - Buttons[i++] = CreatePushButton("Heigth +",70,102,WS_VISIBLE); - Buttons[i++] = CreatePushButton("Heigth -",70,103,WS_VISIBLE); + Buttons[i++] = CreatePushButton("Height +",70,102,WS_VISIBLE); + Buttons[i++] = CreatePushButton("Height -",70,103,WS_VISIBLE); Buttons[i++] = CreatePushButton("CreateWindowA",140,CREATEWINDOW,WS_VISIBLE); Buttons[i++] = CreatePushButton("CreateWindowExA",140,CREATEWINDOWEX,WS_VISIBLE);

1 year, 10 months

[reactos] 01/01: [BROWSEUI] Refactor settings and sync them between windows (#5496)

by Carl J. Bialorucki

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=43b3dc20341682ffe1f5b… commit 43b3dc20341682ffe1f5b29e28b023f5843952b3 Author: Carl J. Bialorucki <cbialo2(a)outlook.com> AuthorDate: Thu Aug 17 03:14:41 2023 -0600 Commit: GitHub <noreply(a)github.com> CommitDate: Thu Aug 17 12:14:41 2023 +0300 [BROWSEUI] Refactor settings and sync them between windows (#5496) This works similarly to how our shell stores its settings from a software design standpoint. - Add settings.cpp file and ShellSettings structure to load and save settings. - Add a registry value to hivedef.inf for the locked toolbar state. This prevents a bug where the associated registry key cannot be opened or saved to. - Add new BWM_SETTINGCHANGE window message to refresh the UI on setting changes and send it to every open window when saving settings to the registry. - Add new BWM_GETSETTINGSPTR window message to share the shellbrowser settings structure pointer with child windows and toolbars. --- boot/bootdata/hivedef.inf | 1 + dll/win32/browseui/CMakeLists.txt | 1 + dll/win32/browseui/internettoolbar.cpp | 92 +++++++++++++++++----------------- dll/win32/browseui/internettoolbar.h | 4 ++ dll/win32/browseui/precomp.h | 14 ++++++ dll/win32/browseui/settings.cpp | 36 +++++++++++++ dll/win32/browseui/shellbrowser.cpp | 66 ++++++++++++++---------- 7 files changed, 141 insertions(+), 73 deletions(-) diff --git a/boot/bootdata/hivedef.inf b/boot/bootdata/hivedef.inf index 6394246fad9..6551f5a7701 100644 --- a/boot/bootdata/hivedef.inf +++ b/boot/bootdata/hivedef.inf @@ -1902,6 +1902,7 @@ HKCU,"SOFTWARE\Microsoft\Windows\CurrentVersion\Explorer\Advanced","ShowSuperHid HKCU,"SOFTWARE\Microsoft\Windows\CurrentVersion\Explorer\CabinetState",,0x00000012 HKCU,"SOFTWARE\Microsoft\Windows\CurrentVersion\Explorer\CabinetState","FullPath",0x00010003,0 HKCU,"SOFTWARE\Microsoft\Internet Explorer\Main","StatusBarOther",0x00010003,1 +HKCU,"SOFTWARE\Microsoft\Internet Explorer\Toolbar","Locked",0x00010003,1 ; ComDlg32 HKCU,"SOFTWARE\Microsoft\Windows\CurrentVersion\Explorer\ComDlg32",,0x00000012 diff --git a/dll/win32/browseui/CMakeLists.txt b/dll/win32/browseui/CMakeLists.txt index 8b9eb4ebd3d..83a78e085dc 100644 --- a/dll/win32/browseui/CMakeLists.txt +++ b/dll/win32/browseui/CMakeLists.txt @@ -30,6 +30,7 @@ list(APPEND SOURCE internettoolbar.cpp parsecmdline.cpp regtreeoptions.cpp + settings.cpp shellbrowser.cpp toolsband.cpp travellog.cpp diff --git a/dll/win32/browseui/internettoolbar.cpp b/dll/win32/browseui/internettoolbar.cpp index 8d51ad6bcfe..b9d3b6273b7 100644 --- a/dll/win32/browseui/internettoolbar.cpp +++ b/dll/win32/browseui/internettoolbar.cpp @@ -250,11 +250,11 @@ HRESULT CDockSite::GetRBBandInfo(REBARBANDINFOW &bandInfo) bandInfo.fStyle |= RBBS_BREAK; if (fDeskBandInfo.dwModeFlags & DBIMF_TOPALIGN) bandInfo.fStyle |= RBBS_TOPALIGN; - if (fFlags & ITF_NOGRIPPER || fToolbar->fLocked) + if (fFlags & ITF_NOGRIPPER || fToolbar->pSettings->fLocked) bandInfo.fStyle |= RBBS_NOGRIPPER; if (fFlags & ITF_NOTITLE) bandInfo.fStyle |= RBBS_HIDETITLE; - if (fFlags & ITF_GRIPPERALWAYS && !fToolbar->fLocked) + if (fFlags & ITF_GRIPPERALWAYS && !fToolbar->pSettings->fLocked) bandInfo.fStyle |= RBBS_GRIPPERALWAYS; if (fFlags & ITF_FIXEDSIZE) bandInfo.fStyle |= RBBS_FIXEDSIZE; @@ -607,17 +607,13 @@ HRESULT STDMETHODCALLTYPE CMenuCallback::CallbackSM(LPSMDATA psmd, UINT uMsg, WP CInternetToolbar::CInternetToolbar() { - fLocked = SHRegGetBoolUSValueW(L"Software\\Microsoft\\Internet Explorer\\Toolbar", - L"Locked", - FALSE, - TRUE); - fMainReBar = NULL; fMenuBandWindow = NULL; fNavigationWindow = NULL; fMenuCallback = new CComObject<CMenuCallback>(); fToolbarWindow = NULL; fAdviseCookie = 0; + pSettings = NULL; } CInternetToolbar::~CInternetToolbar() @@ -714,45 +710,40 @@ HRESULT CInternetToolbar::CreateMenuBar(IShellMenu **pMenuBar) } HRESULT CInternetToolbar::LockUnlockToolbars(bool locked) +{ + if (locked != pSettings->fLocked) + { + pSettings->fLocked = locked; + pSettings->Save(); + RefreshLockedToolbarState(); + } + return S_OK; +} + +void CInternetToolbar::RefreshLockedToolbarState() { REBARBANDINFOW rebarBandInfo; int bandCount; CDockSite *dockSite; - HRESULT hResult; - if (locked != fLocked) + rebarBandInfo.cbSize = sizeof(rebarBandInfo); + rebarBandInfo.fMask = RBBIM_STYLE | RBBIM_LPARAM; + bandCount = (int)SendMessage(fMainReBar, RB_GETBANDCOUNT, 0, 0); + for (INT x = 0; x < bandCount; x++) { - DWORD dwLocked = locked ? 1 : 0; - SHRegSetUSValueW(L"Software\\Microsoft\\Internet Explorer\\Toolbar", - L"Locked", - REG_DWORD, - &dwLocked, - sizeof(dwLocked), - SHREGSET_FORCE_HKCU); - fLocked = locked; - - rebarBandInfo.cbSize = sizeof(rebarBandInfo); - rebarBandInfo.fMask = RBBIM_STYLE | RBBIM_LPARAM; - bandCount = (int)SendMessage(fMainReBar, RB_GETBANDCOUNT, 0, 0); - for (INT x = 0; x < bandCount; x++) + SendMessage(fMainReBar, RB_GETBANDINFOW, x, (LPARAM)&rebarBandInfo); + dockSite = reinterpret_cast<CDockSite *>(rebarBandInfo.lParam); + if (dockSite != NULL) { - SendMessage(fMainReBar, RB_GETBANDINFOW, x, (LPARAM)&rebarBandInfo); - dockSite = reinterpret_cast<CDockSite *>(rebarBandInfo.lParam); - if (dockSite != NULL) - { - rebarBandInfo.fStyle &= ~(RBBS_NOGRIPPER | RBBS_GRIPPERALWAYS); - if (dockSite->fFlags & CDockSite::ITF_NOGRIPPER || fLocked) - rebarBandInfo.fStyle |= RBBS_NOGRIPPER; - if (dockSite->fFlags & CDockSite::ITF_GRIPPERALWAYS && !fLocked) - rebarBandInfo.fStyle |= RBBS_GRIPPERALWAYS; - SendMessage(fMainReBar, RB_SETBANDINFOW, x, (LPARAM)&rebarBandInfo); - } + rebarBandInfo.fStyle &= ~(RBBS_NOGRIPPER | RBBS_GRIPPERALWAYS); + if (dockSite->fFlags & CDockSite::ITF_NOGRIPPER || pSettings->fLocked) + rebarBandInfo.fStyle |= RBBS_NOGRIPPER; + if (dockSite->fFlags & CDockSite::ITF_GRIPPERALWAYS && !pSettings->fLocked) + rebarBandInfo.fStyle |= RBBS_GRIPPERALWAYS; + SendMessage(fMainReBar, RB_SETBANDINFOW, x, (LPARAM)&rebarBandInfo); } - hResult = ReserveBorderSpace(0); - - // TODO: refresh view menu? } - return S_OK; + ReserveBorderSpace(0); } HRESULT CInternetToolbar::SetState(const GUID *pguidCmdGroup, long commandID, OLECMD* pcmd) @@ -980,7 +971,7 @@ HRESULT STDMETHODCALLTYPE CInternetToolbar::ResizeBorderDW(LPCRECT prcBorder, ::GetWindowRect(fMainReBar, &availableBorderSpace); neededBorderSpace.left = 0; neededBorderSpace.top = availableBorderSpace.bottom - availableBorderSpace.top; - if (!fLocked) + if (!pSettings->fLocked) neededBorderSpace.top += 3; neededBorderSpace.right = 0; neededBorderSpace.bottom = 0; @@ -1145,7 +1136,7 @@ HRESULT STDMETHODCALLTYPE CInternetToolbar::QueryStatus(const GUID *pguidCmdGrou break; case ITID_TOOLBARLOCKED: // lock toolbars prgCmds->cmdf = OLECMDF_SUPPORTED | OLECMDF_ENABLED; - if (fLocked) + if (pSettings->fLocked) prgCmds->cmdf |= OLECMDF_LATCHED; break; default: @@ -1184,7 +1175,7 @@ HRESULT STDMETHODCALLTYPE CInternetToolbar::Exec(const GUID *pguidCmdGroup, DWOR // run customize return S_OK; case ITID_TOOLBARLOCKED: - return LockUnlockToolbars(!fLocked); + return LockUnlockToolbars(!pSettings->fLocked); } } return E_FAIL; @@ -1356,6 +1347,9 @@ HRESULT STDMETHODCALLTYPE CInternetToolbar::SetSite(IUnknown *pUnkSite) if (ownerWindow == NULL) return E_FAIL; + // Get settings from owner window + ::SendMessageW(ownerWindow, BWM_GETSETTINGSPTR, 0, (LPARAM)&pSettings); + // create dock container fSite = pUnkSite; dockContainer = SHCreateWorkerWindowW(0, ownerWindow, 0, @@ -1682,7 +1676,6 @@ LRESULT CInternetToolbar::OnContextMenu(UINT uMsg, WPARAM wParam, LPARAM lParam, RBHITTESTINFO hitTestInfo; REBARBANDINFOW rebarBandInfo; int bandID; - BOOL goButtonChecked; clickLocation.x = LOWORD(lParam); clickLocation.y = HIWORD(lParam); @@ -1727,9 +1720,8 @@ LRESULT CInternetToolbar::OnContextMenu(UINT uMsg, WPARAM wParam, LPARAM lParam, SHCheckMenuItem(contextMenu, IDM_TOOLBARS_ADDRESSBAR, IsBandVisible(ITBBID_ADDRESSBAND) == S_OK); SHCheckMenuItem(contextMenu, IDM_TOOLBARS_LINKSBAR, FALSE); SHCheckMenuItem(contextMenu, IDM_TOOLBARS_CUSTOMIZE, FALSE); - SHCheckMenuItem(contextMenu, IDM_TOOLBARS_LOCKTOOLBARS, fLocked); - goButtonChecked = SHRegGetBoolUSValueW(L"Software\\Microsoft\\Internet Explorer\\Main", L"ShowGoButton", FALSE, TRUE); - SHCheckMenuItem(contextMenu, IDM_TOOLBARS_GOBUTTON, goButtonChecked); + SHCheckMenuItem(contextMenu, IDM_TOOLBARS_LOCKTOOLBARS, pSettings->fLocked); + SHCheckMenuItem(contextMenu, IDM_TOOLBARS_GOBUTTON, pSettings->fShowGoButton); // TODO: use GetSystemMetrics(SM_MENUDROPALIGNMENT) to determine menu alignment command = TrackPopupMenu(contextMenu, TPM_LEFTALIGN | TPM_TOPALIGN | TPM_RIGHTBUTTON | TPM_RETURNCMD, @@ -1745,7 +1737,7 @@ LRESULT CInternetToolbar::OnContextMenu(UINT uMsg, WPARAM wParam, LPARAM lParam, case IDM_TOOLBARS_LINKSBAR: // links break; case IDM_TOOLBARS_LOCKTOOLBARS: // lock the toolbars - LockUnlockToolbars(!fLocked); + LockUnlockToolbars(!pSettings->fLocked); break; case IDM_TOOLBARS_CUSTOMIZE: // customize SendMessage(fToolbarWindow, TB_CUSTOMIZE, 0, 0); @@ -1850,7 +1842,7 @@ LRESULT CInternetToolbar::OnNotify(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL LRESULT CInternetToolbar::OnLDown(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL &bHandled) { bHandled = FALSE; - if (fLocked) + if (pSettings->fLocked) return 0; if (wParam & MK_CONTROL) @@ -1928,3 +1920,11 @@ LRESULT CInternetToolbar::OnWinIniChange(UINT uMsg, WPARAM wParam, LPARAM lParam return lres; } + +LRESULT CInternetToolbar::OnSettingsChange(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) +{ + /* Refresh toolbar locked state */ + RefreshLockedToolbarState(); + + return 0; +} diff --git a/dll/win32/browseui/internettoolbar.h b/dll/win32/browseui/internettoolbar.h index e824df72b19..0b154ab07b0 100644 --- a/dll/win32/browseui/internettoolbar.h +++ b/dll/win32/browseui/internettoolbar.h @@ -92,6 +92,7 @@ public: BOOL fSizing; POINT fStartPosition; LONG fStartHeight; + ShellSettings *pSettings; public: CInternetToolbar(); virtual ~CInternetToolbar(); @@ -105,6 +106,7 @@ public: HRESULT IsBandVisible(int BandID); HRESULT ToggleBandVisibility(int BandID); HRESULT SetState(const GUID *pguidCmdGroup, long commandID, OLECMD* pcmd); + void RefreshLockedToolbarState(); public: // *** IInputObject specific methods *** @@ -200,6 +202,7 @@ public: LRESULT OnLUp(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL &bHandled); LRESULT OnMouseMove(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL &bHandled); LRESULT OnWinIniChange(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL &bHandled); + LRESULT OnSettingsChange(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL &bHandled); BEGIN_MSG_MAP(CInternetToolbar) COMMAND_ID_HANDLER(IDM_GOTO_BACK, OnTravelBack) @@ -221,6 +224,7 @@ public: MESSAGE_HANDLER(WM_LBUTTONUP, OnLUp) MESSAGE_HANDLER(WM_MOUSEMOVE, OnMouseMove) MESSAGE_HANDLER(WM_WININICHANGE, OnWinIniChange) + MESSAGE_HANDLER(BWM_SETTINGCHANGE, OnSettingsChange) END_MSG_MAP() DECLARE_REGISTRY_RESOURCEID(IDR_INTERNETTOOLBAR) diff --git a/dll/win32/browseui/precomp.h b/dll/win32/browseui/precomp.h index 67fe4c33208..c3c26e7115d 100644 --- a/dll/win32/browseui/precomp.h +++ b/dll/win32/browseui/precomp.h @@ -37,6 +37,10 @@ #include "resource.h" +#define BWM_SETTINGCHANGE (WM_USER + 300) +#define BWM_GETSETTINGSPTR (WM_USER + 301) +struct ShellSettings; + #include "ACLCustomMRU.h" #include "aclhistory.h" #include "aclistisf.h" @@ -65,4 +69,14 @@ WINE_DEFAULT_DEBUG_CHANNEL(browseui); +struct ShellSettings +{ + BOOL fLocked = FALSE; + BOOL fShowGoButton = FALSE; + BOOL fStatusBarVisible = FALSE; + + BOOL Save(); + BOOL Load(); +}; + #endif /* _BROWSEUI_PCH_ */ diff --git a/dll/win32/browseui/settings.cpp b/dll/win32/browseui/settings.cpp new file mode 100644 index 00000000000..de7e6c04480 --- /dev/null +++ b/dll/win32/browseui/settings.cpp @@ -0,0 +1,36 @@ +/* + * PROJECT: ReactOS browseui + * LICENSE: LGPL-2.1-or-later (https://spdx.org/licenses/LGPL-2.1-or-later) + * PURPOSE: Stores settings for the file explorer UI. + * COPYRIGHT: Copyright 2023 Carl Bialorucki <cbialo2(a)outlook.com> + */ + +#include "precomp.h" + +BOOL ShellSettings::Save() +{ + SHRegSetUSValueW(L"Software\\Microsoft\\Internet Explorer\\Main", L"StatusBarOther", + REG_DWORD, &fStatusBarVisible, sizeof(fStatusBarVisible), SHREGSET_FORCE_HKCU); + + SHRegSetUSValueW(L"Software\\Microsoft\\Internet Explorer\\Main", L"ShowGoButton", + REG_DWORD, &fShowGoButton, sizeof(fShowGoButton), SHREGSET_FORCE_HKCU); + + SHRegSetUSValueW(L"Software\\Microsoft\\Internet Explorer\\Toolbar", L"Locked", + REG_DWORD, &fLocked, sizeof(fLocked), SHREGSET_FORCE_HKCU); + + return TRUE; +} + +BOOL ShellSettings::Load() +{ + fStatusBarVisible = SHRegGetBoolUSValueW(L"Software\\Microsoft\\Internet Explorer\\Main", + L"StatusBarOther", FALSE, FALSE); + + fShowGoButton = SHRegGetBoolUSValueW(L"Software\\Microsoft\\Internet Explorer\\Main", + L"ShowGoButton", FALSE, TRUE); + + fLocked = SHRegGetBoolUSValueW(L"Software\\Microsoft\\Internet Explorer\\Toolbar", + L"Locked", FALSE, TRUE); + + return TRUE; +} diff --git a/dll/win32/browseui/shellbrowser.cpp b/dll/win32/browseui/shellbrowser.cpp index 0c808c29e25..0f1e3814ebe 100644 --- a/dll/win32/browseui/shellbrowser.cpp +++ b/dll/win32/browseui/shellbrowser.cpp @@ -292,7 +292,6 @@ private: CComPtr<IShellView> fCurrentShellView; // LPITEMIDLIST fCurrentDirectoryPIDL; // HWND fStatusBar; - bool fStatusBarVisible; CToolbarProxy fToolbarProxy; barInfo fClientBars[3]; CComPtr<ITravelLog> fTravelLog; @@ -306,6 +305,7 @@ private: IBindCtx *fHistoryBindContext; HDSA menuDsa; HACCEL m_hAccel; + ShellSettings m_settings; public: #if 0 ULONG InternalAddRef() @@ -350,7 +350,7 @@ public: HRESULT UpdateUpState(); void UpdateGotoMenu(HMENU theMenu); void UpdateViewMenu(HMENU theMenu); - void LoadSettings(); + void LoadCabinetState(); /* // *** IDockingWindowFrame methods *** virtual HRESULT STDMETHODCALLTYPE AddToolbar(IUnknown *punkSrc, LPCWSTR pwszItem, DWORD dwAddFlags); @@ -624,6 +624,8 @@ public: LRESULT OnRefresh(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled); LRESULT OnExplorerBar(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled); LRESULT RelayCommands(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL &bHandled); + LRESULT OnSettingsChange(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); + LRESULT OnGetSettingsPtr(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); HRESULT OnSearch(); static ATL::CWndClassInfo& GetWndClassInfo() @@ -674,6 +676,8 @@ public: COMMAND_RANGE_HANDLER(IDM_GOTO_TRAVEL_FIRSTTARGET, IDM_GOTO_TRAVEL_LASTTARGET, OnGoTravel) COMMAND_RANGE_HANDLER(IDM_EXPLORERBAND_BEGINCUSTOM, IDM_EXPLORERBAND_ENDCUSTOM, OnExplorerBar) MESSAGE_HANDLER(WM_COMMAND, RelayCommands) + MESSAGE_HANDLER(BWM_SETTINGCHANGE, OnSettingsChange) + MESSAGE_HANDLER(BWM_GETSETTINGSPTR, OnGetSettingsPtr) END_MSG_MAP() BEGIN_CONNECTION_POINT_MAP(CShellBrowser) @@ -714,6 +718,7 @@ CShellBrowser::CShellBrowser() fHistoryObject = NULL; fHistoryStream = NULL; fHistoryBindContext = NULL; + m_settings.Load(); } CShellBrowser::~CShellBrowser() @@ -783,11 +788,11 @@ HRESULT CShellBrowser::Initialize() fToolbarProxy.Initialize(m_hWnd, clientBar); - LoadSettings(); + LoadCabinetState(); // create status bar DWORD dwStatusStyle = WS_CHILD | WS_CLIPSIBLINGS | SBARS_SIZEGRIP | SBARS_TOOLTIPS; - if (fStatusBarVisible) + if (m_settings.fStatusBarVisible) dwStatusStyle |= WS_VISIBLE; fStatusBar = ::CreateWindowExW(0, STATUSCLASSNAMEW, NULL, dwStatusStyle, 0, 0, 500, 20, m_hWnd, (HMENU)IDC_STATUSBAR, @@ -1449,7 +1454,7 @@ void CShellBrowser::RepositionBars() GetClientRect(&clientRect); - if (fStatusBarVisible && fStatusBar) + if (m_settings.fStatusBarVisible && fStatusBar) { ::GetWindowRect(fStatusBar, &statusRect); ::SetWindowPos(fStatusBar, NULL, clientRect.left, clientRect.bottom - (statusRect.bottom - statusRect.top), @@ -1517,13 +1522,8 @@ void CShellBrowser::RepositionBars() clientRect.bottom - clientRect.top, SWP_NOOWNERZORDER | SWP_NOZORDER); } -void CShellBrowser::LoadSettings() +void CShellBrowser::LoadCabinetState() { - fStatusBarVisible = SHRegGetBoolUSValueW(L"Software\\Microsoft\\Internet Explorer\\Main", - L"StatusBarOther", - FALSE, - FALSE); - fCabinetState.fFullPathTitle = SHRegGetBoolUSValueW(L"Software\\Microsoft\\Windows\\CurrentVersion\\Explorer\\CabinetState", L"FullPath", FALSE, @@ -1757,7 +1757,7 @@ void CShellBrowser::UpdateViewMenu(HMENU theMenu) menuItemInfo.hSubMenu = toolbarMenu; SetMenuItemInfo(theMenu, IDM_VIEW_TOOLBARS, FALSE, &menuItemInfo); } - SHCheckMenuItem(theMenu, IDM_VIEW_STATUSBAR, fStatusBarVisible ? TRUE : FALSE); + SHCheckMenuItem(theMenu, IDM_VIEW_STATUSBAR, m_settings.fStatusBarVisible ? TRUE : FALSE); } HRESULT CShellBrowser::BuildExplorerBandMenu() @@ -3681,21 +3681,9 @@ LRESULT CShellBrowser::OnOrganizeFavorites(WORD wNotifyCode, WORD wID, HWND hWnd LRESULT CShellBrowser::OnToggleStatusBarVisible(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - fStatusBarVisible = !fStatusBarVisible; - if (fStatusBar) - { - ::ShowWindow(fStatusBar, fStatusBarVisible ? SW_SHOW : SW_HIDE); - RepositionBars(); - } - - DWORD dwStatusBarVisible = fStatusBarVisible; - SHRegSetUSValueW(L"Software\\Microsoft\\Internet Explorer\\Main", - L"StatusBarOther", - REG_DWORD, - &dwStatusBarVisible, - sizeof(dwStatusBarVisible), - SHREGSET_FORCE_HKCU); - + m_settings.fStatusBarVisible = !m_settings.fStatusBarVisible; + m_settings.Save(); + SendMessageW(BWM_SETTINGCHANGE, 0, (LPARAM)&m_settings); return 0; } @@ -3800,6 +3788,30 @@ LRESULT CShellBrowser::RelayCommands(UINT uMsg, WPARAM wParam, LPARAM lParam, BO return 0; } +LRESULT CShellBrowser::OnSettingsChange(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) +{ + /* Refresh child windows */ + ::SendMessageW(fClientBars[BIInternetToolbar].hwnd, uMsg, wParam, lParam); + + /* Refresh status bar */ + if (fStatusBar) + { + ::ShowWindow(fStatusBar, m_settings.fStatusBarVisible ? SW_SHOW : SW_HIDE); + RepositionBars(); + } + + return 0; +} + +LRESULT CShellBrowser::OnGetSettingsPtr(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) +{ + if (!lParam) + return ERROR_INVALID_PARAMETER; + + *(ShellSettings**)lParam = &m_settings; + return NO_ERROR; +} + HRESULT CShellBrowser_CreateInstance(REFIID riid, void **ppv) { return ShellObjectCreatorInit<CShellBrowser>(riid, ppv);

1 year, 10 months

[reactos] 01/01: [KERNEL32] CreateProcessInternalW(): Always use Peb variable

by Serge Gautherie

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=8888311452009a409673a… commit 8888311452009a409673adf16d99832ff7e5fe71 Author: Serge Gautherie <reactos-git_serge_171003(a)gautherie.fr> AuthorDate: Thu Aug 17 02:51:09 2023 +0200 Commit: Serge Gautherie <reactos-git_serge_171003(a)gautherie.fr> CommitDate: Thu Aug 17 02:51:09 2023 +0200 [KERNEL32] CreateProcessInternalW(): Always use Peb variable Trivial consistency/optimization. Addendum to d81cd01 (r59637). --- dll/win32/kernel32/client/proc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dll/win32/kernel32/client/proc.c b/dll/win32/kernel32/client/proc.c index fdfd390395e..9f1f1c1a4a0 100644 --- a/dll/win32/kernel32/client/proc.c +++ b/dll/win32/kernel32/client/proc.c @@ -3555,7 +3555,7 @@ StartScan: /* If the process is being debugged, only read IFEO if the PEB says so */ if (!(dwCreationFlags & (DEBUG_PROCESS | DEBUG_ONLY_THIS_PROCESS)) || - (NtCurrentPeb()->ReadImageFileExecOptions)) + (Peb->ReadImageFileExecOptions)) { /* Let's do this! Attempt to open IFEO */ IFEOStatus = LdrOpenImageFileOptionsKey(&PathName, 0, &KeyHandle);

1 year, 10 months

[reactos] 01/01: [KERNEL32] Fix startup context of initial thread

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=5645f7cf8f5b9f41de782… commit 5645f7cf8f5b9f41de782535081983db0a7cfb35 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Mon Aug 14 22:03:00 2023 +0300 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Wed Aug 16 22:03:05 2023 +0300 [KERNEL32] Fix startup context of initial thread When creating a new process the monster function CreateProcessInternalW calls BaseInitializeContext to set up the context for the initial thread. One of the parameters is the PEB pointer. Instead of passing the pointer to the newly created process' PEB, the function was passing it's own PEB address, leading to test failures. How this is not an issue otherwise is a mystery to be resolved by someone else. Also remove ros_skip_flaky marks in kernel32_winetest:process (yes, flaky, because sometimes the PEB addresses actually match) --- dll/win32/kernel32/client/proc.c | 2 +- modules/rostests/winetests/kernel32/process.c | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/dll/win32/kernel32/client/proc.c b/dll/win32/kernel32/client/proc.c index 5a8c78c1364..fdfd390395e 100644 --- a/dll/win32/kernel32/client/proc.c +++ b/dll/win32/kernel32/client/proc.c @@ -4181,7 +4181,7 @@ StartScan: /* Create the Thread's Context */ BaseInitializeContext(&Context, - Peb, + RemotePeb, ImageInformation.TransferAddress, InitialTeb.StackBase, 0); diff --git a/modules/rostests/winetests/kernel32/process.c b/modules/rostests/winetests/kernel32/process.c index 2944c9bf4cb..9a9a583ff53 100644 --- a/modules/rostests/winetests/kernel32/process.c +++ b/modules/rostests/winetests/kernel32/process.c @@ -3439,9 +3439,7 @@ static void test_SuspendProcessState(void) #endif ret = ReadProcessMemory( pi.hProcess, peb_ptr, &child_peb, sizeof(child_peb), NULL ); - ros_skip_flaky ok( ret, "Failed to read PEB (%u)\n", GetLastError() ); - ros_skip_flaky ok( child_peb.ImageBaseAddress == exe_base, "wrong base %p/%p\n", child_peb.ImageBaseAddress, exe_base ); ok( entry_ptr == (char *)exe_base + nt_header.OptionalHeader.AddressOfEntryPoint,

1 year, 10 months

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs