Ros-diffs February 2017

ros-diffs@reactos.org

13 participants
353 discussions

[mjansen] 73686: [HIVECLS|MSPAINT] Add Edit menu item for bitmaps. CORE-12679 #resolve

by mjansen＠svn.reactos.org

Author: mjansen Date: Sat Feb 4 21:26:59 2017 New Revision: 73686 URL: http://svn.reactos.org/svn/reactos?rev=73686&view=rev Log: [HIVECLS|MSPAINT] Add Edit menu item for bitmaps. CORE-12679 #resolve Modified: trunk/reactos/boot/bootdata/hivecls.inf Modified: trunk/reactos/boot/bootdata/hivecls.inf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/bootdata/hivecls.inf?… ============================================================================== --- trunk/reactos/boot/bootdata/hivecls.inf [iso-8859-1] (original) +++ trunk/reactos/boot/bootdata/hivecls.inf [iso-8859-1] Sat Feb 4 21:26:59 2017 @@ -193,10 +193,17 @@ HKCR,".jpeg","",0x00000000,"jpegfile" HKCR,".jpeg","Content Type",0x00000000,"image/jpeg" HKCR,".jpeg","PerceivedType",0x00000000,"image" +HKCR,".jpe","",0x00000000,"jpegfile" +HKCR,".jpe","Content Type",0x00000000,"image/jpeg" +HKCR,".jpe","PerceivedType",0x00000000,"image" +HKCR,".jfif","",0x00000000,"jpegfile" +HKCR,".jfif","Content Type",0x00000000,"image/jpeg" +HKCR,".jfif","PerceivedType",0x00000000,"image" HKCR,"jpegfile","",0x00000000,"JPEG Image" HKCR,"jpegfile","FriendlyTypeName",0x00020000,"@%SystemRoot%\system32\shimgvw.dll,-303" HKCR,"jpegfile\DefaultIcon","",0x00020000,"%SystemRoot%\system32\shimgvw.dll,0" HKCR,"jpegfile\shell\open\command","",0x00020000,"rundll32.exe %SystemRoot%\system32\shimgvw.dll,ImageView_Fullscreen %1" +HKCR,"jpegfile\shell\edit\command","",0x00020000,"%SystemRoot%\system32\mspaint.exe %1" ; Bitmap Images HKCR,".bmp","",0x00000000,"bmpfile" @@ -204,10 +211,14 @@ HKCR,".bmp","PerceivedType",0x00000000,"image" HKCR,".bmp\shellnew","",0x00000000,"" HKCR,".bmp\shellnew","NullFile",0x00020000,"" +HKCR,".dib","",0x00000000,"bmpfile" +HKCR,".dib","Content Type",0x00000000,"image/bmp" +HKCR,".dib","PerceivedType",0x00000000,"image" HKCR,"bmpfile","",0x00000000,"Bitmap Image" HKCR,"bmpfile","FriendlyTypeName",0x00020000,"@%SystemRoot%\system32\shimgvw.dll,-304" HKCR,"bmpfile\DefaultIcon","",0x00020000,"%SystemRoot%\system32\shimgvw.dll,1" HKCR,"bmpfile\shell\open\command","",0x00020000,"rundll32.exe %SystemRoot%\system32\shimgvw.dll,ImageView_Fullscreen %1" +HKCR,"bmpfile\shell\edit\command","",0x00020000,"%SystemRoot%\system32\mspaint.exe %1" ; PNG Images HKCR,".png","",0x00000000,"pngfile" @@ -217,6 +228,7 @@ HKCR,"pngfile","FriendlyTypeName",0x00020000,"@%SystemRoot%\system32\shimgvw.dll,-305" HKCR,"pngfile\DefaultIcon","",0x00020000,"%SystemRoot%\system32\shimgvw.dll,2" HKCR,"pngfile\shell\open\command","",0x00020000,"rundll32.exe %SystemRoot%\system32\shimgvw.dll,ImageView_Fullscreen %1" +HKCR,"pngfile\shell\edit\command","",0x00020000,"%SystemRoot%\system32\mspaint.exe %1" ; TIF Images HKCR,".tif","",0x00000000,"TIFImage.Document" @@ -229,6 +241,7 @@ HKCR,"TIFImage.Document","FriendlyTypeName",0x00020000,"@%SystemRoot%\system32\shimgvw.dll,-306" HKCR,"TIFImage.Document\DefaultIcon","",0x00020000,"%SystemRoot%\system32\shimgvw.dll,0" HKCR,"TIFImage.Document\shell\open\command","",0x00020000,"rundll32.exe %SystemRoot%\system32\shimgvw.dll,ImageView_Fullscreen %1" +HKCR,"TIFImage.Document\shell\edit\command","",0x00020000,"%SystemRoot%\system32\mspaint.exe %1" ; WMF Images HKCR,".wmf","",0x00000000,"wmffile"

7 years, 10 months

[ion] 73685: [BOOTLIB]: Super cool that y'all went and fixed typos in the codebase, but fixing UEFI Official Specification Headers in our source tree is a bit much. Please submit a pull request to ...

by ion＠svn.reactos.org

Author: ion Date: Sat Feb 4 20:45:02 2017 New Revision: 73685 URL: http://svn.reactos.org/svn/reactos?rev=73685&view=rev Log: [BOOTLIB]: Super cool that y'all went and fixed typos in the codebase, but fixing UEFI Official Specification Headers in our source tree is a bit much. Please submit a pull request to EDK-II if you'd like. Modified: trunk/reactos/boot/environ/include/efi/BlockIo.h trunk/reactos/boot/environ/include/efi/ProcessorBind.h trunk/reactos/boot/environ/include/efi/SimpleTextOut.h trunk/reactos/boot/environ/include/efi/UefiSpec.h trunk/reactos/boot/environ/include/efi/UgaDraw.h Modified: trunk/reactos/boot/environ/include/efi/BlockIo.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/environ/include/efi/B… ============================================================================== --- trunk/reactos/boot/environ/include/efi/BlockIo.h [iso-8859-1] (original) +++ trunk/reactos/boot/environ/include/efi/BlockIo.h [iso-8859-1] Sat Feb 4 20:45:02 2017 @@ -96,7 +96,7 @@ @retval EFI_WRITE_PROTECTED The device can not be written to. @retval EFI_DEVICE_ERROR The device reported an error while performing the write. @retval EFI_NO_MEDIA There is no media in the device. - @retval EFI_MEDIA_CHANGED The MediaId does not match the current device. + @retval EFI_MEDIA_CHNAGED The MediaId does not matched the current device. @retval EFI_BAD_BUFFER_SIZE The Buffer was not a multiple of the block size of the device. @retval EFI_INVALID_PARAMETER The write request contains LBAs that are not valid, or the buffer is not on proper alignment. @@ -118,7 +118,7 @@ @param This Indicates a pointer to the calling context. @retval EFI_SUCCESS All outstanding data was written to the device - @retval EFI_DEVICE_ERROR The device reported an error while writing back the data + @retval EFI_DEVICE_ERROR The device reported an error while writting back the data @retval EFI_NO_MEDIA There is no media in the device. **/ @@ -133,7 +133,7 @@ **/ typedef struct { /// - /// The current media Id. If the media changes, this value is changed. + /// The curent media Id. If the media changes, this value is changed. /// UINT32 MediaId; @@ -144,7 +144,7 @@ /// /// TRUE if there is a media currently present in the device; - /// otherwise, FALSE. THis field shows the media present status + /// othersise, FALSE. THis field shows the media present status /// as of the most recent ReadBlocks() or WriteBlocks() call. /// BOOLEAN MediaPresent; Modified: trunk/reactos/boot/environ/include/efi/ProcessorBind.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/environ/include/efi/P… ============================================================================== --- trunk/reactos/boot/environ/include/efi/ProcessorBind.h [iso-8859-1] (original) +++ trunk/reactos/boot/environ/include/efi/ProcessorBind.h [iso-8859-1] Sat Feb 4 20:45:02 2017 @@ -79,7 +79,7 @@ #pragma warning ( disable : 4057 ) // -// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this warning +// ASSERT(FALSE) or while (TRUE) are legal constructes so supress this warning // #pragma warning ( disable : 4127 ) Modified: trunk/reactos/boot/environ/include/efi/SimpleTextOut.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/environ/include/efi/S… ============================================================================== --- trunk/reactos/boot/environ/include/efi/SimpleTextOut.h [iso-8859-1] (original) +++ trunk/reactos/boot/environ/include/efi/SimpleTextOut.h [iso-8859-1] Sat Feb 4 20:45:02 2017 @@ -146,10 +146,10 @@ #define EFI_WIDE_ATTRIBUTE 0x80 /** -Reset the text output device hardware and optionally run diagnostics +Reset the text output device hardware and optionaly run diagnostics @param This The protocol instance pointer. -@param ExtendedVerification Driver may perform more exhaustive verification +@param ExtendedVerification Driver may perform more exhaustive verfication operation of the device during reset. @retval EFI_SUCCESS The text output device was reset. @@ -368,7 +368,7 @@ /// INT32 CursorRow; /// - /// The cursor is currently visible or not. + /// The cursor is currently visbile or not. /// BOOLEAN CursorVisible; } EFI_SIMPLE_TEXT_OUTPUT_MODE; Modified: trunk/reactos/boot/environ/include/efi/UefiSpec.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/environ/include/efi/U… ============================================================================== --- trunk/reactos/boot/environ/include/efi/UefiSpec.h [iso-8859-1] (original) +++ trunk/reactos/boot/environ/include/efi/UefiSpec.h [iso-8859-1] Sat Feb 4 20:45:02 2017 @@ -817,7 +817,7 @@ @retval EFI_SUCCESS Image was loaded into memory correctly. @retval EFI_NOT_FOUND Both SourceBuffer and DevicePath are NULL. -@retval EFI_INVALID_PARAMETER One or more parameters are invalid. +@retval EFI_INVALID_PARAMETER One or more parametes are invalid. @retval EFI_UNSUPPORTED The image type is not supported. @retval EFI_OUT_OF_RESOURCES Image was not loaded due to insufficient resources. @retval EFI_LOAD_ERROR Image was not loaded because the image format was corrupt or not @@ -969,7 +969,7 @@ /// /// Used to induce a system-wide reset. This sets all circuitry within the /// system to its initial state. This type of reset is asynchronous to system - /// operation and operates without regard to cycle boundaries. EfiColdReset + /// operation and operates withgout regard to cycle boundaries. EfiColdReset /// is tantamount to a system power cycle. /// EfiResetCold, @@ -1277,7 +1277,7 @@ @retval EFI_UNSUPPORTED Handle does not support Protocol. @retval EFI_INVALID_PARAMETER One or more parameters are invalid. @retval EFI_ACCESS_DENIED Required attributes can't be supported in current environment. -@retval EFI_ALREADY_STARTED Item on the open list already has required attributes whose agent +@retval EFI_ALREADY_STARTED Item on the open list already has requierd attributes whose agent handle is the same as AgentHandle. **/ @@ -1637,13 +1637,13 @@ @param CapsuleHeaderArray Virtual pointer to an array of virtual pointers to the capsules being passed into update capsule. @param CapsuleCount Number of pointers to EFI_CAPSULE_HEADER in -CapsuleHeaderArray. +CaspuleHeaderArray. @param ScatterGatherList Physical pointer to a set of EFI_CAPSULE_BLOCK_DESCRIPTOR that describes the location in physical memory of a set of capsules. @retval EFI_SUCCESS Valid capsule was passed. If -CAPSULE_FLAGS_PERSIST_ACROSS_RESET is not set, the +CAPSULE_FLAGS_PERSIT_ACROSS_RESET is not set, the capsule has been successfully processed by the firmware. @retval EFI_INVALID_PARAMETER CapsuleSize is NULL, or an incompatible set of flags were set in the capsule header. @@ -1667,8 +1667,8 @@ @param CapsuleHeaderArray Virtual pointer to an array of virtual pointers to the capsules being passed into update capsule. @param CapsuleCount Number of pointers to EFI_CAPSULE_HEADER in -CapsuleHeaderArray. -@param MaximumCapsuleSize On output the maximum size that UpdateCapsule() can +CaspuleHeaderArray. +@param MaxiumCapsuleSize On output the maximum size that UpdateCapsule() can support as an argument to UpdateCapsule() via CapsuleHeaderArray and ScatterGatherList. @param ResetType Returns the type of reset required for the capsule update. Modified: trunk/reactos/boot/environ/include/efi/UgaDraw.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/environ/include/efi/U… ============================================================================== --- trunk/reactos/boot/environ/include/efi/UgaDraw.h [iso-8859-1] (original) +++ trunk/reactos/boot/environ/include/efi/UgaDraw.h [iso-8859-1] Sat Feb 4 20:45:02 2017 @@ -85,7 +85,7 @@ } EFI_UGA_PIXEL_UNION; /// -/// Enumeration value for actions of Blt operations. +/// Enumration value for actions of Blt operations. /// typedef enum { EfiUgaVideoFill, ///< Write data from the BltBuffer pixel (SourceX, SourceY) @@ -112,8 +112,8 @@ ///< (DestinationX, DestinationY) (DestinationX + Width, DestinationY + Height). ///< The BltBuffer and Delta are not used in this mode. - EfiUgaBltMax ///< Maximum value for enumeration value of Blt operation. If a Blt operation - ///< larger or equal to this enumeration value, it is invalid. + EfiUgaBltMax ///< Maxmimum value for enumration value of Blt operation. If a Blt operation + ///< larger or equal to this enumration value, it is invalid. } EFI_UGA_BLT_OPERATION; /** @@ -133,7 +133,7 @@ @retval EFI_SUCCESS - The Blt operation completed. @retval EFI_INVALID_PARAMETER - BltOperation is not valid. - @retval EFI_DEVICE_ERROR - A hardware error occured writing to the video buffer. + @retval EFI_DEVICE_ERROR - A hardware error occured writting to the video buffer. **/ typedef

7 years, 10 months

[ion] 73684: [BOOTMGR]: ThFabba making me jealous. Fix a logic bug that Coverity *didn't* find. Ha. I'm better. [BOOTLIB]: Apply the "Fuck You Binutils" hack only for ld-generated files, to unbreak...

by ion＠svn.reactos.org

Author: ion Date: Sat Feb 4 19:45:38 2017 New Revision: 73684 URL: http://svn.reactos.org/svn/reactos?rev=73684&view=rev Log: [BOOTMGR]: ThFabba making me jealous. Fix a logic bug that Coverity *didn't* find. Ha. I'm better. [BOOTLIB]: Apply the "Fuck You Binutils" hack only for ld-generated files, to unbreak MSVC UEFI boot. Modified: trunk/reactos/boot/environ/app/bootmgr/bootmgr.c trunk/reactos/boot/environ/lib/misc/image.c Modified: trunk/reactos/boot/environ/app/bootmgr/bootmgr.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/environ/app/bootmgr/b… ============================================================================== --- trunk/reactos/boot/environ/app/bootmgr/bootmgr.c [iso-8859-1] (original) +++ trunk/reactos/boot/environ/app/bootmgr/bootmgr.c [iso-8859-1] Sat Feb 4 19:45:38 2017 @@ -2276,7 +2276,7 @@ BcdLibraryDevice_ApplicationDevice, &AppDevice, NULL); - if (NT_SUCCESS(Status)) + if (!NT_SUCCESS(Status)) { /* Force re-enumeration */ Status = BlFwEnumerateDevice(AppDevice); Modified: trunk/reactos/boot/environ/lib/misc/image.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/environ/lib/misc/imag… ============================================================================== --- trunk/reactos/boot/environ/lib/misc/image.c [iso-8859-1] (original) +++ trunk/reactos/boot/environ/lib/misc/image.c [iso-8859-1] Sat Feb 4 19:45:38 2017 @@ -883,7 +883,6 @@ /* Record our current position (right after the headers) */ EndOfHeaders = (ULONG_PTR)VirtualAddress + HeaderSize; - EfiPrintf(L"here\r\n"); /* Get the first section and iterate through each one */ Section = IMAGE_FIRST_SECTION(NtHeaders); @@ -981,17 +980,20 @@ if (!First) { /* FUCK YOU BINUTILS */ - if ((*(PULONG)&Section->Name == 'ler.') && (RawSize < AlignSize)) + if (NtHeaders->OptionalHeader.MajorLinkerVersion < 7) { - /* Piece of shit won't build relocations when you tell it to, - * either by using --emit-relocs or --dynamicbase. People online - * have found out that by using -pie-executable you can get this - * to happen, but then it turns out that the .reloc section is - * incorrectly sized, and results in a corrupt PE. However, they - * still compute the checksum using the correct value. What idiots. - */ - WorkaroundForBinutils = AlignSize - RawSize; - AlignSize -= WorkaroundForBinutils; + if ((*(PULONG)&Section->Name == 'ler.') && (RawSize < AlignSize)) + { + /* Piece of shit won't build relocations when you tell it to, + * either by using --emit-relocs or --dynamicbase. People online + * have found out that by using -pie-executable you can get this + * to happen, but then it turns out that the .reloc section is + * incorrectly sized, and results in a corrupt PE. However, they + * still compute the checksum using the correct value. What idiots. + */ + WorkaroundForBinutils = AlignSize - RawSize; + AlignSize -= WorkaroundForBinutils; + } } /* Yes, read the section data */

7 years, 10 months

[pschweitzer] 73683: [FASTFAT] vfatMakeFCBFromDirEntry(), use appropriate tag on free. Dedicated to Thomas :-)

by pschweitzer＠svn.reactos.org

Author: pschweitzer Date: Sat Feb 4 19:37:40 2017 New Revision: 73683 URL: http://svn.reactos.org/svn/reactos?rev=73683&view=rev Log: [FASTFAT] vfatMakeFCBFromDirEntry(), use appropriate tag on free. Dedicated to Thomas :-) Modified: trunk/reactos/drivers/filesystems/fastfat/fcb.c Modified: trunk/reactos/drivers/filesystems/fastfat/fcb.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat/fcb.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat/fcb.c [iso-8859-1] Sat Feb 4 19:37:40 2017 @@ -732,7 +732,7 @@ if (!NT_SUCCESS(Status)) { vfatReleaseFCB(vcb, rcFCB); - ExFreePool(NameU.Buffer); + ExFreePoolWithTag(NameU.Buffer, TAG_FCB); return Status; } } @@ -741,7 +741,7 @@ vfatAddFCBToTable(vcb, rcFCB); *fileFCB = rcFCB; - ExFreePool(NameU.Buffer); + ExFreePoolWithTag(NameU.Buffer, TAG_FCB); return STATUS_SUCCESS; }

7 years, 10 months

[ion] 73682: [CONFIGURE]: Better fix thanks to gigaherz.

by ion＠svn.reactos.org

Author: ion Date: Sat Feb 4 18:55:47 2017 New Revision: 73682 URL: http://svn.reactos.org/svn/reactos?rev=73682&view=rev Log: [CONFIGURE]: Better fix thanks to gigaherz. Modified: trunk/reactos/configure.cmd Modified: trunk/reactos/configure.cmd URL: http://svn.reactos.org/svn/reactos/trunk/reactos/configure.cmd?rev=73682&r1… ============================================================================== --- trunk/reactos/configure.cmd [iso-8859-1] (original) +++ trunk/reactos/configure.cmd [iso-8859-1] Sat Feb 4 18:55:47 2017 @@ -58,7 +58,7 @@ cl 2>&1 | find "17.00." > NUL && set VS_VERSION=11 cl 2>&1 | find "18.00." > NUL && set VS_VERSION=12 cl 2>&1 | find "19.00." > NUL && set VS_VERSION=14 - cl 2>&1 | find "19.10." > NUL && set VS_VERSION=14 + cl 2>&1 | find "19.10." > NUL && set VS_VERSION=15 if not defined VS_VERSION ( echo Error: Visual Studio version too old or version detection failed. endlocal @@ -147,6 +147,15 @@ ) else ( set CMAKE_GENERATOR="Visual Studio 14" ) + ) else if "!VS_VERSION!" == "15" ( + if "!ARCH!" == "amd64" ( + set CMAKE_GENERATOR="Visual Studio 15 Win64" + ) else if "!ARCH!" == "arm" ( + set CMAKE_GENERATOR="Visual Studio 15 ARM" + set CMAKE_GENERATOR_HOST="Visual Studio 15" + ) else ( + set CMAKE_GENERATOR="Visual Studio 15" + ) ) ) else if /I "%1" == "RTC" ( echo Runtime checks enabled

7 years, 10 months

[tfaber] 73681: [LIBUSB] - Avoid pointless assignment in CDMAMemoryManager::Initialize. CID 515285

by tfaber＠svn.reactos.org

Author: tfaber Date: Sat Feb 4 18:36:13 2017 New Revision: 73681 URL: http://svn.reactos.org/svn/reactos?rev=73681&view=rev Log: [LIBUSB] - Avoid pointless assignment in CDMAMemoryManager::Initialize. CID 515285 Modified: trunk/reactos/sdk/lib/drivers/libusb/memory_manager.cpp Modified: trunk/reactos/sdk/lib/drivers/libusb/memory_manager.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/sdk/lib/drivers/libusb/mem… ============================================================================== --- trunk/reactos/sdk/lib/drivers/libusb/memory_manager.cpp [iso-8859-1] (original) +++ trunk/reactos/sdk/lib/drivers/libusb/memory_manager.cpp [iso-8859-1] Sat Feb 4 18:36:13 2017 @@ -118,7 +118,6 @@ m_PhysicalAddress = PhysicalAddress; m_VirtualBase = VirtualBase; m_DmaBufferSize = DmaBufferSize; - m_BitmapBuffer = m_BitmapBuffer; m_Lock = Lock; m_BlockSize = DefaultBlockSize;

7 years, 10 months

[ion] 73680: [CONFIGURE]: Hack-support VS15 (VS 2017)

by ion＠svn.reactos.org

Author: ion Date: Sat Feb 4 17:59:32 2017 New Revision: 73680 URL: http://svn.reactos.org/svn/reactos?rev=73680&view=rev Log: [CONFIGURE]: Hack-support VS15 (VS 2017) Modified: trunk/reactos/configure.cmd Modified: trunk/reactos/configure.cmd URL: http://svn.reactos.org/svn/reactos/trunk/reactos/configure.cmd?rev=73680&r1… ============================================================================== --- trunk/reactos/configure.cmd [iso-8859-1] (original) +++ trunk/reactos/configure.cmd [iso-8859-1] Sat Feb 4 17:59:32 2017 @@ -58,6 +58,7 @@ cl 2>&1 | find "17.00." > NUL && set VS_VERSION=11 cl 2>&1 | find "18.00." > NUL && set VS_VERSION=12 cl 2>&1 | find "19.00." > NUL && set VS_VERSION=14 + cl 2>&1 | find "19.10." > NUL && set VS_VERSION=14 if not defined VS_VERSION ( echo Error: Visual Studio version too old or version detection failed. endlocal

7 years, 10 months

[pschweitzer] 73679: [FASTFAT] In vfatMakeFCBFromDirEntry(), handle vfatFCBInitializeCacheFromVolume() failure

by pschweitzer＠svn.reactos.org

Author: pschweitzer Date: Sat Feb 4 17:54:38 2017 New Revision: 73679 URL: http://svn.reactos.org/svn/reactos?rev=73679&view=rev Log: [FASTFAT] In vfatMakeFCBFromDirEntry(), handle vfatFCBInitializeCacheFromVolume() failure Modified: trunk/reactos/drivers/filesystems/fastfat/fcb.c Modified: trunk/reactos/drivers/filesystems/fastfat/fcb.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat/fcb.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat/fcb.c [iso-8859-1] Sat Feb 4 17:54:38 2017 @@ -728,7 +728,13 @@ rcFCB->RefCount = 1; if (vfatFCBIsDirectory(rcFCB)) { - vfatFCBInitializeCacheFromVolume(vcb, rcFCB); + Status = vfatFCBInitializeCacheFromVolume(vcb, rcFCB); + if (!NT_SUCCESS(Status)) + { + vfatReleaseFCB(vcb, rcFCB); + ExFreePool(NameU.Buffer); + return Status; + } } rcFCB->parentFcb = directoryFCB; InsertTailList(&directoryFCB->ParentListHead, &rcFCB->ParentListEntry);

7 years, 10 months

[pschweitzer] 73678: [FASTFAT] Drop a hack that seems no longer needed

by pschweitzer＠svn.reactos.org

Author: pschweitzer Date: Sat Feb 4 17:52:39 2017 New Revision: 73678 URL: http://svn.reactos.org/svn/reactos?rev=73678&view=rev Log: [FASTFAT] Drop a hack that seems no longer needed Modified: trunk/reactos/drivers/filesystems/fastfat/close.c Modified: trunk/reactos/drivers/filesystems/fastfat/close.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat/close.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat/close.c [iso-8859-1] Sat Feb 4 17:52:39 2017 @@ -86,13 +86,7 @@ IrpContext->Irp->IoStatus.Information = 0; return STATUS_SUCCESS; } -#if 0 - /* There occurs a dead look at the call to CcRosDeleteFileCache/ObDereferenceObject/VfatClose - in CmLazyCloseThreadMain if VfatClose is execute asynchronous in a worker thread. */ if (!ExAcquireResourceExclusiveLite(&IrpContext->DeviceExt->DirResource, BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) -#else - if (!ExAcquireResourceExclusiveLite(&IrpContext->DeviceExt->DirResource, TRUE)) -#endif { return VfatMarkIrpContextForQueue(IrpContext); }

7 years, 10 months

[tfaber] 73677: [BOOTLIB] - Zero the correct structure in EtfsOpen. CID 1341378

by tfaber＠svn.reactos.org

Author: tfaber Date: Sat Feb 4 17:41:14 2017 New Revision: 73677 URL: http://svn.reactos.org/svn/reactos?rev=73677&view=rev Log: [BOOTLIB] - Zero the correct structure in EtfsOpen. CID 1341378 Modified: trunk/reactos/boot/environ/lib/io/etfs.c Modified: trunk/reactos/boot/environ/lib/io/etfs.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/environ/lib/io/etfs.c… ============================================================================== --- trunk/reactos/boot/environ/lib/io/etfs.c [iso-8859-1] (original) +++ trunk/reactos/boot/environ/lib/io/etfs.c [iso-8859-1] Sat Feb 4 17:41:14 2017 @@ -601,7 +601,7 @@ } /* Zero it out */ - RtlZeroMemory(NewFile, sizeof(*EtfsFile)); + RtlZeroMemory(EtfsFile, sizeof(*EtfsFile)); /* Capture the device ID of the directory */ NewFile->DeviceId = Directory->DeviceId;

7 years, 10 months

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs February 2017