Ros-diffs

ros-diffs@reactos.org

81390 discussions

[ion] 22868: [FORMATTING] - Fix formatting in IoCreateFile
by ion＠svn.reactos.org 05 Jul '06

05 Jul '06

Author: ion Date: Wed Jul 5 23:55:25 2006 New Revision: 22868 URL: http://svn.reactos.org/svn/reactos?rev=22868&view=rev Log: [FORMATTING] - Fix formatting in IoCreateFile Modified: trunk/reactos/ntoskrnl/io/iomgr/file.c Modified: trunk/reactos/ntoskrnl/io/iomgr/file.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/io/iomgr/file.c?r… ============================================================================== --- trunk/reactos/ntoskrnl/io/iomgr/file.c (original) +++ trunk/reactos/ntoskrnl/io/iomgr/file.c Wed Jul 5 23:55:25 2006 @@ -795,199 +795,121 @@ } /* - * NAME EXPORTED - * IoCreateFile@56 - * - * DESCRIPTION - * Either causes a new file or directory to be created, or it - * opens an existing file, device, directory or volume, giving - * the caller a handle for the file object. This handle can be - * used by subsequent calls to manipulate data within the file - * or the file object's state of attributes. - * - * ARGUMENTS - * FileHandle (OUT) - * Points to a variable which receives the file handle - * on return; - * - * DesiredAccess - * Desired access to the file; - * - * ObjectAttributes - * Structure describing the file; - * - * IoStatusBlock (OUT) - * Receives information about the operation on return; - * - * AllocationSize [OPTIONAL] - * Initial size of the file in bytes; - * - * FileAttributes - * Attributes to create the file with; - * - * ShareAccess - * Type of shared access the caller would like to the - * file; - * - * CreateDisposition - * Specifies what to do, depending on whether the - * file already exists; - * - * CreateOptions - * Options for creating a new file; - * - * EaBuffer [OPTIONAL] - * Undocumented; - * - * EaLength - * Undocumented; - * - * CreateFileType - * Type of file (normal, named pipe, mailslot) to create; - * - * ExtraCreateParameters [OPTIONAL] - * Additional creation data for named pipe and mailsots; - * - * Options - * Undocumented. - * - * RETURN VALUE - * Status - * - * NOTE - * Prototype taken from Bo Branten's ntifs.h v15. - * Description taken from old NtCreateFile's which is - * now a wrapper of this call. - * - * REVISIONS - * * @implemented */ NTSTATUS -STDCALL -IoCreateFile(OUT PHANDLE FileHandle, - IN ACCESS_MASK DesiredAccess, +NTAPI +IoCreateFile(OUT PHANDLE FileHandle, + IN ACCESS_MASK DesiredAccess, IN POBJECT_ATTRIBUTES ObjectAttributes, OUT PIO_STATUS_BLOCK IoStatusBlock, - IN PLARGE_INTEGER AllocationSize OPTIONAL, - IN ULONG FileAttributes, - IN ULONG ShareAccess, - IN ULONG CreateDisposition, - IN ULONG CreateOptions, - IN PVOID EaBuffer OPTIONAL, - IN ULONG EaLength, + IN PLARGE_INTEGER AllocationSize OPTIONAL, + IN ULONG FileAttributes, + IN ULONG ShareAccess, + IN ULONG CreateDisposition, + IN ULONG CreateOptions, + IN PVOID EaBuffer OPTIONAL, + IN ULONG EaLength, IN CREATE_FILE_TYPE CreateFileType, - IN PVOID ExtraCreateParameters OPTIONAL, - IN ULONG Options) -{ - PFILE_OBJECT FileObject = NULL; - //PDEVICE_OBJECT DeviceObject; - PIRP Irp; - PEXTENDED_IO_STACK_LOCATION StackLoc; - IO_SECURITY_CONTEXT SecurityContext; - KPROCESSOR_MODE AccessMode; - HANDLE LocalHandle; - LARGE_INTEGER SafeAllocationSize; - PVOID SystemEaBuffer = NULL; - NTSTATUS Status = STATUS_SUCCESS; - AUX_DATA AuxData; - ACCESS_STATE AccessState; + IN PVOID ExtraCreateParameters OPTIONAL, + IN ULONG Options) +{ + PFILE_OBJECT FileObject = NULL; + PIRP Irp; + PEXTENDED_IO_STACK_LOCATION StackLoc; + IO_SECURITY_CONTEXT SecurityContext; + KPROCESSOR_MODE AccessMode; + HANDLE LocalHandle = 0; + LARGE_INTEGER SafeAllocationSize; + PVOID SystemEaBuffer = NULL; + NTSTATUS Status = STATUS_SUCCESS; + AUX_DATA AuxData; + ACCESS_STATE AccessState; KIRQL OldIrql; PKNORMAL_ROUTINE NormalRoutine; PVOID NormalContext; - - DPRINT("IoCreateFile(FileHandle 0x%p, DesiredAccess %x, " - "ObjectAttributes 0x%p ObjectAttributes->ObjectName->Buffer %S)\n", - FileHandle,DesiredAccess,ObjectAttributes, - ObjectAttributes->ObjectName->Buffer); - - ASSERT_IRQL(PASSIVE_LEVEL); - - if (IoStatusBlock == NULL || FileHandle == NULL) - return STATUS_ACCESS_VIOLATION; - - LocalHandle = 0; - - if(Options & IO_NO_PARAMETER_CHECKING) - AccessMode = KernelMode; - else - AccessMode = ExGetPreviousMode(); - - if(AccessMode != KernelMode) - { - _SEH_TRY - { - ProbeForWriteHandle(FileHandle); - ProbeForWrite(IoStatusBlock, - sizeof(IO_STATUS_BLOCK), - sizeof(ULONG)); - if(AllocationSize != NULL) - { - SafeAllocationSize = ProbeForReadLargeInteger(AllocationSize); - } - else - SafeAllocationSize.QuadPart = 0; - - if(EaBuffer != NULL && EaLength > 0) - { - ProbeForRead(EaBuffer, - EaLength, - sizeof(ULONG)); - - /* marshal EaBuffer */ - SystemEaBuffer = ExAllocatePool(NonPagedPool, - EaLength); - if(SystemEaBuffer == NULL) - { - Status = STATUS_INSUFFICIENT_RESOURCES; - _SEH_LEAVE; - } - - RtlCopyMemory(SystemEaBuffer, - EaBuffer, - EaLength); - } - } - _SEH_HANDLE - { - Status = _SEH_GetExceptionCode(); - } - _SEH_END; - - if(!NT_SUCCESS(Status)) - { - return Status; - } - } - else - { - if(AllocationSize != NULL) - SafeAllocationSize = *AllocationSize; - else - SafeAllocationSize.QuadPart = 0; - - if(EaBuffer != NULL && EaLength > 0) - { - SystemEaBuffer = EaBuffer; - } - } - - if(Options & IO_CHECK_CREATE_PARAMETERS) - { - DPRINT1("FIXME: IO_CHECK_CREATE_PARAMETERS not yet supported!\n"); - } - - RtlMapGenericMask(&DesiredAccess, &IoFileObjectType->TypeInfo.GenericMapping); - - /* First try to open an existing named object */ - Status = ObOpenObjectByName(ObjectAttributes, - NULL, - AccessMode, - NULL, - DesiredAccess, - NULL, - &LocalHandle); - + PAGED_CODE(); + + if(Options & IO_NO_PARAMETER_CHECKING) + { + AccessMode = KernelMode; + } + else + { + AccessMode = ExGetPreviousMode(); + } + + if(AccessMode != KernelMode) + { + _SEH_TRY + { + ProbeForWriteHandle(FileHandle); + ProbeForWrite(IoStatusBlock, + sizeof(IO_STATUS_BLOCK), + sizeof(ULONG)); + if (AllocationSize) + { + SafeAllocationSize = ProbeForReadLargeInteger(AllocationSize); + } + else + { + SafeAllocationSize.QuadPart = 0; + } + + if ((EaBuffer) && (EaLength)) + { + ProbeForRead(EaBuffer, + EaLength, + sizeof(ULONG)); + + /* marshal EaBuffer */ + SystemEaBuffer = ExAllocatePool(NonPagedPool, EaLength); + if(!SystemEaBuffer) + { + Status = STATUS_INSUFFICIENT_RESOURCES; + _SEH_LEAVE; + } + + RtlCopyMemory(SystemEaBuffer, EaBuffer, EaLength); + } + } + _SEH_HANDLE + { + Status = _SEH_GetExceptionCode(); + } + _SEH_END; + + if(!NT_SUCCESS(Status)) return Status; + } + else + { + if (AllocationSize) + { + SafeAllocationSize = *AllocationSize; + } + else + { + SafeAllocationSize.QuadPart = 0; + } + + if ((EaBuffer) && (EaLength)) SystemEaBuffer = EaBuffer; + } + + if(Options & IO_CHECK_CREATE_PARAMETERS) + { + DPRINT1("FIXME: IO_CHECK_CREATE_PARAMETERS not yet supported!\n"); + } + + RtlMapGenericMask(&DesiredAccess, &IoFileObjectType->TypeInfo.GenericMapping); + + /* First try to open an existing named object */ + Status = ObOpenObjectByName(ObjectAttributes, + NULL, + AccessMode, + NULL, + DesiredAccess, + NULL, + &LocalHandle); ObReferenceObjectByHandle(LocalHandle, DesiredAccess, NULL, @@ -999,10 +921,6 @@ FileObject->Type = IO_TYPE_FILE; FileObject->Size = sizeof(FILE_OBJECT); - // - // stop stuff that should be in IopParseDevice - // - if (CreateOptions & (FILE_SYNCHRONOUS_IO_ALERT | FILE_SYNCHRONOUS_IO_NONALERT)) { @@ -1034,107 +952,84 @@ { FileObject->Flags |= FO_OPENED_CASE_SENSITIVE; } - /* - * FIXME: We should get the access state from Ob once this function becomes - * a parse routine once the Ob is refactored. - */ - SeCreateAccessState(&AccessState, &AuxData, FILE_ALL_ACCESS, NULL); - - SecurityContext.SecurityQos = NULL; /* ?? */ - SecurityContext.AccessState = &AccessState; - SecurityContext.DesiredAccess = DesiredAccess; - SecurityContext.FullCreateOptions = CreateOptions; - - KeInitializeEvent(&FileObject->Lock, SynchronizationEvent, TRUE); - KeInitializeEvent(&FileObject->Event, NotificationEvent, FALSE); - - DPRINT("FileObject 0x%p\n", FileObject); - DPRINT("FileObject->DeviceObject 0x%p\n", FileObject->DeviceObject); - /* - * Create a new IRP to hand to - * the FS driver: this may fail - * due to resource shortage. - */ - Irp = IoAllocateIrp(FileObject->DeviceObject->StackSize, FALSE); - if (Irp == NULL) - { - ZwClose(LocalHandle); - return STATUS_UNSUCCESSFUL; - } - - /* Now set the IRP data */ - Irp->Tail.Overlay.OriginalFileObject = FileObject; - Irp->RequestorMode = AccessMode; - Irp->Flags = IRP_CREATE_OPERATION | - IRP_SYNCHRONOUS_API | - IRP_DEFER_IO_COMPLETION; - Irp->Tail.Overlay.Thread = PsGetCurrentThread(); - Irp->UserEvent = &FileObject->Event; - Irp->UserIosb = IoStatusBlock; - Irp->MdlAddress = NULL; - Irp->PendingReturned = FALSE; - Irp->UserEvent = NULL; - Irp->Cancel = FALSE; - Irp->CancelRoutine = NULL; - Irp->Tail.Overlay.AuxiliaryBuffer = NULL; - - /* - * Get the stack location for the new - * IRP and prepare it. - */ - StackLoc = (PEXTENDED_IO_STACK_LOCATION)IoGetNextIrpStackLocation(Irp); - StackLoc->Control = 0; - StackLoc->FileObject = FileObject; - - switch (CreateFileType) - { - default: - case CreateFileTypeNone: - StackLoc->MajorFunction = IRP_MJ_CREATE; - StackLoc->Flags = Options; - StackLoc->Parameters.Create.EaLength = SystemEaBuffer != NULL ? EaLength : 0; - StackLoc->Flags |= !(ObjectAttributes->Attributes & OBJ_CASE_INSENSITIVE) ? SL_CASE_SENSITIVE: 0; - break; - - case CreateFileTypeNamedPipe: - StackLoc->MajorFunction = IRP_MJ_CREATE_NAMED_PIPE; - StackLoc->Parameters.CreatePipe.Parameters = ExtraCreateParameters; - break; - - case CreateFileTypeMailslot: - StackLoc->MajorFunction = IRP_MJ_CREATE_MAILSLOT; - StackLoc->Parameters.CreateMailslot.Parameters = ExtraCreateParameters; - break; - } - - /* Set the common data */ - Irp->Overlay.AllocationSize = SafeAllocationSize; - Irp->AssociatedIrp.SystemBuffer = SystemEaBuffer; - StackLoc->Parameters.Create.Options = (CreateDisposition << 24) | (CreateOptions & 0x00FFFFFF); - StackLoc->Parameters.Create.FileAttributes = FileAttributes; - StackLoc->Parameters.Create.ShareAccess = ShareAccess; - StackLoc->Parameters.Create.SecurityContext = &SecurityContext; - - /* - * Now call the driver and - * possibly wait if it can - * not complete the request - * immediately. - */ - Status = IofCallDriver(FileObject->DeviceObject, Irp ); - DPRINT("Status :%x\n", Status); - - if (Status == STATUS_PENDING) - { - KeWaitForSingleObject(&FileObject->Event, - Executive, - AccessMode, - FALSE, - NULL); - Status = IoStatusBlock->Status; - } - else - { + + SeCreateAccessState(&AccessState, &AuxData, FILE_ALL_ACCESS, NULL); + SecurityContext.SecurityQos = NULL; /* ?? */ + SecurityContext.AccessState = &AccessState; + SecurityContext.DesiredAccess = DesiredAccess; + SecurityContext.FullCreateOptions = CreateOptions; + + KeInitializeEvent(&FileObject->Lock, SynchronizationEvent, TRUE); + KeInitializeEvent(&FileObject->Event, NotificationEvent, FALSE); + + Irp = IoAllocateIrp(FileObject->DeviceObject->StackSize, FALSE); + if (!Irp) + { + ZwClose(LocalHandle); + return STATUS_UNSUCCESSFUL; + } + + /* Now set the IRP data */ + Irp->Tail.Overlay.OriginalFileObject = FileObject; + Irp->RequestorMode = AccessMode; + Irp->Flags = IRP_CREATE_OPERATION | + IRP_SYNCHRONOUS_API | + IRP_DEFER_IO_COMPLETION; + Irp->Tail.Overlay.Thread = PsGetCurrentThread(); + Irp->UserEvent = &FileObject->Event; + Irp->UserIosb = IoStatusBlock; + Irp->MdlAddress = NULL; + Irp->PendingReturned = FALSE; + Irp->UserEvent = NULL; + Irp->Cancel = FALSE; + Irp->CancelRoutine = NULL; + Irp->Tail.Overlay.AuxiliaryBuffer = NULL; + + StackLoc = (PEXTENDED_IO_STACK_LOCATION)IoGetNextIrpStackLocation(Irp); + StackLoc->Control = 0; + StackLoc->FileObject = FileObject; + + switch (CreateFileType) + { + default: + case CreateFileTypeNone: + StackLoc->MajorFunction = IRP_MJ_CREATE; + StackLoc->Flags = Options; + StackLoc->Parameters.Create.EaLength = SystemEaBuffer != NULL ? EaLength : 0; + StackLoc->Flags |= !(ObjectAttributes->Attributes & OBJ_CASE_INSENSITIVE) ? SL_CASE_SENSITIVE: 0; + break; + + case CreateFileTypeNamedPipe: + StackLoc->MajorFunction = IRP_MJ_CREATE_NAMED_PIPE; + StackLoc->Parameters.CreatePipe.Parameters = ExtraCreateParameters; + break; + + case CreateFileTypeMailslot: + StackLoc->MajorFunction = IRP_MJ_CREATE_MAILSLOT; + StackLoc->Parameters.CreateMailslot.Parameters = ExtraCreateParameters; + break; + } + + /* Set the common data */ + Irp->Overlay.AllocationSize = SafeAllocationSize; + Irp->AssociatedIrp.SystemBuffer = SystemEaBuffer; + StackLoc->Parameters.Create.Options = (CreateDisposition << 24) | (CreateOptions & 0x00FFFFFF); + StackLoc->Parameters.Create.FileAttributes = FileAttributes; + StackLoc->Parameters.Create.ShareAccess = ShareAccess; + StackLoc->Parameters.Create.SecurityContext = &SecurityContext; + + Status = IofCallDriver(FileObject->DeviceObject, Irp ); + if (Status == STATUS_PENDING) + { + KeWaitForSingleObject(&FileObject->Event, + Executive, + AccessMode, + FALSE, + NULL); + Status = IoStatusBlock->Status; + } + else + { /* We'll have to complete it ourselves */ ASSERT(!Irp->PendingReturned); KeRaiseIrql(APC_LEVEL, &OldIrql); @@ -1144,38 +1039,34 @@ (PVOID*)&FileObject, &NormalContext); KeLowerIrql(OldIrql); - } - if (!NT_SUCCESS(Status)) - { - DPRINT("Failing create request with status %x\n", Status); - FileObject->DeviceObject = NULL; - FileObject->Vpb = NULL; - ObDereferenceObject(FileObject); - } - else - { - _SEH_TRY - { - *FileHandle = LocalHandle; - } - _SEH_HANDLE - { - Status = _SEH_GetExceptionCode(); - } - _SEH_END; - } - - /* cleanup EABuffer if captured */ - if(AccessMode != KernelMode && SystemEaBuffer != NULL) - { - ExFreePool(SystemEaBuffer); - } - - ASSERT_IRQL(PASSIVE_LEVEL); - - DPRINT("Finished IoCreateFile() (*FileHandle) 0x%p\n", (*FileHandle)); - - return Status; + } + + if (!NT_SUCCESS(Status)) + { + FileObject->DeviceObject = NULL; + FileObject->Vpb = NULL; + ObDereferenceObject(FileObject); + } + else + { + _SEH_TRY + { + *FileHandle = LocalHandle; + } + _SEH_HANDLE + { + Status = _SEH_GetExceptionCode(); + } + _SEH_END; + } + + /* cleanup EABuffer if captured */ + if (AccessMode != KernelMode && (SystemEaBuffer)) + { + ExFreePool(SystemEaBuffer); + } + + return Status; } /*

1 0

[greatlrd] 22867: [AUDIT] after looking through code no signs of reverse engineering were found. Almost ever thing is almost complete from djgpp, few funcion is writen by us
by greatlrd＠svn.reactos.org 05 Jul '06

05 Jul '06

Author: greatlrd Date: Wed Jul 5 23:09:29 2006 New Revision: 22867 URL: http://svn.reactos.org/svn/reactos?rev=22867&view=rev Log: [AUDIT] after looking through code no signs of reverse engineering were found. Almost ever thing is almost complete from djgpp, few funcion is writen by us Modified: trunk/reactos/lib/crt/stdlib/qsort.c (props changed) trunk/reactos/lib/crt/stdlib/rand.c (props changed) trunk/reactos/lib/crt/stdlib/rot.c (props changed) trunk/reactos/lib/crt/stdlib/senv.c (props changed) trunk/reactos/lib/crt/stdlib/splitp.c (props changed) trunk/reactos/lib/crt/stdlib/strtod.c (props changed) trunk/reactos/lib/crt/stdlib/strtol.c (props changed) trunk/reactos/lib/crt/stdlib/strtold.c (props changed) trunk/reactos/lib/crt/stdlib/strtoul.c (props changed) trunk/reactos/lib/crt/stdlib/strtoull.c (props changed) trunk/reactos/lib/crt/stdlib/wcstod.c (props changed) trunk/reactos/lib/crt/stdlib/wcstol.c (props changed) trunk/reactos/lib/crt/stdlib/wcstom.c (props changed) trunk/reactos/lib/crt/stdlib/wcstomb.c (props changed) trunk/reactos/lib/crt/stdlib/wcstombs.c (props changed) trunk/reactos/lib/crt/stdlib/wcstoul.c (props changed) trunk/reactos/lib/crt/stdlib/wctomb.c (props changed) trunk/reactos/lib/crt/stdlib/witow.c (props changed) Propchange: trunk/reactos/lib/crt/stdlib/qsort.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/rand.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/rot.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/senv.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/splitp.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/strtod.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/strtol.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/strtold.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/strtoul.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/strtoull.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/wcstod.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/wcstol.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/wcstom.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/wcstomb.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/wcstombs.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/wcstoul.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/wctomb.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/stdlib/witow.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -*

1 0

[greatlrd] 22866: [AUDIT] after looking through code no signs of reverse engineering were found. 1. Follow fix need be done remove hardcode exe, cmd extions in wstart* use PATHEXT to found right excutions.
by greatlrd＠svn.reactos.org 05 Jul '06

05 Jul '06

Author: greatlrd Date: Wed Jul 5 22:57:25 2006 New Revision: 22866 URL: http://svn.reactos.org/svn/reactos?rev=22866&view=rev Log: [AUDIT] after looking through code no signs of reverse engineering were found. 1. Follow fix need be done remove hardcode exe, cmd extions in wstart* use PATHEXT to found right excutions. Modified: trunk/reactos/lib/crt/sys_stat/fstat.c (props changed) trunk/reactos/lib/crt/sys_stat/fstati64.c (props changed) trunk/reactos/lib/crt/sys_stat/futime.c (props changed) trunk/reactos/lib/crt/sys_stat/stat.c (props changed) trunk/reactos/lib/crt/sys_stat/systime.c (props changed) trunk/reactos/lib/crt/sys_stat/wstat.c (props changed) Propchange: trunk/reactos/lib/crt/sys_stat/fstat.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/sys_stat/fstati64.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/sys_stat/futime.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/sys_stat/stat.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/sys_stat/systime.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/sys_stat/wstat.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -*

1 0

[greatlrd] 22865: [AUDIT] after looking through code no signs of reverse engineering were found. Alot stuff taken from wine, it contain hard code string like date, contry names, that need to move to msvcrt dll rc file
by greatlrd＠svn.reactos.org 05 Jul '06

05 Jul '06

Author: greatlrd Date: Wed Jul 5 22:49:03 2006 New Revision: 22865 URL: http://svn.reactos.org/svn/reactos?rev=22865&view=rev Log: [AUDIT] after looking through code no signs of reverse engineering were found. Alot stuff taken from wine, it contain hard code string like date, contry names, that need to move to msvcrt dll rc file Modified: trunk/reactos/lib/crt/locale/locale.c (props changed) Propchange: trunk/reactos/lib/crt/locale/locale.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -*

1 0

[greatlrd] 22864: [AUDIT] after looking through code no signs of reverse engineering were found. Almost all code is from djgpp, few functions is from us.
by greatlrd＠svn.reactos.org 05 Jul '06

05 Jul '06

Author: greatlrd Date: Wed Jul 5 22:43:17 2006 New Revision: 22864 URL: http://svn.reactos.org/svn/reactos?rev=22864&view=rev Log: [AUDIT] after looking through code no signs of reverse engineering were found. Almost all code is from djgpp, few functions is from us. Modified: trunk/reactos/lib/crt/time/clock.c (props changed) trunk/reactos/lib/crt/time/ctime.c (props changed) trunk/reactos/lib/crt/time/difftime.c (props changed) trunk/reactos/lib/crt/time/ftime.c (props changed) trunk/reactos/lib/crt/time/posixrul.h (props changed) trunk/reactos/lib/crt/time/strdate.c (props changed) trunk/reactos/lib/crt/time/strftime.c (props changed) trunk/reactos/lib/crt/time/strtime.c (props changed) trunk/reactos/lib/crt/time/time.c (props changed) trunk/reactos/lib/crt/time/tz_vars.c (props changed) trunk/reactos/lib/crt/time/tzfile.h (props changed) trunk/reactos/lib/crt/time/wctime.c (props changed) trunk/reactos/lib/crt/time/wstrdate.c (props changed) trunk/reactos/lib/crt/time/wstrtime.c (props changed) Propchange: trunk/reactos/lib/crt/time/clock.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/time/ctime.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/time/difftime.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/time/ftime.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/time/posixrul.h ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/time/strdate.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/time/strftime.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/time/strtime.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/time/time.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/time/tz_vars.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/time/tzfile.h ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/time/wctime.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/time/wstrdate.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/time/wstrtime.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -*

1 0

[greatlrd] 22863: [AUDIT] after looking through code no signs of reverse engineering were found.
by greatlrd＠svn.reactos.org 05 Jul '06

05 Jul '06

Author: greatlrd Date: Wed Jul 5 22:36:44 2006 New Revision: 22863 URL: http://svn.reactos.org/svn/reactos?rev=22863&view=rev Log: [AUDIT] after looking through code no signs of reverse engineering were found. Modified: trunk/reactos/lib/crt/except/abnorter.c (props changed) trunk/reactos/lib/crt/except/exhand2.c (props changed) trunk/reactos/lib/crt/except/matherr.c (props changed) trunk/reactos/lib/crt/except/seh.s (props changed) trunk/reactos/lib/crt/except/unwind.c (props changed) trunk/reactos/lib/crt/except/xcptfil.c (props changed) Propchange: trunk/reactos/lib/crt/except/abnorter.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/except/exhand2.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/except/matherr.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/except/seh.s ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/except/unwind.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/except/xcptfil.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -*

1 0

[greatlrd] 22862: [AUDIT] after looking through code no signs of reverse engineering were found, and most code is base on djgpp
by greatlrd＠svn.reactos.org 05 Jul '06

05 Jul '06

Author: greatlrd Date: Wed Jul 5 22:34:01 2006 New Revision: 22862 URL: http://svn.reactos.org/svn/reactos?rev=22862&view=rev Log: [AUDIT] after looking through code no signs of reverse engineering were found, and most code is base on djgpp Modified: trunk/reactos/lib/crt/direct/chdir.c (props changed) trunk/reactos/lib/crt/direct/chdrive.c (props changed) trunk/reactos/lib/crt/direct/getcwd.c (props changed) trunk/reactos/lib/crt/direct/getdcwd.c (props changed) trunk/reactos/lib/crt/direct/getdfree.c (props changed) trunk/reactos/lib/crt/direct/getdrive.c (props changed) trunk/reactos/lib/crt/direct/mkdir.c (props changed) trunk/reactos/lib/crt/direct/rmdir.c (props changed) trunk/reactos/lib/crt/direct/wchdir.c (props changed) trunk/reactos/lib/crt/direct/wgetcwd.c (props changed) trunk/reactos/lib/crt/direct/wgetdcwd.c (props changed) trunk/reactos/lib/crt/direct/wmkdir.c (props changed) trunk/reactos/lib/crt/direct/wrmdir.c (props changed) Propchange: trunk/reactos/lib/crt/direct/chdir.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/direct/chdrive.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/direct/getcwd.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/direct/getdcwd.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/direct/getdfree.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/direct/getdrive.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/direct/mkdir.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/direct/rmdir.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/direct/wchdir.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/direct/wgetcwd.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/direct/wgetdcwd.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/direct/wmkdir.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -* Propchange: trunk/reactos/lib/crt/direct/wrmdir.c ------------------------------------------------------------------------------ --- svn:needs-lock (original) +++ svn:needs-lock (removed) @@ -1,1 +1,0 @@ -*

1 0

[gedmurphy] 22861: clean up code and use HeapAlloc + fix memory leaks
by gedmurphy＠svn.reactos.org 05 Jul '06

05 Jul '06

Author: gedmurphy Date: Wed Jul 5 19:29:37 2006 New Revision: 22861 URL: http://svn.reactos.org/svn/reactos?rev=22861&view=rev Log: clean up code and use HeapAlloc + fix memory leaks Modified: trunk/reactos/base/applications/network/ipconfig/ipconfig.c Modified: trunk/reactos/base/applications/network/ipconfig/ipconfig.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/network/… ============================================================================== --- trunk/reactos/base/applications/network/ipconfig/ipconfig.c (original) +++ trunk/reactos/base/applications/network/ipconfig/ipconfig.c Wed Jul 5 19:29:37 2006 @@ -1,15 +1,9 @@ /* - * ReactOS Win32 Applications - * Copyright (C) 2005 ReactOS Team - * - * COPYRIGHT: See COPYING in the top level directory - * PROJECT: ReactOS arp utility + * PROJECT: ReactOS ipconfig utility + * LICENSE: GPL - See COPYING in the top level directory * FILE: apps/utils/net/ipconfig/ipconfig.c - * PURPOSE: - * PROGRAMMERS: Ged Murphy (gedmurphy(a)gmail.com) - * REVISIONS: - * GM 14/09/05 Created - * + * PURPOSE: Display IP info for net adapters + * PROGRAMMERS: Copyright 2005 - 2006 Ged Murphy (gedmurphy(a)gmail.com) */ /* * TODO: @@ -17,36 +11,32 @@ * implement flushdns, registerdns, displaydns, showclassid, setclassid * allow globbing on adapter names */ + #define WIN32_LEAN_AND_MEAN #include <windows.h> #include <stdio.h> -#include <stdlib.h> +#include <tchar.h> #include <time.h> -#include <tchar.h> -#include <string.h> -#include <ctype.h> -#include <winsock2.h> #include <iphlpapi.h> -#define UNICODE -#define _UNICODE - - +HANDLE ProcessHeap; LPCTSTR GetNodeTypeName(UINT NodeType) { - switch (NodeType) { - case 1: return _T("Broadcast"); - case 2: return _T("Peer To Peer"); - case 4: return _T("Mixed"); - case 8: return _T("Hybrid"); + switch (NodeType) + { + case 1: return _T("Broadcast"); + case 2: return _T("Peer To Peer"); + case 4: return _T("Mixed"); + case 8: return _T("Hybrid"); default : return _T("unknown"); } } LPCTSTR GetInterfaceTypeName(UINT InterfaceType) { - switch (InterfaceType) { + switch (InterfaceType) + { case MIB_IF_TYPE_OTHER: return _T("Other Type Of Adapter"); case MIB_IF_TYPE_ETHERNET: return _T("Ethernet Adapter"); case MIB_IF_TYPE_TOKENRING: return _T("Token Ring Adapter"); @@ -54,7 +44,7 @@ case MIB_IF_TYPE_PPP: return _T("PPP Adapter"); case MIB_IF_TYPE_LOOPBACK: return _T("Loopback Adapter"); case MIB_IF_TYPE_SLIP: return _T("SLIP Adapter"); - default: return _T("unknown"); + default: return _T("unknown"); } } @@ -87,62 +77,83 @@ _tprintf(_T("%s"), (LPTSTR)lpMsgBuf); LocalFree(lpMsgBuf); - /* return number of TCHAR's stored in output buffer - * excluding '\0' - as FormatMessage does*/ return RetVal; } else return 0; } -INT ShowInfo(BOOL bAll) +VOID ShowInfo(BOOL bAll) { PIP_ADAPTER_INFO pAdapterInfo = NULL; PIP_ADAPTER_INFO pAdapter = NULL; - ULONG adaptOutBufLen; - - PFIXED_INFO pFixedInfo; - ULONG netOutBufLen; + ULONG adaptOutBufLen = 0; + + PFIXED_INFO pFixedInfo = NULL; PIP_ADDR_STRING pIPAddr = NULL; - - DWORD ErrRet = 0; + ULONG netOutBufLen = 0; + + DWORD ErrRet = 0; /* assign memory for call to GetNetworkParams */ - pFixedInfo = (FIXED_INFO *) GlobalAlloc( GPTR, sizeof( FIXED_INFO ) ); - netOutBufLen = sizeof(FIXED_INFO); + pFixedInfo = (FIXED_INFO *)HeapAlloc(ProcessHeap, 0, sizeof(FIXED_INFO)); + if (pFixedInfo == NULL) + { + _tprintf(_T("memory allocation error")); + return; + } + + /* set required buffer size */ + if(GetNetworkParams(pFixedInfo, &netOutBufLen) == ERROR_BUFFER_OVERFLOW) + { + HeapFree(ProcessHeap, 0, pFixedInfo); + pFixedInfo = (FIXED_INFO *)HeapAlloc(ProcessHeap, 0, netOutBufLen); + if (pFixedInfo == NULL) + { + _tprintf(_T("memory allocation error")); + return; + } + } + + if ((ErrRet = GetNetworkParams(pFixedInfo, &netOutBufLen)) != NO_ERROR) + { + _tprintf(_T("GetNetworkParams failed : ")); + DoFormatMessage(ErrRet); + HeapFree(ProcessHeap, 0, pFixedInfo); + return; + } + + /* assign memory for call to GetAdapterInfo */ - pAdapterInfo = (IP_ADAPTER_INFO *) malloc( sizeof(IP_ADAPTER_INFO) ); - adaptOutBufLen = sizeof(IP_ADAPTER_INFO); + pAdapterInfo = (IP_ADAPTER_INFO *)HeapAlloc(ProcessHeap, 0, sizeof(IP_ADAPTER_INFO)); + if (pAdapterInfo == NULL) + { + _tprintf(_T("memory allocation error")); + return; + } /* set required buffer size */ - if(GetNetworkParams(pFixedInfo, &netOutBufLen) == ERROR_BUFFER_OVERFLOW) - { - GlobalFree(pFixedInfo); - pFixedInfo = (FIXED_INFO *) GlobalAlloc(GPTR, netOutBufLen); - } - - /* set required buffer size */ - if (GetAdaptersInfo( pAdapterInfo, &adaptOutBufLen) == ERROR_BUFFER_OVERFLOW) - { - free(pAdapterInfo); - pAdapterInfo = (IP_ADAPTER_INFO *) malloc (adaptOutBufLen); + if (GetAdaptersInfo( pAdapterInfo, &adaptOutBufLen) == ERROR_BUFFER_OVERFLOW) + { + HeapFree(ProcessHeap, 0, pAdapterInfo); + pAdapterInfo = (IP_ADAPTER_INFO *)HeapAlloc(ProcessHeap, 0, adaptOutBufLen); + if (pAdapterInfo == NULL) + { + _tprintf(_T("memory allocation error")); + return; + } } if ((ErrRet = GetAdaptersInfo(pAdapterInfo, &adaptOutBufLen)) != NO_ERROR) - { - _tprintf(_T("GetAdaptersInfo failed : ")); - DoFormatMessage(ErrRet); - return EXIT_FAILURE; - } - - if ((ErrRet = GetNetworkParams(pFixedInfo, &netOutBufLen)) != NO_ERROR) - { - _tprintf(_T("GetNetworkParams failed : ")); - DoFormatMessage(ErrRet); - return EXIT_FAILURE; - } - + { + _tprintf(_T("GetAdaptersInfo failed : ")); + DoFormatMessage(ErrRet); + HeapFree(ProcessHeap, 0, pAdapterInfo); + return ; + } + + pAdapter = pAdapterInfo; _tprintf(_T("\nReactOS IP Configuration\n\n")); @@ -163,11 +174,10 @@ _tprintf(_T("\tDNS Suffix Search List. . . . . . : %s\n"), pFixedInfo->DomainName); } - while (pAdapter) - { - + while (pAdapter) + { _tprintf(_T("\n%s ...... : \n\n"), GetInterfaceTypeName(pAdapter->Type)); - + /* check if the adapter is connected to the media */ if (_tcscmp(pAdapter->IpAddressList.IpAddress.String, "0.0.0.0") == 0) { @@ -175,7 +185,7 @@ pAdapter = pAdapter->Next; continue; } - + _tprintf(_T("\tConnection-specific DNS Suffix. . : %s\n"), pFixedInfo->DomainName); if (bAll) @@ -218,39 +228,50 @@ } } _tprintf(_T("\n")); - - pAdapter = pAdapter->Next; - - } - - return 0; -} - -INT Release(TCHAR Index) + + pAdapter = pAdapter->Next; + + } + + HeapFree(ProcessHeap, 0, pFixedInfo); + HeapFree(ProcessHeap, 0, pAdapterInfo); +} + +VOID Release(LPTSTR Index) { IP_ADAPTER_INDEX_MAP AdapterInfo; DWORD dwRetVal = 0; /* if interface is not given, query GetInterfaceInfo */ - if (Index == (TCHAR)NULL) - { - PIP_INTERFACE_INFO pInfo; - ULONG ulOutBufLen; - pInfo = (IP_INTERFACE_INFO *) malloc(sizeof(IP_INTERFACE_INFO)); - ulOutBufLen = 0; + if (Index == NULL) + { + PIP_INTERFACE_INFO pInfo = NULL; + ULONG ulOutBufLen = 0; + + pInfo = (IP_INTERFACE_INFO *)HeapAlloc(ProcessHeap, 0, sizeof(IP_INTERFACE_INFO)); + if (pInfo == NULL) + { + _tprintf(_T("memory allocation error")); + return; + } /* Make an initial call to GetInterfaceInfo to get * the necessary size into the ulOutBufLen variable */ if ( GetInterfaceInfo(pInfo, &ulOutBufLen) == ERROR_INSUFFICIENT_BUFFER) { - GlobalFree(pInfo); - pInfo = (IP_INTERFACE_INFO *) malloc (ulOutBufLen); + HeapFree(ProcessHeap, 0, pInfo); + pInfo = (IP_INTERFACE_INFO *)HeapAlloc(ProcessHeap, 0, ulOutBufLen); + if (pInfo == NULL) + { + _tprintf(_T("memory allocation error")); + return; + } } /* Make a second call to GetInterfaceInfo to get the actual data we want */ if ((dwRetVal = GetInterfaceInfo(pInfo, &ulOutBufLen)) == NO_ERROR ) { - AdapterInfo = pInfo->Adapter[0]; + CopyMemory(&AdapterInfo, &pInfo->Adapter[0], sizeof(IP_ADAPTER_INDEX_MAP)); _tprintf(_T("name - %S\n"), pInfo->Adapter[0].Name); } else @@ -258,11 +279,14 @@ _tprintf(_T("\nGetInterfaceInfo failed : ")); DoFormatMessage(dwRetVal); } + + HeapFree(ProcessHeap, 0, pInfo); } else { ; - /* we need to be able to release connections by name with support for globbing + /* FIXME: + * we need to be able to release connections by name with support for globbing * i.e. ipconfig /release Eth* will release all cards starting with Eth... * ipconfig /release *con* will release all cards with 'con' in their name */ @@ -275,47 +299,62 @@ _tprintf(_T("\nAn error occured while releasing interface %s : "), _T("*name*")); DoFormatMessage(dwRetVal); } - return 0; -} - - - - -INT Renew(TCHAR Index) + +} + + + + +VOID Renew(LPTSTR Index) { IP_ADAPTER_INDEX_MAP AdapterInfo; DWORD dwRetVal = 0; /* if interface is not given, query GetInterfaceInfo */ - if (Index == (TCHAR)NULL) + if (Index == NULL) { PIP_INTERFACE_INFO pInfo; - ULONG ulOutBufLen; - pInfo = (IP_INTERFACE_INFO *) malloc(sizeof(IP_INTERFACE_INFO)); - ulOutBufLen = 0; + ULONG ulOutBufLen = 0; + + pInfo = (IP_INTERFACE_INFO *)HeapAlloc(ProcessHeap, 0, sizeof(IP_INTERFACE_INFO)); + if (pInfo == NULL) + { + _tprintf(_T("memory allocation error")); + return; + } /* Make an initial call to GetInterfaceInfo to get * the necessary size into the ulOutBufLen variable */ if ( GetInterfaceInfo(pInfo, &ulOutBufLen) == ERROR_INSUFFICIENT_BUFFER) { - GlobalFree(pInfo); - pInfo = (IP_INTERFACE_INFO *) malloc (ulOutBufLen); + HeapFree(ProcessHeap, 0, pInfo); + pInfo = (IP_INTERFACE_INFO *)HeapAlloc(ProcessHeap, 0, ulOutBufLen); + if (pInfo == NULL) + { + _tprintf(_T("memory allocation error")); + return; + } } /* Make a second call to GetInterfaceInfo to get the actual data we want */ if ((dwRetVal = GetInterfaceInfo(pInfo, &ulOutBufLen)) == NO_ERROR ) { - AdapterInfo = pInfo->Adapter[0]; + CopyMemory(&AdapterInfo, &pInfo->Adapter[0], sizeof(IP_ADAPTER_INDEX_MAP)); _tprintf(_T("name - %S\n"), pInfo->Adapter[0].Name); - } else { + } + else + { _tprintf(_T("\nGetInterfaceInfo failed : ")); DoFormatMessage(dwRetVal); } + + HeapFree(ProcessHeap, 0, pInfo); } else { ; - /* we need to be able to renew connections by name with support for globbing + /* FIXME: + * we need to be able to renew connections by name with support for globbing * i.e. ipconfig /renew Eth* will renew all cards starting with Eth... * ipconfig /renew *con* will renew all cards with 'con' in their name */ @@ -328,48 +367,8 @@ _tprintf(_T("\nAn error occured while renew interface %s : "), _T("*name*")); DoFormatMessage(dwRetVal); } - return 0; -} - -/* temp func for testing purposes */ -VOID Info() -{ - // Declare and initialize variables - PIP_INTERFACE_INFO pInfo; - ULONG ulOutBufLen; - DWORD dwRetVal; - - pInfo = (IP_INTERFACE_INFO *) malloc( sizeof(IP_INTERFACE_INFO) ); - ulOutBufLen = sizeof(IP_INTERFACE_INFO); - dwRetVal = 0; - - - // Make an initial call to GetInterfaceInfo to get - // the necessary size in the ulOutBufLen variable - if ( GetInterfaceInfo(pInfo, &ulOutBufLen) == ERROR_INSUFFICIENT_BUFFER) - { - free(pInfo); - pInfo = (IP_INTERFACE_INFO *) malloc (ulOutBufLen); - } - - // Make a second call to GetInterfaceInfo to get - // the actual data we need - if ((dwRetVal = GetInterfaceInfo(pInfo, &ulOutBufLen)) == NO_ERROR ) - { - int i; - for (i=0; i<pInfo->NumAdapters; i++) - { - printf("\tAdapter Name: %S\n", pInfo->Adapter[i].Name); - printf("\tAdapter Index: %ld\n", pInfo->Adapter[i].Index); - printf("\tNum Adapters: %ld\n", pInfo->NumAdapters); - } - } - else - { - printf("GetInterfaceInfo failed.\n"); - DoFormatMessage(dwRetVal); - } -} +} + VOID Usage(VOID) @@ -426,6 +425,8 @@ BOOL DoShowclassid=FALSE; BOOL DoSetclassid=FALSE; + ProcessHeap = GetProcessHeap(); + /* Parse command line for options we have been given. */ if ( (argc > 1)&&(argv[1][0]=='/') ) { @@ -472,15 +473,15 @@ case 1: /* Default behaviour if no options are given*/ ShowInfo(FALSE); break; - case 2: /* Process all the options that take no paramiters */ + case 2: /* Process all the options that take no parameters */ if (DoUsage) Usage(); else if (DoAll) ShowInfo(TRUE); else if (DoRelease) - Release((TCHAR)NULL); + Release(NULL); else if (DoRenew) - Renew((TCHAR)NULL); + Renew(NULL); else if (DoFlushdns) _tprintf(_T("\nSorry /flushdns is not implemented yet\n")); else if (DoRegisterdns) @@ -490,7 +491,7 @@ else Usage(); break; - case 3: /* Process all the options that can have 1 paramiters */ + case 3: /* Process all the options that can have 1 parameter */ if (DoRelease) _tprintf(_T("\nSorry /release [adapter] is not implemented yet\n")); //Release(argv[2]); @@ -503,7 +504,7 @@ else Usage(); break; - case 4: /* Process all the options that can have 2 paramiters */ + case 4: /* Process all the options that can have 2 parameters */ if (DoSetclassid) _tprintf(_T("\nSorry /setclassid adapter [classid]is not implemented yet\n")); else @@ -515,3 +516,4 @@ return 0; } +

1 0

[hpoussin] 22860: Fix handling of dllname_ros.diff file
by hpoussin＠svn.reactos.org 05 Jul '06

05 Jul '06

Author: hpoussin Date: Wed Jul 5 13:14:28 2006 New Revision: 22860 URL: http://svn.reactos.org/svn/reactos?rev=22860&view=rev Log: Fix handling of dllname_ros.diff file Modified: trunk/tools/wineimport.cmd Modified: trunk/tools/wineimport.cmd URL: http://svn.reactos.org/svn/reactos/trunk/tools/wineimport.cmd?rev=22860&r1=… ============================================================================== --- trunk/tools/wineimport.cmd (original) +++ trunk/tools/wineimport.cmd Wed Jul 5 13:14:28 2006 @@ -120,6 +120,7 @@ GOTO :eof :merge +SETLOCAL ENABLEDELAYEDEXPANSION IF "%3" == "" GOTO help IF NOT EXIST "%2\ReactOS.rbuild" ( ECHO %2\ReactOS.rbuild doesn't exit. @@ -139,7 +140,7 @@ DEL "%WINE_ROS_DIR%\dll\win32\%2\%%f" ) ELSE IF NOT EXIST "wine\dlls\%2\%%f" ( SET WINE_FILE=%%f - IF NOT "%WINE_FILE:~-9" == "_ros.diff" ( + IF NOT "!WINE_FILE:~-9!" == "_ros.diff" ( svn.exe delete "%WINE_ROS_DIR%\dll\win32\%2\%%f" 2>NUL IF ERRORLEVEL 2 GOTO :helpsvn ) @@ -154,7 +155,9 @@ IF ERRORLEVEL 2 GOTO :helpsvn svn.exe propset svn:eol-style native "%WINE_ROS_DIR%\dll\win32\%2\%2_ros.diff" >NUL 2>NUL PUSHD "%WINE_ROS_DIR%" - patch.exe -p0 -N < "dll\win32\%2\%2_ros.diff" + PUSHD "dll\win32\%2" + patch.exe -p0 -N < "%2_ros.diff" + POPD POPD IF ERRORLEVEL 1 ( ECHO Error when executing patch.exe. Try to download the lastest version at

1 0

[ion] 22859: - Make IoCreateFile use deferred I/O completion. - Fixed a bug in TCP/IP that was making it complete IRP_MJ_CREATE with STATUS_SUCCESS yet with the IRP marked as pending.
by ion＠svn.reactos.org 05 Jul '06

05 Jul '06

Author: ion Date: Wed Jul 5 10:45:00 2006 New Revision: 22859 URL: http://svn.reactos.org/svn/reactos?rev=22859&view=rev Log: - Make IoCreateFile use deferred I/O completion. - Fixed a bug in TCP/IP that was making it complete IRP_MJ_CREATE with STATUS_SUCCESS yet with the IRP marked as pending. Modified: trunk/reactos/drivers/network/tcpip/tcpip/irp.c trunk/reactos/drivers/network/tcpip/tcpip/main.c trunk/reactos/ntoskrnl/io/iomgr/file.c Modified: trunk/reactos/drivers/network/tcpip/tcpip/irp.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/tcpip/tcpi… ============================================================================== --- trunk/reactos/drivers/network/tcpip/tcpip/irp.c (original) +++ trunk/reactos/drivers/network/tcpip/tcpip/irp.c Wed Jul 5 10:45:00 2006 @@ -18,7 +18,7 @@ } NTSTATUS IRPFinish( PIRP Irp, NTSTATUS Status ) { - TI_DbgPrint(MID_TRACE,("Called: Irp %x, Status %x Event %x\n", Irp, Status, Irp->UserEvent)); + DbgPrint("Called: Irp %x, Status %x Event %x\n", Irp, Status, Irp->UserEvent); #ifdef MEMTRACK UntrackFL( __FILE__, __LINE__, Irp ); Modified: trunk/reactos/drivers/network/tcpip/tcpip/main.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/tcpip/tcpi… ============================================================================== --- trunk/reactos/drivers/network/tcpip/tcpip/main.c (original) +++ trunk/reactos/drivers/network/tcpip/tcpip/main.c Wed Jul 5 10:45:00 2006 @@ -385,11 +385,7 @@ RIRP(Irp); - TI_DbgPrint(DEBUG_IRP, ("Called. DeviceObject is at (0x%X), IRP is at (0x%X).\n", DeviceObject, Irp)); - - IoMarkIrpPending(Irp); - Irp->IoStatus.Status = STATUS_PENDING; - Irp->IoStatus.Information = 0; + DbgPrint("Called. DeviceObject is at (0x%X), IRP is at (0x%X).\n", DeviceObject, Irp); IrpSp = IoGetCurrentIrpStackLocation(Irp); @@ -417,7 +413,7 @@ Status = STATUS_INVALID_DEVICE_REQUEST; } - TI_DbgPrint(DEBUG_IRP, ("Leaving. Status is (0x%X)\n", Status)); + DbgPrint("Leaving. Status is (0x%X)\n", Status); return IRPFinish( Irp, Status ); } Modified: trunk/reactos/ntoskrnl/io/iomgr/file.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/io/iomgr/file.c?r… ============================================================================== --- trunk/reactos/ntoskrnl/io/iomgr/file.c (original) +++ trunk/reactos/ntoskrnl/io/iomgr/file.c Wed Jul 5 10:45:00 2006 @@ -892,6 +892,9 @@ NTSTATUS Status = STATUS_SUCCESS; AUX_DATA AuxData; ACCESS_STATE AccessState; + KIRQL OldIrql; + PKNORMAL_ROUTINE NormalRoutine; + PVOID NormalContext; DPRINT("IoCreateFile(FileHandle 0x%p, DesiredAccess %x, " "ObjectAttributes 0x%p ObjectAttributes->ObjectName->Buffer %S)\n", @@ -1063,8 +1066,8 @@ Irp->Tail.Overlay.OriginalFileObject = FileObject; Irp->RequestorMode = AccessMode; Irp->Flags = IRP_CREATE_OPERATION | - IRP_SYNCHRONOUS_API;// | - //IRP_DEFER_IO_COMPLETION; + IRP_SYNCHRONOUS_API | + IRP_DEFER_IO_COMPLETION; Irp->Tail.Overlay.Thread = PsGetCurrentThread(); Irp->UserEvent = &FileObject->Event; Irp->UserIosb = IoStatusBlock; @@ -1130,21 +1133,18 @@ NULL); Status = IoStatusBlock->Status; } -#if 0 else { /* We'll have to complete it ourselves */ ASSERT(!Irp->PendingReturned); KeRaiseIrql(APC_LEVEL, &OldIrql); - FileObject->Event.Header.SignalState = 1; - if ((Irp->Flags & IRP_BUFFERED_IO) && (Irp->Flags & IRP_DEALLOCATE_BUFFER)) - { - ExFreePool(Irp->AssociatedIrp.SystemBuffer); - } - IoFreeIrp(Irp); + IopCompleteRequest(&Irp->Tail.Apc, + &NormalRoutine, + &NormalContext, + (PVOID*)&FileObject, + &NormalContext); KeLowerIrql(OldIrql); } -#endif if (!NT_SUCCESS(Status)) { DPRINT("Failing create request with status %x\n", Status);

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs