Ros-diffs December 2012

ros-diffs@reactos.org

12 participants
287 discussions

[pschweitzer] 57792: [NTOSKRNL] ExFreePool -> ExFreePoolWithTag
by pschweitzer＠svn.reactos.org 03 Dec '12

03 Dec '12

Author: pschweitzer Date: Mon Dec 3 22:15:28 2012 New Revision: 57792 URL: http://svn.reactos.org/svn/reactos?rev=57792&view=rev Log: [NTOSKRNL] ExFreePool -> ExFreePoolWithTag Modified: trunk/reactos/ntoskrnl/fsrtl/filelock.c Modified: trunk/reactos/ntoskrnl/fsrtl/filelock.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/fsrtl/filelock.c?… ============================================================================== --- trunk/reactos/ntoskrnl/fsrtl/filelock.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/fsrtl/filelock.c [iso-8859-1] Mon Dec 3 22:15:28 2012 @@ -925,7 +925,7 @@ Find.Exclusive.FileLock.EndingByte = SharedRange->End; SharedEntry = SharedRange->Entry.Flink; RemoveEntryList(&SharedRange->Entry); - ExFreePool(SharedRange); + ExFreePoolWithTag(SharedRange, 'FSRA'); /* We need to rebuild the list of shared ranges. */ DPRINT("Removing the lock entry %wZ (%08x%08x:%08x%08x)\n", &FileObject->FileName, @@ -1287,7 +1287,7 @@ SharedRange = CONTAINING_RECORD(SharedEntry, LOCK_SHARED_RANGE, Entry); SharedEntry = SharedEntry->Flink; RemoveEntryList(SharedEntry); - ExFreePool(SharedRange); + ExFreePoolWithTag(SharedRange, 'FSRA'); } while ((Entry = RtlGetElementGenericTable(&InternalInfo->RangeTable, 0)) != NULL) {

1 0

[pschweitzer] 57791: [CDFS] Allow apply the fix from r57789 to CDFS. Even if it doesn't solve the issue, it keeps being right Ref: WDK
by pschweitzer＠svn.reactos.org 02 Dec '12

02 Dec '12

Author: pschweitzer Date: Sun Dec 2 21:04:31 2012 New Revision: 57791 URL: http://svn.reactos.org/svn/reactos?rev=57791&view=rev Log: [CDFS] Allow apply the fix from r57789 to CDFS. Even if it doesn't solve the issue, it keeps being right Ref: WDK Modified: trunk/reactos/drivers/filesystems/cdfs/dirctl.c Modified: trunk/reactos/drivers/filesystems/cdfs/dirctl.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/cdfs/d… ============================================================================== --- trunk/reactos/drivers/filesystems/cdfs/dirctl.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/cdfs/dirctl.c [iso-8859-1] Sun Dec 2 21:04:31 2012 @@ -586,7 +586,15 @@ Buffer = Irp->UserBuffer; } - if (SearchPattern != NULL) + /* Allocate search pattern in case: + * -> We don't have one already in context + * -> We have been given an input pattern + * -> The pattern length is not null + * -> The pattern buffer is not null + * Otherwise, we'll fall later and allocate a match all (*) pattern + */ + if (SearchPattern != NULL && + SearchPattern->Length != 0 && SearchPattern->Buffer != NULL) { if (Ccb->DirectorySearchPattern.Buffer == NULL) {

1 0

[hbelusca] 57790: [USETUP] Beautify "Entrée" and "Échap"
by hbelusca＠svn.reactos.org 02 Dec '12

02 Dec '12

Author: hbelusca Date: Sun Dec 2 20:17:17 2012 New Revision: 57790 URL: http://svn.reactos.org/svn/reactos?rev=57790&view=rev Log: [USETUP] Beautify "Entrée" and "Échap" Modified: trunk/reactos/base/setup/usetup/lang/fr-FR.h Modified: trunk/reactos/base/setup/usetup/lang/fr-FR.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/setup/usetup/lang/fr-… ============================================================================== --- trunk/reactos/base/setup/usetup/lang/fr-FR.h [iso-8859-1] (original) +++ trunk/reactos/base/setup/usetup/lang/fr-FR.h [iso-8859-1] Sun Dec 2 20:17:17 2012 @@ -30,7 +30,7 @@ { 8, 11, - " et appuyer sur ENTER.", + " et appuyer sur ENTRE.", TEXT_STYLE_NORMAL }, { @@ -42,7 +42,7 @@ { 0, 0, - " ENTER = Continuer F3 = Quitter", + " ENTRE = Continuer F3 = Quitter", TEXT_TYPE_STATUS }, { @@ -82,7 +82,7 @@ { 8, 15, - "\x07 Appuyer sur ENTER pour installer ReactOS.", + "\x07 Appuyer sur ENTRE pour installer ReactOS.", TEXT_STYLE_NORMAL }, { @@ -118,7 +118,7 @@ { 0, 0, - " ENTER = Continuer R = Rparer F3 = Quitter", + " ENTRE = Continuer R = Rparer F3 = Quitter", TEXT_TYPE_STATUS }, { @@ -206,7 +206,7 @@ { 8, 23, - "\x07 Appuyer sur ENTER pour installer ReactOS.", + "\x07 Appuyer sur ENTRE pour installer ReactOS.", TEXT_STYLE_NORMAL }, { @@ -218,7 +218,7 @@ { 0, 0, - " ENTER = Continuer F3 = Quitter", + " ENTRE = Continuer F3 = Quitter", TEXT_TYPE_STATUS }, { @@ -342,7 +342,7 @@ { 0, 0, - " ENTER = Retour", + " ENTRE = Retour", TEXT_TYPE_STATUS }, { @@ -417,7 +417,7 @@ { 6, 21, - "Appuyer sur ENTER pour choisir un autre rglage.", + "Appuyer sur ENTRE pour choisir un autre rglage.", TEXT_STYLE_NORMAL }, { @@ -429,13 +429,13 @@ { 6, 24, - "ces rglages matriels\" et appuyer sur ENTER.", - TEXT_STYLE_NORMAL - }, - { - 0, - 0, - " ENTER = Continuer F3 = Quitter", + "ces rglages matriels\" et appuyer sur ENTRE.", + TEXT_STYLE_NORMAL + }, + { + 0, + 0, + " ENTRE = Continuer F3 = Quitter", TEXT_TYPE_STATUS }, { @@ -493,19 +493,19 @@ { 8, 19, - "\x07 Appuyer sur ESC pour retourner la page principale.", + "\x07 Appuyer sur CHAP pour retourner la page principale.", TEXT_STYLE_NORMAL }, { 8, 21, - "\x07 Appuyer sur ENTER pour redmarrer votre ordinateur.", - TEXT_STYLE_NORMAL - }, - { - 0, - 0, - " ESC = Page principale ENTER = Redmarrer", + "\x07 Appuyer sur ENTRE pour redmarrer votre ordinateur.", + TEXT_STYLE_NORMAL + }, + { + 0, + 0, + " CHAP = Page principale ENTRE = Redmarrer", TEXT_TYPE_STATUS }, { @@ -538,13 +538,13 @@ { 8, 11, - " Puis appuyer sur ENTER.", + " Puis appuyer sur ENTRE.", TEXT_STYLE_NORMAL }, { 8, 13, - "\x07 Appuyer sur ESC pour revenir la page prcdente sans changer", + "\x07 Appuyer sur CHAP pour revenir la page prcdente sans changer", TEXT_STYLE_NORMAL }, { @@ -556,7 +556,7 @@ { 0, 0, - " ENTER = Continuer ESC = Annuler F3 = Quitter", + " ENTRE = Continuer CHAP = Annuler F3 = Quitter", TEXT_TYPE_STATUS }, { @@ -636,7 +636,7 @@ { 10, 11, - "Appuyer sur ENTER pour redmarrer votre ordinateur.", + "Appuyer sur ENTRE pour redmarrer votre ordinateur.", TEXT_STYLE_NORMAL }, { @@ -675,13 +675,13 @@ { 8, 11, - " Appuyer sur ENTER.", + " Appuyer sur ENTRE.", TEXT_STYLE_NORMAL }, { 8, 13, - "\x07 Appuyer sur ESC pour revenir la page prcdente sans changer", + "\x07 Appuyer sur CHAP pour revenir la page prcdente sans changer", TEXT_STYLE_NORMAL }, { @@ -693,7 +693,7 @@ { 0, 0, - " ENTER = Continuer ESC = Annuler F3 = Quitter", + " ENTRE = Continuer CHAP = Annuler F3 = Quitter", TEXT_TYPE_STATUS }, { @@ -733,13 +733,13 @@ { 10, 11, - "Appuyer sur ENTER pour redmarrer votre ordinateur.", - TEXT_STYLE_NORMAL - }, - { - 0, - 0, - " ENTER = Redmarrer l'ordinateur", + "Appuyer sur ENTRE pour redmarrer votre ordinateur.", + TEXT_STYLE_NORMAL + }, + { + 0, + 0, + " ENTRE = Redmarrer l'ordinateur", TEXT_TYPE_STATUS }, { @@ -779,13 +779,13 @@ { 6, 14, - "appuyer sur ENTER.", + "appuyer sur ENTRE.", TEXT_STYLE_NORMAL, }, { 0, 0, - " ENTER = Continuer F3 = Quitter", + " ENTRE = Continuer F3 = Quitter", TEXT_TYPE_STATUS }, { @@ -826,7 +826,7 @@ { 8, 13, - "\x07 Appuyer sur ENTER pour installer ReactOS sur la partition choisie.", + "\x07 Appuyer sur ENTRE pour installer ReactOS sur la partition choisie.", TEXT_STYLE_NORMAL }, { @@ -872,13 +872,13 @@ { 6, 10, - "Setup va formater la partition. Appuyer sur ENTER pour continuer.", - TEXT_STYLE_NORMAL - }, - { - 0, - 0, - " ENTER = Continuer F3 = Quitter", + "Setup va formater la partition. Appuyer sur ENTRE pour continuer.", + TEXT_STYLE_NORMAL + }, + { + 0, + 0, + " ENTRE = Continuer F3 = Quitter", TEXT_TYPE_STATUS }, { @@ -930,7 +930,7 @@ { 0, 0, - " ENTER = Continuer F3 = Quitter", + " ENTRE = Continuer F3 = Quitter", TEXT_TYPE_STATUS }, { @@ -1022,7 +1022,7 @@ { 0, 0, - " ENTER = Continuer F3 = Quitter", + " ENTRE = Continuer F3 = Quitter", TEXT_TYPE_STATUS }, { @@ -1056,13 +1056,13 @@ { 8, 11, - " puis appuyez sur ENTER.", + " puis appuyez sur ENTRE.", TEXT_STYLE_NORMAL }, { 8, 13, - "\x07 Appuyez sur ESC pour revenir a la page prcdente sans changer", + "\x07 Appuyez sur CHAP pour revenir a la page prcdente sans changer", TEXT_STYLE_NORMAL }, { @@ -1074,7 +1074,7 @@ { 0, 0, - " ENTER = Continuer ESC = Annuler F3 = Quitter", + " ENTRE = Continuer CHAP = Annuler F3 = Quitter", TEXT_TYPE_STATUS }, { @@ -1108,13 +1108,13 @@ { 8, 11, - " choisie. Puis appuyez sur ENTER.", + " choisie. Puis appuyez sur ENTRE.", TEXT_STYLE_NORMAL }, { 8, 13, - "\x07 Appuyez sur ESC pour revenir a la page prcdente sans changer", + "\x07 Appuyez sur CHAP pour revenir a la page prcdente sans changer", TEXT_STYLE_NORMAL }, { @@ -1126,7 +1126,7 @@ { 0, 0, - " ENTER = Continuer ESC = Annuler F3 = Quitter", + " ENTRE = Continuer CHAP = Annuler F3 = Quitter", TEXT_TYPE_STATUS }, { @@ -1190,19 +1190,19 @@ { 8, 21, - "\x07 Appuyez sur ENTER pour formater la partition.", + "\x07 Appuyez sur ENTRE pour formater la partition.", 0 }, { 8, 23, - "\x07 Appuyez sur ESC pour slectionner une autre partition.", - 0 - }, - { - 0, - 0, - " ENTER = Continuer ESC = Annuler F3 = Quitter", + "\x07 Appuyez sur CHAP pour slectionner une autre partition.", + 0 + }, + { + 0, + 0, + " ENTRE = Continuer CHAP = Annuler F3 = Quitter", TEXT_TYPE_STATUS }, @@ -1243,13 +1243,13 @@ { 8, 21, - "\x07 Appuyez sur ESC pour annuler.", - TEXT_STYLE_NORMAL - }, - { - 0, - 0, - " D = Supprimer la Partition ESC = Annuler F3 = Quitter", + "\x07 Appuyez sur CHAP pour annuler.", + TEXT_STYLE_NORMAL + }, + { + 0, + 0, + " D = Supprimer la Partition CHAP = Annuler F3 = Quitter", TEXT_TYPE_STATUS }, { @@ -1297,64 +1297,64 @@ "ordinateur. Si vous quittez Setup maintenant, vous devrez\n" "lancer Setup de nouveau pour installer ReactOS.\n" "\n" - " \x07 Appuyer sur ENTER pour continuer Setup.\n" + " \x07 Appuyer sur ENTRE pour continuer Setup.\n" " \x07 Appuyer sur F3 pour quitter Setup.", - "F3= Quitter ENTER = Continuer" + "F3= Quitter ENTRE = Continuer" }, { //ERROR_NO_HDD "Setup n'a pu trouver un disque dur.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_NO_SOURCE_DRIVE "Setup n'a pu trouver son lecteur source.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_LOAD_TXTSETUPSIF "Setup n'a pas russi charger le fichier TXTSETUP.SIF.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_CORRUPT_TXTSETUPSIF "Setup a trouv un fichier TXTSETUP.SIF corrompu.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_SIGNATURE_TXTSETUPSIF, "Setup a trouv une signature invalide dans TXTSETUP.SIF.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_DRIVE_INFORMATION "Setup n'a pu rcuprer les informations du disque systme.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_WRITE_BOOT, "Echec de l'installation du code de dmarrage FAT sur la partition systme.", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_LOAD_COMPUTER, "Setup n'a pu charger la liste de type d'ordinateurs.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_LOAD_DISPLAY, "Setup n'a pu charger la liste de rglages des crans.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_LOAD_KEYBOARD, "Setup n'a pu charger la liste de types de claviers.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_LOAD_KBLAYOUT, "Setup n'a pu charger la liste de dispositions de claviers.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_WARN_PARTITION, @@ -1364,8 +1364,8 @@ "Crer ou effacer des partitions peut dtruire la table de partition.\n" "\n" " \x07 Appuyer sur F3 pour quitter Setup." - " \x07 Appuyer sur ENTER pour continuer Setup.", - "F3= Quitter ENTER = Continuer" + " \x07 Appuyer sur ENTRE pour continuer Setup.", + "F3= Quitter ENTRE = Continuer" }, { //ERROR_NEW_PARTITION, @@ -1385,115 +1385,115 @@ { //ERROR_INSTALL_BOOTCODE, "Echec de l'installation du code de dmarrage FAT sur la partition systme.", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_NO_FLOPPY, "Pas de disque dans le lecteur A:.", - "ENTER = Continuer" + "ENTRE = Continuer" }, { //ERROR_UPDATE_KBSETTINGS, "Setup n'a pu mettre jour les rglages de disposition du clavier.", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_UPDATE_DISPLAY_SETTINGS, "Setup n'a pu mettre jour les rglages de l'cran.", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_IMPORT_HIVE, "Setup n'a pu importer un fichier ruche.", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_FIND_REGISTRY "Setup n'a pu trouver les fichiers de la base de registres.", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_CREATE_HIVE, "Setup n'a pu crer les ruches de la base de registres.", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_INITIALIZE_REGISTRY, "Setup n'a pu initialiser la base de registres.", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_INVALID_CABINET_INF, "Le Cabinet n'a pas de fichier inf valide.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_CABINET_MISSING, "Cabinet non trouv.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_CABINET_SCRIPT, "Cabinet n'a pas de script de setup.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_COPY_QUEUE, "Setup n'a pu ouvrir la file d'attente de copie de fichiers.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_CREATE_DIR, "Setup n'a pu crer les rpertoires d'installation.", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_TXTSETUP_SECTION, "Setup n'a pu trouver la section 'Directories'\n" "dans TXTSETUP.SIF.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_CABINET_SECTION, "Setup n'a pu trouver la section 'Directories\n" "dans le cabinet.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_CREATE_INSTALL_DIR "Setup n'a pu crer le rpertoire d'installation.", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_FIND_SETUPDATA, "Setup n'a pu trouver la section 'SetupData'\n" "dans TXTSETUP.SIF.\n", - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_WRITE_PTABLE, "Setup n'a pu crire les tables de partition.\n" - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_ADDING_CODEPAGE, "Setup n'a pu ajouter la page de codes la base de registres.\n" - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_UPDATE_LOCALESETTINGS, "Setup n'a pu changer la langue systme.\n" - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_ADDING_KBLAYOUTS, "Setup n'a pas pu ajouter les dispositions de clavier au registre.\n" - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_UPDATE_GEOID, "Setup n'a pas pu dfinir la geo id.\n" - "ENTER = Redmarrer l'ordinateur" + "ENTRE = Redmarrer l'ordinateur" }, { //ERROR_INSUFFICIENT_DISKSPACE, @@ -1613,9 +1613,9 @@ {STRING_PLEASEWAIT, " Veuillez patienter..."}, {STRING_INSTALLCREATEPARTITION, - " ENTER = Installer C = Crer Partition F3 = Quitter"}, + " ENTRE = Installer C = Crer Partition F3 = Quitter"}, {STRING_INSTALLDELETEPARTITION, - " ENTER = Installer D = Supprimer Partition F3 = Quitter"}, + " ENTRE = Installer D = Supprimer Partition F3 = Quitter"}, {STRING_PARTITIONSIZE, "Taille de la nouvelle partition :"}, {STRING_CHOOSENEWPARTITION, @@ -1623,7 +1623,7 @@ {STRING_HDDSIZE, "Veuillez entrer la taille de la nouvelle partition en mgaoctets."}, {STRING_CREATEPARTITION, - " ENTER = Crer Partition ESC = Annuler F3 = Quitter"}, + " ENTRE = Crer Partition CHAP = Annuler F3 = Quitter"}, {STRING_PARTFORMAT, "Cette Partition sera ensuite formate."}, {STRING_NONFORMATTEDPART, @@ -1633,9 +1633,9 @@ {STRING_CHECKINGPART, "Setup vrifie la partition slectionne."}, {STRING_QUITCONTINUE, - "F3= Quitter ENTER = Continuer"}, + "F3= Quitter ENTRE = Continuer"}, {STRING_REBOOTCOMPUTER, - "ENTER = Redmarrer l'ordinateur"}, + "ENTRE = Redmarrer l'ordinateur"}, {STRING_TXTSETUPFAILED, "Setup n'a pu trouver la section '%S'\ndans TXTSETUP.SIF.\n"}, {STRING_COPYING, @@ -1657,7 +1657,7 @@ {STRING_DONE, " Termin..."}, {STRING_REBOOTCOMPUTER2, - " ENTER = Redmarrer l'ordinateur"}, + " ENTRE = Redmarrer l'ordinateur"}, {STRING_CONSOLEFAIL1, "Impossible d'ouvrir la console\n\n"}, {STRING_CONSOLEFAIL2,

1 0

[pschweitzer] 57789: [FASTFAT] Handle special cases with search patterns in DoQuery(). This may (should?) fix kernel32_winetest::path test Ref: WDK
by pschweitzer＠svn.reactos.org 02 Dec '12

02 Dec '12

Author: pschweitzer Date: Sun Dec 2 16:42:54 2012 New Revision: 57789 URL: http://svn.reactos.org/svn/reactos?rev=57789&view=rev Log: [FASTFAT] Handle special cases with search patterns in DoQuery(). This may (should?) fix kernel32_winetest::path test Ref: WDK Modified: trunk/reactos/drivers/filesystems/fastfat/dir.c Modified: trunk/reactos/drivers/filesystems/fastfat/dir.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat/dir.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat/dir.c [iso-8859-1] Sun Dec 2 16:42:54 2012 @@ -427,7 +427,15 @@ #endif FileInformationClass = Stack->Parameters.QueryDirectory.FileInformationClass; - if (pSearchPattern) + /* Allocate search pattern in case: + * -> We don't have one already in context + * -> We have been given an input pattern + * -> The pattern length is not null + * -> The pattern buffer is not null + * Otherwise, we'll fall later and allocate a match all (*) pattern + */ + if (pSearchPattern && + pSearchPattern->Length != 0 && pSearchPattern->Buffer != NULL) { if (!pCcb->SearchPattern.Buffer) { @@ -606,4 +614,4 @@ return Status; } -/* EOF */ +/* EOF */

1 0

[hbelusca] 57788: Fix a counting bug (spotted by HeisSpiter a.k.a. pschweitzer :) )
by hbelusca＠svn.reactos.org 02 Dec '12

02 Dec '12

Author: hbelusca Date: Sun Dec 2 15:36:18 2012 New Revision: 57788 URL: http://svn.reactos.org/svn/reactos?rev=57788&view=rev Log: Fix a counting bug (spotted by HeisSpiter a.k.a. pschweitzer :) ) Modified: trunk/reactos/ntoskrnl/kdbg/kdb.c Modified: trunk/reactos/ntoskrnl/kdbg/kdb.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kdbg/kdb.c?rev=57… ============================================================================== --- trunk/reactos/ntoskrnl/kdbg/kdb.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kdbg/kdb.c [iso-8859-1] Sun Dec 2 15:36:18 2012 @@ -1705,7 +1705,7 @@ KdbpGetCommandLineSettings( PCHAR p1) { -#define CONST_STR_LEN(x) (sizeof(x)/sizeof(x[0])) +#define CONST_STR_LEN(x) (sizeof(x)/sizeof(x[0]) - 1) while (p1 && (p1 = strchr(p1, ' '))) {

1 0

[hbelusca] 57787: [NTOSKRNL] - Skip other spaces preceding a command option instead of increasing the reading pointer of a constant (should fix KDSERIAL). - Use a macro for representing length in c...
by hbelusca＠svn.reactos.org 02 Dec '12

02 Dec '12

Author: hbelusca Date: Sun Dec 2 15:30:59 2012 New Revision: 57787 URL: http://svn.reactos.org/svn/reactos?rev=57787&view=rev Log: [NTOSKRNL] - Skip other spaces preceding a command option instead of increasing the reading pointer of a constant (should fix KDSERIAL). - Use a macro for representing length in chars of constant strings instead of putting magic values. Modified: trunk/reactos/ntoskrnl/kdbg/kdb.c Modified: trunk/reactos/ntoskrnl/kdbg/kdb.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kdbg/kdb.c?rev=57… ============================================================================== --- trunk/reactos/ntoskrnl/kdbg/kdb.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kdbg/kdb.c [iso-8859-1] Sun Dec 2 15:30:59 2012 @@ -1705,30 +1705,29 @@ KdbpGetCommandLineSettings( PCHAR p1) { - PCHAR p2; - - while (p1 && (p2 = strchr(p1, ' '))) - { - p2 += 2; - - if (!_strnicmp(p2, "KDSERIAL", 8)) - { - p2 += 8; +#define CONST_STR_LEN(x) (sizeof(x)/sizeof(x[0])) + + while (p1 && (p1 = strchr(p1, ' '))) + { + /* Skip other spaces */ + while (*p1 == ' ') ++p1; + + if (!_strnicmp(p1, "KDSERIAL", CONST_STR_LEN("KDSERIAL"))) + { + p1 += CONST_STR_LEN("KDSERIAL"); KdbDebugState |= KD_DEBUG_KDSERIAL; KdpDebugMode.Serial = TRUE; } - else if (!_strnicmp(p2, "KDNOECHO", 8)) - { - p2 += 8; + else if (!_strnicmp(p1, "KDNOECHO", CONST_STR_LEN("KDNOECHO"))) + { + p1 += CONST_STR_LEN("KDNOECHO"); KdbDebugState |= KD_DEBUG_KDNOECHO; } - else if (!_strnicmp(p2, "FIRSTCHANCE", 11)) - { - p2 += 11; + else if (!_strnicmp(p1, "FIRSTCHANCE", CONST_STR_LEN("FIRSTCHANCE"))) + { + p1 += CONST_STR_LEN("FIRSTCHANCE"); KdbpSetEnterCondition(-1, TRUE, KdbEnterAlways); } - - p1 = p2; } }

1 0

[tkreuzer] 57786: [VGA/VGA_NEW] Fix MSVC warnings
by tkreuzer＠svn.reactos.org 01 Dec '12

01 Dec '12

Author: tkreuzer Date: Sat Dec 1 11:49:59 2012 New Revision: 57786 URL: http://svn.reactos.org/svn/reactos?rev=57786&view=rev Log: [VGA/VGA_NEW] Fix MSVC warnings Modified: trunk/reactos/win32ss/drivers/miniport/vga/initvga.c trunk/reactos/win32ss/drivers/miniport/vga_new/vbemodes.c Modified: trunk/reactos/win32ss/drivers/miniport/vga/initvga.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/drivers/miniport/v… ============================================================================== --- trunk/reactos/win32ss/drivers/miniport/vga/initvga.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/drivers/miniport/vga/initvga.c [iso-8859-1] Sat Dec 1 11:49:59 2012 @@ -21,7 +21,7 @@ static VOID FASTCALL vgaSaveRegisters(PVGA_REGISTERS Registers) { - SIZE_T i; + UCHAR i; for (i = 0; i < sizeof(Registers->CRT); i++) { @@ -54,7 +54,7 @@ static VOID FASTCALL vgaSetRegisters(PVGA_REGISTERS Registers) { - SIZE_T i; + UCHAR i; /* Update misc output register */ VideoPortWritePortUchar(MISC, Registers->Misc); Modified: trunk/reactos/win32ss/drivers/miniport/vga_new/vbemodes.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/drivers/miniport/v… ============================================================================== --- trunk/reactos/win32ss/drivers/miniport/vga_new/vbemodes.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/drivers/miniport/vga_new/vbemodes.c [iso-8859-1] Sat Dec 1 11:49:59 2012 @@ -168,7 +168,7 @@ ULONG Size, ScreenSize; PVIDEOMODE VgaMode; PVOID BaseAddress; - ULONG ScreenStride = 0; + ULONG ScreenStride; PHYSICAL_ADDRESS PhysicalAddress; /* Enable only default vga modes if no vesa */ @@ -345,7 +345,8 @@ { /* Read the screen stride (scanline size) */ ScreenStride = RaiseToPower2(VbeModeInfo->BytesPerScanLine); - VgaMode->wbytes = ScreenStride; + //ASSERT(ScreenStride <= MAX_USHORT); + VgaMode->wbytes = (USHORT)ScreenStride; //VideoPortDebugPrint(0, "ScanLines: %lx Stride: %lx\n", VbeModeInfo->BytesPerScanLine, VgaMode->Stride); /* Size of frame buffer is Height X ScanLine, align to bank/page size */ @@ -372,7 +373,8 @@ //VideoPortDebugPrint(0, "LINEAR MODE!!!\n"); ScreenStride = (VbeVersion >= 0x300) ? VbeModeInfo->LinBytesPerScanLine : 0; if (!ScreenStride) ScreenStride = VbeModeInfo->BytesPerScanLine; - VgaMode->wbytes = ScreenStride; + //ASSERT(ScreenStride <= MAX_USHORT); + VgaMode->wbytes = (USHORT)ScreenStride; //VideoPortDebugPrint(0, "ScanLines: %lx Stride: %lx\n", VbeModeInfo->BytesPerScanLine, VgaMode->Stride); /* Size of frame buffer is Height X ScanLine, align to page size */

1 0

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 December 2012