Ros-diffs

ros-diffs@reactos.org

81390 discussions

[sginsberg] 43341: - Fix DbgLoadImageSymbols -- it doesn't return anything. - Fix DbgQueryDebugFilterState -- it returns NTSTATUS, not BOOLEAN! Also fix the callers that depended on this.
by sginsberg＠svn.reactos.org 09 Oct '09

09 Oct '09

Author: sginsberg Date: Fri Oct 9 13:51:07 2009 New Revision: 43341 URL: http://svn.reactos.org/svn/reactos?rev=43341&view=rev Log: - Fix DbgLoadImageSymbols -- it doesn't return anything. - Fix DbgQueryDebugFilterState -- it returns NTSTATUS, not BOOLEAN! Also fix the callers that depended on this. Modified: trunk/reactos/include/ddk/winddk.h trunk/reactos/include/ndk/rtlfuncs.h trunk/reactos/lib/drivers/ip/network/routines.c trunk/reactos/lib/rtl/debug.c Modified: trunk/reactos/include/ddk/winddk.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/ddk/winddk.h?rev=4… ============================================================================== --- trunk/reactos/include/ddk/winddk.h [iso-8859-1] (original) +++ trunk/reactos/include/ddk/winddk.h [iso-8859-1] Fri Oct 9 13:51:07 2009 @@ -10317,8 +10317,16 @@ IN PCH Format, IN ...); -NTKERNELAPI -BOOLEAN +ULONG +NTAPI +DbgPrompt( + IN PCCH Prompt, + OUT PCH Response, + IN ULONG MaximumResponseLength +); + +NTKERNELAPI +NTSTATUS NTAPI DbgQueryDebugFilterState( IN ULONG ComponentId, Modified: trunk/reactos/include/ndk/rtlfuncs.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/ndk/rtlfuncs.h?rev… ============================================================================== --- trunk/reactos/include/ndk/rtlfuncs.h [iso-8859-1] (original) +++ trunk/reactos/include/ndk/rtlfuncs.h [iso-8859-1] Fri Oct 9 13:51:07 2009 @@ -2666,7 +2666,7 @@ VOID ); -NTSTATUS +VOID NTAPI DbgLoadImageSymbols( IN PANSI_STRING Name, Modified: trunk/reactos/lib/drivers/ip/network/routines.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/drivers/ip/network/rou… ============================================================================== --- trunk/reactos/lib/drivers/ip/network/routines.c [iso-8859-1] (original) +++ trunk/reactos/lib/drivers/ip/network/routines.c [iso-8859-1] Fri Oct 9 13:51:07 2009 @@ -94,8 +94,8 @@ UINT Length; PCHAR Buffer; - if (!DbgQueryDebugFilterState(DPFLTR_TCPIP_ID, DEBUG_PBUFFER | DPFLTR_MASK) || - !DbgQueryDebugFilterState(DPFLTR_TCPIP_ID, DEBUG_TCP | DPFLTR_MASK)) { + if (!NT_SUCCESS(DbgQueryDebugFilterState(DPFLTR_TCPIP_ID, DEBUG_PBUFFER | DPFLTR_MASK)) || + !NT_SUCCESS(DbgQueryDebugFilterState(DPFLTR_TCPIP_ID, DEBUG_TCP | DPFLTR_MASK))) { return; } @@ -139,8 +139,8 @@ PNDIS_BUFFER NextBuffer; PCHAR CharBuffer; - if (!DbgQueryDebugFilterState(DPFLTR_TCPIP_ID, DEBUG_PBUFFER | DPFLTR_MASK) || - !DbgQueryDebugFilterState(DPFLTR_TCPIP_ID, DEBUG_IP | DPFLTR_MASK)) { + if (!NT_SUCCESS(DbgQueryDebugFilterState(DPFLTR_TCPIP_ID, DEBUG_PBUFFER | DPFLTR_MASK)) || + !NT_SUCCESS(DbgQueryDebugFilterState(DPFLTR_TCPIP_ID, DEBUG_IP | DPFLTR_MASK))) { return; } Modified: trunk/reactos/lib/rtl/debug.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/debug.c?rev=43341&… ============================================================================== --- trunk/reactos/lib/rtl/debug.c [iso-8859-1] (original) +++ trunk/reactos/lib/rtl/debug.c [iso-8859-1] Fri Oct 9 13:51:07 2009 @@ -317,7 +317,7 @@ /* * @implemented */ -NTSTATUS +VOID NTAPI DbgLoadImageSymbols(IN PANSI_STRING Name, IN PVOID Base, @@ -346,7 +346,6 @@ /* Load the symbols */ DebugService2(Name, &SymbolInfo, BREAKPOINT_LOAD_SYMBOLS); - return STATUS_SUCCESS; } /*

1 0

[sginsberg] 43340: Actually define STATUS_ASSERTION_FAILURE (missed in 43291).
by sginsberg＠svn.reactos.org 09 Oct '09

09 Oct '09

Author: sginsberg Date: Fri Oct 9 13:27:41 2009 New Revision: 43340 URL: http://svn.reactos.org/svn/reactos?rev=43340&view=rev Log: Actually define STATUS_ASSERTION_FAILURE (missed in 43291). Modified: trunk/reactos/include/ddk/ntstatus.h trunk/reactos/include/psdk/ntstatus.h Modified: trunk/reactos/include/ddk/ntstatus.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/ddk/ntstatus.h?rev… ============================================================================== --- trunk/reactos/include/ddk/ntstatus.h [iso-8859-1] (original) +++ trunk/reactos/include/ddk/ntstatus.h [iso-8859-1] Fri Oct 9 13:27:41 2009 @@ -873,6 +873,7 @@ #define STATUS_PKINIT_CLIENT_FAILURE ((NTSTATUS)0xC000038CL) #define STATUS_SMARTCARD_CERT_EXPIRED ((NTSTATUS)0xC000038DL) #define STATUS_DRIVER_FAILED_PRIOR_UNLOAD ((NTSTATUS)0xC000038EL) +#define STATUS_ASSERTION_FAILURE ((NTSTATUS)0xC0000420L) #define STATUS_CALLBACK_POP_STACK ((NTSTATUS)0xC0000423L) #define STATUS_WOW_ASSERTION ((NTSTATUS)0xC0009898L) #define RPC_NT_INVALID_STRING_BINDING ((NTSTATUS)0xC0020001L) Modified: trunk/reactos/include/psdk/ntstatus.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/ntstatus.h?re… ============================================================================== --- trunk/reactos/include/psdk/ntstatus.h [iso-8859-1] (original) +++ trunk/reactos/include/psdk/ntstatus.h [iso-8859-1] Fri Oct 9 13:27:41 2009 @@ -922,6 +922,7 @@ #define STATUS_AUTHENTICATION_FIREWALL_FAILED ((NTSTATUS)0xC0000413) #define STATUS_VDM_DISALLOWED ((NTSTATUS)0xC0000414) #define STATUS_HUNG_DISPLAY_DRIVER_THREAD ((NTSTATUS)0xC0000415) +#define STATUS_ASSERTION_FAILURE ((NTSTATUS)0xC0000420L) #define STATUS_CALLBACK_POP_STACK ((NTSTATUS)0xC0000423) #define STATUS_WOW_ASSERTION ((NTSTATUS)0xC0009898)

1 0

[cgutman] 43339: - Fix some typos from my last commit - Remove more hacks from iphlpapi
by cgutman＠svn.reactos.org 09 Oct '09

09 Oct '09

Author: cgutman Date: Fri Oct 9 07:18:35 2009 New Revision: 43339 URL: http://svn.reactos.org/svn/reactos?rev=43339&view=rev Log: - Fix some typos from my last commit - Remove more hacks from iphlpapi Modified: trunk/reactos/dll/win32/iphlpapi/ipstats_reactos.c trunk/reactos/dll/win32/iphlpapi/route_reactos.c trunk/reactos/drivers/network/tcpip/include/info.h trunk/reactos/drivers/network/tcpip/include/router.h trunk/reactos/drivers/network/tcpip/tcpip/info.c trunk/reactos/drivers/network/tcpip/tcpip/ninfo.c trunk/reactos/lib/drivers/ip/network/router.c Modified: trunk/reactos/dll/win32/iphlpapi/ipstats_reactos.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/iphlpapi/ipstats… ============================================================================== --- trunk/reactos/dll/win32/iphlpapi/ipstats_reactos.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/iphlpapi/ipstats_reactos.c [iso-8859-1] Fri Oct 9 07:18:35 2009 @@ -423,7 +423,6 @@ { RouteTable *out_route_table; DWORD numRoutes = getNumRoutes(), routesAdded = 0; - IPSNMPInfo snmpInfo; TDIEntityID ent; HANDLE tcpFile; NTSTATUS status = openTcpFile( &tcpFile ); @@ -444,31 +443,28 @@ out_route_table->numRoutes = numRoutes; - for( i = 0; routesAdded < numRoutes; i++ ) { + for( i = 0; routesAdded < out_route_table->numRoutes; i++ ) { int j; IPRouteEntry *route_set; getNthIpEntity( tcpFile, i, &ent ); - tdiGetMibForIpEntity( tcpFile, &ent, &snmpInfo ); - - TRACE( "%d routes in instance %d\n", snmpInfo.ipsi_numroutes, i ); tdiGetRoutesForIpEntity( tcpFile, &ent, &route_set, &numRoutes ); - + if( !route_set ) { closeTcpFile( tcpFile ); HeapFree( GetProcessHeap(), 0, out_route_table ); return 0; } - TRACE("Route set returned\n"); + TRACE( "%d routes in instance %d\n", numRoutes, i ); #if 0 HexDump( route_set, sizeof( IPRouteEntry ) * snmpInfo.ipsi_numroutes ); #endif - for( j = 0; j < snmpInfo.ipsi_numroutes; j++ ) { + for( j = 0; j < numRoutes; j++ ) { int routeNum = j + routesAdded; out_route_table->routes[routeNum].dest = route_set[j].ire_dest; @@ -484,7 +480,7 @@ if( route_set ) tdiFreeThingSet( route_set ); - routesAdded += snmpInfo.ipsi_numroutes; + routesAdded += numRoutes; } closeTcpFile( tcpFile ); Modified: trunk/reactos/dll/win32/iphlpapi/route_reactos.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/iphlpapi/route_r… ============================================================================== --- trunk/reactos/dll/win32/iphlpapi/route_reactos.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/iphlpapi/route_reactos.c [iso-8859-1] Fri Oct 9 07:18:35 2009 @@ -64,14 +64,14 @@ TRACE("Called.\n"); if( NT_SUCCESS(status) ) { - status = getNthIpEntity( tcpFile, 0, &id ); + status = getNthIpEntity( tcpFile, pRoute->dwForwardIfIndex, &id ); if( NT_SUCCESS(status) ) { req.Req.ID.toi_class = INFO_CLASS_PROTOCOL; req.Req.ID.toi_type = INFO_TYPE_PROVIDER; req.Req.ID.toi_id = IP_MIB_ARPTABLE_ENTRY_ID; req.Req.ID.toi_entity.tei_instance = id.tei_instance; - req.Req.ID.toi_entity.tei_instance = CL_NL_ENTITY; + req.Req.ID.toi_entity.tei_entity = CL_NL_ENTITY; req.Req.BufferSize = sizeof(*rte); rte = (IPRouteEntry *)&req.Req.Buffer[0]; @@ -129,7 +129,7 @@ TRACE("Called.\n"); if( NT_SUCCESS(status) ) { - status = getNthIpEntity( tcpFile, 0, &id ); + status = getNthIpEntity( tcpFile, pRoute->dwForwardIfIndex, &id ); if( NT_SUCCESS(status) ) { req.Req.ID.toi_class = INFO_CLASS_PROTOCOL; Modified: trunk/reactos/drivers/network/tcpip/include/info.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/tcpip/incl… ============================================================================== --- trunk/reactos/drivers/network/tcpip/include/info.h [iso-8859-1] (original) +++ trunk/reactos/drivers/network/tcpip/include/info.h [iso-8859-1] Fri Oct 9 07:18:35 2009 @@ -152,13 +152,15 @@ PUINT BufferSize); TDI_STATUS InfoTdiQueryGetIPSnmpInfo( TDIEntityID ID, + PIP_INTERFACE IF, PNDIS_BUFFER Buffer, PUINT BufferSize ); -TDI_STATUS InfoTdiQueryGetRouteTable( PNDIS_BUFFER Buffer, +TDI_STATUS InfoTdiQueryGetRouteTable( PIP_INTERFACE IF, + PNDIS_BUFFER Buffer, PUINT BufferSize ); -TDI_STATUS InfoTdiSetRoute(PIPROUTE_ENTRY Route); +TDI_STATUS InfoTdiSetRoute(PIP_INTERFACE IF, PIPROUTE_ENTRY Route); TDI_STATUS InfoTdiQueryGetArptableMIB(TDIEntityID ID, PIP_INTERFACE Interface, Modified: trunk/reactos/drivers/network/tcpip/include/router.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/tcpip/incl… ============================================================================== --- trunk/reactos/drivers/network/tcpip/include/router.h [iso-8859-1] (original) +++ trunk/reactos/drivers/network/tcpip/include/router.h [iso-8859-1] Fri Oct 9 07:18:35 2009 @@ -43,9 +43,9 @@ NTSTATUS RouterShutdown( VOID); -UINT CountFIBs(VOID); +UINT CountFIBs(PIP_INTERFACE IF); -UINT CopyFIBs( PFIB_ENTRY Target ); +UINT CopyFIBs( PIP_INTERFACE IF, PFIB_ENTRY Target ); #endif /* __ROUTER_H */ Modified: trunk/reactos/drivers/network/tcpip/tcpip/info.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/tcpip/tcpi… ============================================================================== --- trunk/reactos/drivers/network/tcpip/tcpip/info.c [iso-8859-1] (original) +++ trunk/reactos/drivers/network/tcpip/tcpip/info.c [iso-8859-1] Fri Oct 9 07:18:35 2009 @@ -188,7 +188,10 @@ return TDI_INVALID_PARAMETER; else if (ID->toi_entity.tei_entity == CL_NL_ENTITY || ID->toi_entity.tei_entity == CO_NL_ENTITY) - return InfoTdiQueryGetIPSnmpInfo(ID->toi_entity, Buffer, BufferSize); + if ((EntityListContext = GetContext(ID->toi_entity))) + return InfoTdiQueryGetIPSnmpInfo(ID->toi_entity, EntityListContext, Buffer, BufferSize); + else + return TDI_INVALID_PARAMETER; else return TDI_INVALID_PARAMETER; @@ -214,7 +217,10 @@ return TDI_INVALID_PARAMETER; else if (ID->toi_entity.tei_entity == CO_NL_ENTITY || ID->toi_entity.tei_entity == CL_NL_ENTITY) - return InfoTdiQueryGetRouteTable(Buffer, BufferSize); + if ((EntityListContext = GetContext(ID->toi_entity))) + return InfoTdiQueryGetRouteTable(EntityListContext, Buffer, BufferSize); + else + return TDI_INVALID_PARAMETER; else return TDI_INVALID_PARAMETER; @@ -251,20 +257,25 @@ * Status of operation */ { + PVOID EntityListContext; + switch (ID->toi_class) { case INFO_CLASS_PROTOCOL: switch (ID->toi_id) { case IP_MIB_ARPTABLE_ENTRY_ID: - if (ID->toi_id != INFO_TYPE_PROVIDER) + if (ID->toi_type != INFO_TYPE_PROVIDER) return TDI_INVALID_PARAMETER; if (ID->toi_entity.tei_entity != CL_NL_ENTITY && ID->toi_entity.tei_entity != CO_NL_ENTITY) return TDI_INVALID_PARAMETER; - return InfoTdiSetRoute((PIPROUTE_ENTRY)Buffer); + if ((EntityListContext = GetContext(ID->toi_entity))) + return InfoTdiSetRoute(EntityListContext, (PIPROUTE_ENTRY)Buffer); + else + return TDI_INVALID_PARAMETER; default: return TDI_INVALID_REQUEST; Modified: trunk/reactos/drivers/network/tcpip/tcpip/ninfo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/tcpip/tcpi… ============================================================================== --- trunk/reactos/drivers/network/tcpip/tcpip/ninfo.c [iso-8859-1] (original) +++ trunk/reactos/drivers/network/tcpip/tcpip/ninfo.c [iso-8859-1] Fri Oct 9 07:18:35 2009 @@ -16,16 +16,17 @@ /* Get IPRouteEntry s for each of the routes in the system */ -TDI_STATUS InfoTdiQueryGetRouteTable( PNDIS_BUFFER Buffer, PUINT BufferSize ) { +TDI_STATUS InfoTdiQueryGetRouteTable( PIP_INTERFACE IF, PNDIS_BUFFER Buffer, PUINT BufferSize ) { TDI_STATUS Status; KIRQL OldIrql; - UINT RtCount = CountFIBs(); + UINT RtCount = CountFIBs(IF); UINT Size = sizeof( IPROUTE_ENTRY ) * RtCount; PFIB_ENTRY RCache = exAllocatePool( NonPagedPool, sizeof( FIB_ENTRY ) * RtCount ), RCacheCur = RCache; PIPROUTE_ENTRY RouteEntries = exAllocatePool( NonPagedPool, Size ), RtCurrent = RouteEntries; + UINT i; TI_DbgPrint(DEBUG_INFO, ("Called, routes = %d, RCache = %08x\n", RtCount, RCache)); @@ -38,7 +39,7 @@ RtlZeroMemory( RouteEntries, Size ); - RtCount = CopyFIBs( RCache ); + RtCount = CopyFIBs( IF, RCache ); while( RtCurrent < RouteEntries + RtCount ) { ASSERT(RCacheCur->Router); @@ -66,13 +67,15 @@ RtCurrent->Metric1 )); TcpipAcquireSpinLock(&EntityListLock, &OldIrql); - for( RtCurrent->Index = EntityCount; - RtCurrent->Index > 0 && - RCacheCur->Router->Interface != - EntityList[RtCurrent->Index - 1].context; - RtCurrent->Index-- ); - - RtCurrent->Index = EntityList[RtCurrent->Index - 1].tei_instance; + for (i = 0; i < EntityCount; i++) + if (EntityList[i].context == IF) + break; + + if (i < EntityCount) + RtCurrent->Index = EntityList[i].tei_instance; + else + RtCurrent->Index = 0; + TcpipReleaseSpinLock(&EntityListLock, OldIrql); RtCurrent++; RCacheCur++; @@ -146,11 +149,12 @@ } TDI_STATUS InfoTdiQueryGetIPSnmpInfo( TDIEntityID ID, + PIP_INTERFACE IF, PNDIS_BUFFER Buffer, PUINT BufferSize ) { IPSNMP_INFO SnmpInfo; UINT IfCount = CountInterfaces(); - UINT RouteCount = CountFIBs(); + UINT RouteCount = CountFIBs(IF); TDI_STATUS Status = TDI_INVALID_REQUEST; TI_DbgPrint(DEBUG_INFO, ("Called.\n")); @@ -169,10 +173,9 @@ return Status; } -TDI_STATUS InfoTdiSetRoute(PIPROUTE_ENTRY Route) +TDI_STATUS InfoTdiSetRoute(PIP_INTERFACE IF, PIPROUTE_ENTRY Route) { IP_ADDRESS Address, Netmask, Router; - PNEIGHBOR_CACHE_ENTRY NCE; AddrInitIPv4( &Address, Route->Dest ); AddrInitIPv4( &Netmask, Route->Mask ); @@ -180,13 +183,8 @@ if( Route->Type == IP_ROUTE_TYPE_ADD ) { /* Add the route */ TI_DbgPrint(DEBUG_INFO,("Adding route (%s)\n", A2S(&Address))); - /* Find the existing route this belongs to */ - NCE = RouterGetRoute( &Router ); - if (!NCE) return TDI_INVALID_PARAMETER; - - /* Really add the route */ if (!RouterCreateRoute( &Address, &Netmask, &Router, - NCE->Interface, Route->Metric1)) + IF, Route->Metric1)) return TDI_NO_RESOURCES; return TDI_SUCCESS; Modified: trunk/reactos/lib/drivers/ip/network/router.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/drivers/ip/network/rou… ============================================================================== --- trunk/reactos/lib/drivers/ip/network/router.c [iso-8859-1] (original) +++ trunk/reactos/lib/drivers/ip/network/router.c [iso-8859-1] Fri Oct 9 07:18:35 2009 @@ -99,37 +99,41 @@ } -UINT CountFIBs() { +UINT CountFIBs(PIP_INTERFACE IF) { UINT FibCount = 0; PLIST_ENTRY CurrentEntry; PLIST_ENTRY NextEntry; - - /* Search the list and remove every FIB entry we find */ - CurrentEntry = FIBListHead.Flink; - while (CurrentEntry != &FIBListHead) { - NextEntry = CurrentEntry->Flink; + PFIB_ENTRY Current; + + CurrentEntry = FIBListHead.Flink; + while (CurrentEntry != &FIBListHead) { + NextEntry = CurrentEntry->Flink; + Current = CONTAINING_RECORD(CurrentEntry, FIB_ENTRY, ListEntry); + if (Current->Router->Interface == IF) + FibCount++; CurrentEntry = NextEntry; - FibCount++; } return FibCount; } -UINT CopyFIBs( PFIB_ENTRY Target ) { +UINT CopyFIBs( PIP_INTERFACE IF, PFIB_ENTRY Target ) { UINT FibCount = 0; PLIST_ENTRY CurrentEntry; PLIST_ENTRY NextEntry; PFIB_ENTRY Current; - /* Search the list and remove every FIB entry we find */ CurrentEntry = FIBListHead.Flink; while (CurrentEntry != &FIBListHead) { NextEntry = CurrentEntry->Flink; Current = CONTAINING_RECORD(CurrentEntry, FIB_ENTRY, ListEntry); - Target[FibCount] = *Current; + if (Current->Router->Interface == IF) + { + Target[FibCount] = *Current; + FibCount++; + } CurrentEntry = NextEntry; - FibCount++; } return FibCount;

1 0

[cgutman] 43338: - Rewrite IOCTL_TCP_QUERY_INFORMATION_EX/IOCTL_TCP_SET_INFORMATION_EX handling - Remove duplicate defines for the same IOCTL - Remove unneeded code - Fix a completely wrong implementation of ENTITY_TYPE_ID - Perform parameter checking - Remove several hacks in iphlpapi that worked around bugs in the previous implementation - We now support retrieving interface-specific routing tables and ARP tables
by cgutman＠svn.reactos.org 09 Oct '09

09 Oct '09

Author: cgutman Date: Fri Oct 9 04:40:38 2009 New Revision: 43338 URL: http://svn.reactos.org/svn/reactos?rev=43338&view=rev Log: - Rewrite IOCTL_TCP_QUERY_INFORMATION_EX/IOCTL_TCP_SET_INFORMATION_EX handling - Remove duplicate defines for the same IOCTL - Remove unneeded code - Fix a completely wrong implementation of ENTITY_TYPE_ID - Perform parameter checking - Remove several hacks in iphlpapi that worked around bugs in the previous implementation - We now support retrieving interface-specific routing tables and ARP tables Modified: trunk/reactos/dll/win32/iphlpapi/ifenum_reactos.c trunk/reactos/dll/win32/iphlpapi/iphlpapi_private.h trunk/reactos/dll/win32/iphlpapi/ipstats_reactos.c trunk/reactos/dll/win32/iphlpapi/route_reactos.c trunk/reactos/drivers/network/tcpip/include/info.h trunk/reactos/drivers/network/tcpip/include/tcpip.h trunk/reactos/drivers/network/tcpip/tcpip/iinfo.c trunk/reactos/drivers/network/tcpip/tcpip/info.c trunk/reactos/drivers/network/tcpip/tcpip/main.c trunk/reactos/drivers/network/tcpip/tcpip/ninfo.c Modified: trunk/reactos/dll/win32/iphlpapi/ifenum_reactos.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/iphlpapi/ifenum_… ============================================================================== --- trunk/reactos/dll/win32/iphlpapi/ifenum_reactos.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/iphlpapi/ifenum_reactos.c [iso-8859-1] Fri Oct 9 04:40:38 2009 @@ -323,36 +323,8 @@ loop_maybe, &entryInfo ); - return NT_SUCCESS(status) && (!entryInfo.ent.if_type || - entryInfo.ent.if_type == IFENT_SOFTWARE_LOOPBACK); -} - -NTSTATUS tdiGetEntityType( HANDLE tcpFile, TDIEntityID *ent, PULONG type ) { - TCP_REQUEST_QUERY_INFORMATION_EX req = TCP_REQUEST_QUERY_INFORMATION_INIT; - NTSTATUS status = STATUS_SUCCESS; - DWORD returnSize; - - TRACE("TdiGetEntityType(tcpFile %x,entityId %x)\n", - (DWORD)tcpFile, ent->tei_instance); - - req.ID.toi_class = INFO_CLASS_GENERIC; - req.ID.toi_type = INFO_TYPE_PROVIDER; - req.ID.toi_id = ENTITY_TYPE_ID; - req.ID.toi_entity.tei_entity = ent->tei_entity; - req.ID.toi_entity.tei_instance = ent->tei_instance; - - status = DeviceIoControl( tcpFile, - IOCTL_TCP_QUERY_INFORMATION_EX, - &req, - sizeof(req), - type, - sizeof(*type), - &returnSize, - NULL ); - - TRACE("TdiGetEntityType() => %08x %08x\n", *type, status); - - return (status ? STATUS_SUCCESS : STATUS_UNSUCCESSFUL); + return NT_SUCCESS(status) && + (entryInfo.ent.if_type == IFENT_SOFTWARE_LOOPBACK); } BOOL hasArp( HANDLE tcpFile, TDIEntityID *arp_maybe ) { @@ -374,9 +346,9 @@ sizeof(type), &returnSize, NULL ); - if( !NT_SUCCESS(status) ) return FALSE; - return type == AT_ENTITY; + + return (type & AT_ARP); } static NTSTATUS getInterfaceInfoSet( HANDLE tcpFile, @@ -386,7 +358,6 @@ TDIEntityID *entIDSet = 0; NTSTATUS status = tdiGetEntityIDSet( tcpFile, &entIDSet, &numEntities ); IFInfo *infoSetInt = 0; - BOOL interfaceInfoComplete; int curInterf = 0, i; if (!NT_SUCCESS(status)) { @@ -412,8 +383,7 @@ TDIEntityID ip_ent; int j; - interfaceInfoComplete = FALSE; - status = getNthIpEntity( tcpFile, 0, &ip_ent ); + status = getNthIpEntity( tcpFile, curInterf, &ip_ent ); if( NT_SUCCESS(status) ) status = tdiGetIpAddrsForIpEntity ( tcpFile, &ip_ent, &addrs, &numAddrs ); Modified: trunk/reactos/dll/win32/iphlpapi/iphlpapi_private.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/iphlpapi/iphlpap… ============================================================================== --- trunk/reactos/dll/win32/iphlpapi/iphlpapi_private.h [iso-8859-1] (original) +++ trunk/reactos/dll/win32/iphlpapi/iphlpapi_private.h [iso-8859-1] Fri Oct 9 04:40:38 2009 @@ -59,8 +59,6 @@ #define TCP_REQUEST_QUERY_INFORMATION_INIT { { { 0 } } } #define TCP_REQUEST_SET_INFORMATION_INIT { { 0 } } - -#define IP_MIB_ROUTETABLE_ENTRY_ID 0x101 // As in the mib from RFC 1213 Modified: trunk/reactos/dll/win32/iphlpapi/ipstats_reactos.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/iphlpapi/ipstats… ============================================================================== --- trunk/reactos/dll/win32/iphlpapi/ipstats_reactos.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/iphlpapi/ipstats_reactos.c [iso-8859-1] Fri Oct 9 04:40:38 2009 @@ -62,30 +62,8 @@ #endif BOOL isIpEntity( HANDLE tcpFile, TDIEntityID *ent ) { - DWORD entityType, returnedLen; - NTSTATUS status; - TCP_REQUEST_QUERY_INFORMATION_EX req; - - req.ID.toi_class = INFO_CLASS_GENERIC; - req.ID.toi_type = INFO_TYPE_PROVIDER; - req.ID.toi_id = ENTITY_TYPE_ID; - req.ID.toi_entity = *ent; - - status = - DeviceIoControl - ( tcpFile, - IOCTL_TCP_QUERY_INFORMATION_EX, - &req, - sizeof(req), - &entityType, - sizeof(entityType), - &returnedLen, - NULL ); - - TRACE("Ent: %04x:d -> %04x\n", - ent->tei_entity, ent->tei_instance, entityType ); - - return NT_SUCCESS(status) && entityType == CL_NL_IP; + return (ent->tei_entity == CL_NL_ENTITY || + ent->tei_entity == CO_NL_ENTITY); } NTSTATUS getNthIpEntity( HANDLE tcpFile, DWORD index, TDIEntityID *ent ) { @@ -182,9 +160,9 @@ status = tdiGetSetOfThings( tcpFile, INFO_CLASS_PROTOCOL, INFO_TYPE_PROVIDER, - IP_MIB_ROUTETABLE_ENTRY_ID, - ent->tei_entity, - 0, + IP_MIB_ARPTABLE_ENTRY_ID, + CL_NL_ENTITY, + ent->tei_instance, 0, sizeof(IPRouteEntry), (PVOID *)routes, @@ -204,8 +182,8 @@ INFO_CLASS_PROTOCOL, INFO_TYPE_PROVIDER, IP_MIB_ADDRTABLE_ENTRY_ID, - ent->tei_entity, - 0, + CL_NL_ENTITY, + ent->tei_instance, 0, sizeof(IPAddrEntry), (PVOID *)addrs, Modified: trunk/reactos/dll/win32/iphlpapi/route_reactos.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/iphlpapi/route_r… ============================================================================== --- trunk/reactos/dll/win32/iphlpapi/route_reactos.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/iphlpapi/route_reactos.c [iso-8859-1] Fri Oct 9 04:40:38 2009 @@ -69,8 +69,9 @@ if( NT_SUCCESS(status) ) { req.Req.ID.toi_class = INFO_CLASS_PROTOCOL; req.Req.ID.toi_type = INFO_TYPE_PROVIDER; - req.Req.ID.toi_id = IP_MIB_ROUTETABLE_ENTRY_ID; - req.Req.ID.toi_entity = id; + req.Req.ID.toi_id = IP_MIB_ARPTABLE_ENTRY_ID; + req.Req.ID.toi_entity.tei_instance = id.tei_instance; + req.Req.ID.toi_entity.tei_instance = CL_NL_ENTITY; req.Req.BufferSize = sizeof(*rte); rte = (IPRouteEntry *)&req.Req.Buffer[0]; @@ -133,8 +134,9 @@ if( NT_SUCCESS(status) ) { req.Req.ID.toi_class = INFO_CLASS_PROTOCOL; req.Req.ID.toi_type = INFO_TYPE_PROVIDER; - req.Req.ID.toi_id = IP_MIB_ROUTETABLE_ENTRY_ID; - req.Req.ID.toi_entity = id; + req.Req.ID.toi_id = IP_MIB_ARPTABLE_ENTRY_ID; + req.Req.ID.toi_entity.tei_instance = id.tei_instance; + req.Req.ID.toi_entity.tei_entity = CL_NL_ENTITY; req.Req.BufferSize = sizeof(*rte); rte = (IPRouteEntry *)&req.Req.Buffer[0]; Modified: trunk/reactos/drivers/network/tcpip/include/info.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/tcpip/incl… ============================================================================== --- trunk/reactos/drivers/network/tcpip/include/info.h [iso-8859-1] (original) +++ trunk/reactos/drivers/network/tcpip/include/info.h [iso-8859-1] Fri Oct 9 04:40:38 2009 @@ -6,6 +6,8 @@ */ #ifndef __INFO_H #define __INFO_H + +#include <tcpioctl.h> #define MAX_PHYSADDR_LEN 8 #define MAX_IFDESCR_LEN 256 @@ -100,23 +102,6 @@ ULONG DescrLen; } IFENTRY, *PIFENTRY; -#define IP_MIB_STATS_ID 1 -#define IF_MIB_STATS_ID 1 - -#ifndef IP_MIB_ROUTETABLE_ENTRY_ID -#define IP_MIB_ROUTETABLE_ENTRY_ID 0x101 -#endif -#ifndef IP_MIB_ADDRTABLE_ENTRY_ID -#define IP_MIB_ADDRTABLE_ENTRY_ID 0x102 -#endif -#ifndef IP_MIB_ARPTABLE_ENTRY_ID -#define IP_MIB_ARPTABLE_ENTRY_ID 0x101 -#endif -#ifndef MAX_PHYSADDR_SIZE -#define MAX_PHYSADDR_SIZE 8 -#endif - - /* Only UDP is supported */ #define TDI_SERVICE_FLAGS (TDI_SERVICE_CONNECTIONLESS_MODE | \ TDI_SERVICE_BROADCAST_SUPPORTED) @@ -157,54 +142,28 @@ PVOID Buffer, UINT BufferSize); -/* Network layer info functions */ -TDI_STATUS InfoNetworkLayerTdiQueryEx( UINT InfoClass, - UINT InfoType, - UINT InfoId, - PVOID Context, - TDIEntityID *id, +TDI_STATUS InfoTdiQueryGetAddrTable(TDIEntityID ID, + PNDIS_BUFFER Buffer, + PUINT BufferSize); + +TDI_STATUS InfoTdiQueryGetInterfaceMIB(TDIEntityID ID, + PIP_INTERFACE Interface, PNDIS_BUFFER Buffer, - PUINT BufferSize ); + PUINT BufferSize); -TDI_STATUS InfoNetworkLayerTdiSetEx( UINT InfoClass, - UINT InfoType, - UINT InfoId, - PVOID Context, - TDIEntityID *id, - PCHAR Buffer, - UINT BufferSize ); +TDI_STATUS InfoTdiQueryGetIPSnmpInfo( TDIEntityID ID, + PNDIS_BUFFER Buffer, + PUINT BufferSize ); -TDI_STATUS InfoTransportLayerTdiQueryEx( UINT InfoClass, - UINT InfoType, - UINT InfoId, - PVOID Context, - TDIEntityID *id, - PNDIS_BUFFER Buffer, - PUINT BufferSize ); +TDI_STATUS InfoTdiQueryGetRouteTable( PNDIS_BUFFER Buffer, + PUINT BufferSize ); -TDI_STATUS InfoTransportLayerTdiSetEx( UINT InfoClass, - UINT InfoType, - UINT InfoId, - PVOID Context, - TDIEntityID *id, - PCHAR Buffer, - UINT BufferSize ); +TDI_STATUS InfoTdiSetRoute(PIPROUTE_ENTRY Route); -TDI_STATUS InfoInterfaceTdiQueryEx( UINT InfoClass, - UINT InfoType, - UINT InfoId, - PVOID Context, - TDIEntityID *id, - PNDIS_BUFFER Buffer, - PUINT BufferSize ); - -TDI_STATUS InfoInterfaceTdiSetEx( UINT InfoClass, - UINT InfoType, - UINT InfoId, - PVOID Context, - TDIEntityID *id, - PCHAR Buffer, - UINT BufferSize ); +TDI_STATUS InfoTdiQueryGetArptableMIB(TDIEntityID ID, + PIP_INTERFACE Interface, + PNDIS_BUFFER Buffer, + PUINT BufferSize); /* Insert and remove interface entities */ VOID InsertTDIInterfaceEntity( PIP_INTERFACE Interface ); Modified: trunk/reactos/drivers/network/tcpip/include/tcpip.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/tcpip/incl… ============================================================================== --- trunk/reactos/drivers/network/tcpip/include/tcpip.h [iso-8859-1] (original) +++ trunk/reactos/drivers/network/tcpip/include/tcpip.h [iso-8859-1] Fri Oct 9 04:40:38 2009 @@ -149,29 +149,12 @@ char sa_data[14]; }; -typedef TDI_STATUS (*InfoRequest_f)( UINT InfoClass, - UINT InfoType, - UINT InfoId, - PVOID Context, - TDIEntityID *id, - PNDIS_BUFFER Buffer, - PUINT BufferSize ); - -typedef TDI_STATUS (*InfoSet_f)( UINT InfoClass, - UINT InfoType, - UINT InfoId, - PVOID Context, - TDIEntityID *id, - PCHAR Buffer, - UINT BufferSize ); - /* Sufficient information to manage the entity list */ typedef struct { UINT tei_entity; UINT tei_instance; PVOID context; - InfoRequest_f info_req; - InfoSet_f info_set; + UINT flags; } TDIEntityInfo; #ifndef htons Modified: trunk/reactos/drivers/network/tcpip/tcpip/iinfo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/tcpip/tcpi… ============================================================================== --- trunk/reactos/drivers/network/tcpip/tcpip/iinfo.c [iso-8859-1] (original) +++ trunk/reactos/drivers/network/tcpip/tcpip/iinfo.c [iso-8859-1] Fri Oct 9 04:40:38 2009 @@ -10,21 +10,26 @@ #include "precomp.h" -TDI_STATUS InfoTdiQueryGetInterfaceMIB(TDIEntityID *ID, +TDI_STATUS InfoTdiQueryGetInterfaceMIB(TDIEntityID ID, PIP_INTERFACE Interface, PNDIS_BUFFER Buffer, PUINT BufferSize) { TDI_STATUS Status = TDI_INVALID_REQUEST; PIFENTRY OutData; - PLAN_ADAPTER IF = (PLAN_ADAPTER)Interface->Context; + PLAN_ADAPTER IF; PCHAR IFDescr; ULONG Size; UINT DescrLenMax = MAX_IFDESCR_LEN - 1; NDIS_STATUS NdisStatus; + if (!Interface) + return TDI_INVALID_PARAMETER; + + IF = (PLAN_ADAPTER)Interface->Context; + TI_DbgPrint(DEBUG_INFO, ("Getting IFEntry MIB (IF %08x LA %08x) (%04x:%d)\n", - Interface, IF, ID->tei_entity, ID->tei_instance)); + Interface, IF, ID.tei_entity, ID.tei_instance)); OutData = (PIFENTRY)exAllocatePool( NonPagedPool, @@ -91,7 +96,7 @@ Size = IFDescr - (PCHAR)OutData + 1; TI_DbgPrint(DEBUG_INFO, ("Finished IFEntry MIB (%04x:%d) size %d\n", - ID->tei_entity, ID->tei_instance, Size)); + ID.tei_entity, ID.tei_instance, Size)); Status = InfoCopyOut( (PCHAR)OutData, Size, Buffer, BufferSize ); exFreePool( OutData ); @@ -101,7 +106,7 @@ return Status; } -TDI_STATUS InfoTdiQueryGetArptableMIB(TDIEntityID *ID, +TDI_STATUS InfoTdiQueryGetArptableMIB(TDIEntityID ID, PIP_INTERFACE Interface, PNDIS_BUFFER Buffer, PUINT BufferSize) { @@ -122,58 +127,106 @@ return Status; } -TDI_STATUS InfoTdiQueryGetArpCapability(TDIEntityID *ID, - PIP_INTERFACE Interface, - PNDIS_BUFFER Buffer, - PUINT BufferSize) { - NTSTATUS Status; - ULONG Capability = 0x280; - - TI_DbgPrint(MID_TRACE,("Copying out %d bytes (AT_ENTITY capability)\n", - sizeof(Capability))); - Status = InfoCopyOut - ( (PVOID)&Capability, sizeof(Capability), Buffer, BufferSize ); - - return Status; -} - -TDI_STATUS InfoInterfaceTdiQueryEx( UINT InfoClass, - UINT InfoType, - UINT InfoId, - PVOID Context, - TDIEntityID *id, - PNDIS_BUFFER Buffer, - PUINT BufferSize ) { - if( InfoClass == INFO_CLASS_GENERIC && - InfoType == INFO_TYPE_PROVIDER && - InfoId == ENTITY_TYPE_ID && - id->tei_entity == AT_ENTITY ) { - return InfoTdiQueryGetArpCapability( id, Context, Buffer, BufferSize ); - } else if( InfoClass == INFO_CLASS_PROTOCOL && - InfoType == INFO_TYPE_PROVIDER && - InfoId == IF_MIB_STATS_ID ) { - return InfoTdiQueryGetInterfaceMIB( id, Context, Buffer, BufferSize ); - } else if( InfoClass == INFO_CLASS_GENERIC && - InfoType == INFO_TYPE_PROVIDER && - InfoId == ENTITY_TYPE_ID ) { - ULONG Temp = IF_MIB; - return InfoCopyOut( (PCHAR)&Temp, sizeof(Temp), Buffer, BufferSize ); - } else if( InfoClass == INFO_CLASS_PROTOCOL && - InfoType == INFO_TYPE_PROVIDER && - InfoId == IP_MIB_ARPTABLE_ENTRY_ID ) { - return InfoTdiQueryGetArptableMIB( id, Context, Buffer, BufferSize ); - } else - return TDI_INVALID_REQUEST; -} - -TDI_STATUS InfoInterfaceTdiSetEx( UINT InfoClass, - UINT InfoType, - UINT InfoId, - PVOID Context, - TDIEntityID *id, - PCHAR Buffer, - UINT BufferSize ) { - TI_DbgPrint(DEBUG_INFO, ("Got Request: Class %x Type %x Id %x, EntityID %x:%x\n", - InfoClass, InfoId, id->tei_entity, id->tei_instance)); - return TDI_INVALID_REQUEST; -} +VOID InsertTDIInterfaceEntity( PIP_INTERFACE Interface ) { + KIRQL OldIrql; + UINT IFCount = 0, CLNLCount = 0, CLTLCount = 0, COTLCount = 0, ATCount = 0, ERCount = 0, i; + + TI_DbgPrint(DEBUG_INFO, + ("Inserting interface %08x (%d entities already)\n", + Interface, EntityCount)); + + TcpipAcquireSpinLock( &EntityListLock, &OldIrql ); + + /* Count IP Entities */ + for( i = 0; i < EntityCount; i++ ) + switch( EntityList[i].tei_entity ) + { + case IF_ENTITY: + IFCount++; + break; + + case CL_NL_ENTITY: + CLNLCount++; + break; + + case CL_TL_ENTITY: + CLTLCount++; + break; + + case CO_TL_ENTITY: + COTLCount++; + break; + + case AT_ENTITY: + ATCount++; + break; + + case ER_ENTITY: + ERCount++; + break; + + default: + break; + } + + EntityList[EntityCount].tei_entity = IF_ENTITY; + EntityList[EntityCount].tei_instance = IFCount; + EntityList[EntityCount].context = Interface; + EntityList[EntityCount].flags = IF_MIB; + EntityCount++; + EntityList[EntityCount].tei_entity = CL_NL_ENTITY; + EntityList[EntityCount].tei_instance = CLNLCount; + EntityList[EntityCount].context = Interface; + EntityList[EntityCount].flags = CL_NL_IP; + EntityCount++; + EntityList[EntityCount].tei_entity = CL_TL_ENTITY; + EntityList[EntityCount].tei_instance = CLTLCount; + EntityList[EntityCount].context = Interface; + EntityList[EntityCount].flags = CL_TL_UDP; + EntityCount++; + EntityList[EntityCount].tei_entity = CO_TL_ENTITY; + EntityList[EntityCount].tei_instance = COTLCount; + EntityList[EntityCount].context = Interface; + EntityList[EntityCount].flags = CO_TL_TCP; + EntityCount++; + EntityList[EntityCount].tei_entity = ER_ENTITY; + EntityList[EntityCount].tei_instance = ERCount; + EntityList[EntityCount].context = Interface; + EntityList[EntityCount].flags = ER_ICMP; + EntityCount++; + EntityList[EntityCount].tei_entity = AT_ENTITY; + EntityList[EntityCount].tei_instance = ATCount; + EntityList[EntityCount].context = Interface; + EntityList[EntityCount].flags = AT_ARP; + EntityCount++; + + TcpipReleaseSpinLock( &EntityListLock, OldIrql ); +} + +VOID RemoveTDIInterfaceEntity( PIP_INTERFACE Interface ) { + KIRQL OldIrql; + UINT i; + + TI_DbgPrint(DEBUG_INFO,("Removing TDI entry 0x%x\n", Interface)); + + TcpipAcquireSpinLock( &EntityListLock, &OldIrql ); + + /* Remove entities that have this interface as context + * In the future, this might include AT_ENTITY types, too + */ + for( i = 0; i < EntityCount; i++ ) { + TI_DbgPrint(DEBUG_INFO,("--> examining TDI entry 0x%x\n", EntityList[i].context)); + if( EntityList[i].context == Interface ) { + if( i != EntityCount-1 ) { + memcpy( &EntityList[i], + &EntityList[--EntityCount], + sizeof(EntityList[i]) ); + } else { + EntityCount--; + } + } + } + + TcpipReleaseSpinLock( &EntityListLock, OldIrql ); +} + Modified: trunk/reactos/drivers/network/tcpip/tcpip/info.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/tcpip/tcpi… ============================================================================== --- trunk/reactos/drivers/network/tcpip/tcpip/info.c [iso-8859-1] (original) +++ trunk/reactos/drivers/network/tcpip/tcpip/info.c [iso-8859-1] Fri Oct 9 04:40:38 2009 @@ -12,6 +12,35 @@ #include <debug.h> #include <route.h> +PVOID GetContext(TDIEntityID ID) +{ + UINT i; + KIRQL OldIrql; + PVOID Context; + + TcpipAcquireSpinLock(&EntityListLock, &OldIrql); + + for (i = 0; i < EntityCount; i++) + { + if (EntityList[i].tei_entity == ID.tei_entity && + EntityList[i].tei_instance == ID.tei_instance) + break; + } + + if (i == EntityCount) + { + TcpipReleaseSpinLock(&EntityListLock, OldIrql); + DbgPrint("WARNING: Unable to get context for %d %d\n", ID.tei_entity, ID.tei_instance); + return NULL; + } + + Context = EntityList[i].context; + + TcpipReleaseSpinLock(&EntityListLock, OldIrql); + + return Context; +} + TDI_STATUS InfoCopyOut( PCHAR DataOut, UINT SizeOut, PNDIS_BUFFER ClientBuf, PUINT ClientBufSize ) { UINT RememberedCBSize = *ClientBufSize; @@ -27,60 +56,38 @@ } } -VOID InsertTDIInterfaceEntity( PIP_INTERFACE Interface ) { +TDI_STATUS InfoTdiQueryEntityType(TDIEntityID ID, + PNDIS_BUFFER Buffer, + PUINT BufferSize) +{ KIRQL OldIrql; - UINT Count = 0, i; - - TI_DbgPrint(DEBUG_INFO, - ("Inserting interface %08x (%d entities already)\n", - Interface, EntityCount)); - - TcpipAcquireSpinLock( &EntityListLock, &OldIrql ); - - /* Count IP Entities */ - for( i = 0; i < EntityCount; i++ ) - if( EntityList[i].tei_entity == IF_ENTITY ) { - Count++; - TI_DbgPrint(DEBUG_INFO, ("Entity %d is an IF. Found %d\n", - i, Count)); - } - - EntityList[EntityCount].tei_entity = IF_ENTITY; - EntityList[EntityCount].tei_instance = Count; - EntityList[EntityCount].context = Interface; - EntityList[EntityCount].info_req = InfoInterfaceTdiQueryEx; - EntityList[EntityCount].info_set = InfoInterfaceTdiSetEx; - - EntityCount++; - - TcpipReleaseSpinLock( &EntityListLock, OldIrql ); -} - -VOID RemoveTDIInterfaceEntity( PIP_INTERFACE Interface ) { - KIRQL OldIrql; - UINT i; - - TI_DbgPrint(DEBUG_INFO,("Removing TDI entry 0x%x\n", Interface)); - - TcpipAcquireSpinLock( &EntityListLock, &OldIrql ); - - /* Remove entities that have this interface as context - * In the future, this might include AT_ENTITY types, too - */ - for( i = 0; i < EntityCount; i++ ) { - TI_DbgPrint(DEBUG_INFO,("--> examining TDI entry 0x%x\n", EntityList[i].context)); - if( EntityList[i].context == Interface ) { - if( i != EntityCount-1 ) { - memcpy( &EntityList[i], - &EntityList[--EntityCount], - sizeof(EntityList[i]) ); - } else { - EntityCount--; - } - } - } - - TcpipReleaseSpinLock( &EntityListLock, OldIrql ); + UINT i, Flags = 0; + + TcpipAcquireSpinLock(&EntityListLock, &OldIrql); + + for (i = 0; i < EntityCount; i++) + { + if (EntityList[i].tei_entity == ID.tei_entity && + EntityList[i].tei_instance == ID.tei_instance) + break; + } + + if (i == EntityCount) + { + TcpipReleaseSpinLock(&EntityListLock, OldIrql); + return TDI_INVALID_PARAMETER; + } + + Flags = EntityList[i].flags; + + InfoCopyOut((PCHAR)&Flags, + sizeof(ULONG), + Buffer, + BufferSize); + + TcpipReleaseSpinLock(&EntityListLock, OldIrql); + + return TDI_SUCCESS; } TDI_STATUS InfoTdiQueryListEntities(PNDIS_BUFFER Buffer, @@ -139,12 +146,7 @@ * Status of operation */ { - KIRQL OldIrql; - UINT i; - PVOID context = NULL; - NTSTATUS Status = TDI_INVALID_PARAMETER; - BOOLEAN FoundEntity = FALSE; - InfoRequest_f InfoRequest = NULL; + PVOID EntityListContext; TI_DbgPrint(DEBUG_INFO, ("InfoEx Req: %x %x %x!%04x:%d\n", @@ -154,79 +156,83 @@ ID->toi_entity.tei_entity, ID->toi_entity.tei_instance)); - /* Check wether it is a query for a list of entities */ - if (ID->toi_entity.tei_entity == GENERIC_ENTITY) - { - if ((ID->toi_class != INFO_CLASS_GENERIC) || - (ID->toi_type != INFO_TYPE_PROVIDER) || - (ID->toi_id != ENTITY_LIST_ID)) { - TI_DbgPrint(DEBUG_INFO,("Invalid parameter\n")); - Status = TDI_INVALID_PARAMETER; - } else - Status = InfoTdiQueryListEntities(Buffer, BufferSize); - } else if (ID->toi_entity.tei_entity == AT_ENTITY) { - TcpipAcquireSpinLock( &EntityListLock, &OldIrql ); - - for( i = 0; i < EntityCount; i++ ) { - if( EntityList[i].tei_entity == IF_ENTITY && - EntityList[i].tei_instance == ID->toi_entity.tei_instance ) { - InfoRequest = EntityList[i].info_req; - context = EntityList[i].context; - FoundEntity = TRUE; - break; - } - } - - TcpipReleaseSpinLock( &EntityListLock, OldIrql ); - - if( FoundEntity ) { - TI_DbgPrint(DEBUG_INFO, - ("Calling AT Entity %d (%04x:%d) InfoEx (%x,%x,%x)\n", - i, ID->toi_entity.tei_entity, - ID->toi_entity.tei_instance, - ID->toi_class, ID->toi_type, ID->toi_id)); - Status = InfoRequest( ID->toi_class, - ID->toi_type, - ID->toi_id, - context, - &ID->toi_entity, - Buffer, - BufferSize ); - } - } else { - TcpipAcquireSpinLock( &EntityListLock, &OldIrql ); - - for( i = 0; i < EntityCount; i++ ) { - if( EntityList[i].tei_entity == ID->toi_entity.tei_entity && - EntityList[i].tei_instance == ID->toi_entity.tei_instance ) { - InfoRequest = EntityList[i].info_req; - context = EntityList[i].context; - FoundEntity = TRUE; - break; - } - } - - TcpipReleaseSpinLock( &EntityListLock, OldIrql ); - - if( FoundEntity ) { - TI_DbgPrint(DEBUG_INFO, - ("Calling Entity %d (%04x:%d) InfoEx (%x,%x,%x)\n", - i, ID->toi_entity.tei_entity, - ID->toi_entity.tei_instance, - ID->toi_class, ID->toi_type, ID->toi_id)); - Status = InfoRequest( ID->toi_class, - ID->toi_type, - ID->toi_id, - context, - &ID->toi_entity, - Buffer, - BufferSize ); - } - } - - TI_DbgPrint(DEBUG_INFO,("Status: %08x\n", Status)); - - return Status; + switch (ID->toi_class) + { + case INFO_CLASS_GENERIC: + switch (ID->toi_id) + { + case ENTITY_LIST_ID: + if (ID->toi_type != INFO_TYPE_PROVIDER) + return TDI_INVALID_PARAMETER; + + return InfoTdiQueryListEntities(Buffer, BufferSize); + + case ENTITY_TYPE_ID: + if (ID->toi_type != INFO_TYPE_PROVIDER) + return TDI_INVALID_PARAMETER; + + return InfoTdiQueryEntityType(ID->toi_entity, Buffer, BufferSize); + + default: + return TDI_INVALID_REQUEST; + } + + case INFO_CLASS_PROTOCOL: + switch (ID->toi_id) + { + case IF_MIB_STATS_ID: + if (ID->toi_entity.tei_entity == IF_ENTITY) + if ((EntityListContext = GetContext(ID->toi_entity))) + return InfoTdiQueryGetInterfaceMIB(ID->toi_entity, EntityListContext, Buffer, BufferSize); + else + return TDI_INVALID_PARAMETER; + else if (ID->toi_entity.tei_entity == CL_NL_ENTITY || + ID->toi_entity.tei_entity == CO_NL_ENTITY) + return InfoTdiQueryGetIPSnmpInfo(ID->toi_entity, Buffer, BufferSize); + else + return TDI_INVALID_PARAMETER; + + case IP_MIB_ADDRTABLE_ENTRY_ID: + if (ID->toi_entity.tei_entity != CL_NL_ENTITY && + ID->toi_entity.tei_entity != CO_NL_ENTITY) + return TDI_INVALID_PARAMETER; + + if (ID->toi_type != INFO_TYPE_PROVIDER) + return TDI_INVALID_PARAMETER; + + return InfoTdiQueryGetAddrTable(ID->toi_entity, Buffer, BufferSize); + + case IP_MIB_ARPTABLE_ENTRY_ID: + if (ID->toi_type != INFO_TYPE_PROVIDER) + return TDI_INVALID_PARAMETER; + + if (ID->toi_entity.tei_entity == AT_ENTITY) + if ((EntityListContext = GetContext(ID->toi_entity))) + return InfoTdiQueryGetArptableMIB(ID->toi_entity, EntityListContext, + Buffer, BufferSize); + else + return TDI_INVALID_PARAMETER; + else if (ID->toi_entity.tei_entity == CO_NL_ENTITY || + ID->toi_entity.tei_entity == CL_NL_ENTITY) + return InfoTdiQueryGetRouteTable(Buffer, BufferSize); + else + return TDI_INVALID_PARAMETER; + +#if 0 + case IP_INTFC_INFO_ID: + if (ID->toi_type != INFO_TYPE_PROVIDER) + return TDI_INVALID_PARAMETER; + + return InfoTdiQueryGetIFInfo(Context, Buffer, BufferSize); +#endif + + default: + return TDI_INVALID_REQUEST; + } + + default: + return TDI_INVALID_REQUEST; + } } TDI_STATUS InfoTdiSetInformationEx @@ -245,24 +251,26 @@ * Status of operation */ { - switch( ID->toi_class ) { - case INFO_CLASS_PROTOCOL: - switch( ID->toi_type ) { - case INFO_TYPE_PROVIDER: - switch( ID->toi_id ) { - case IP_MIB_ROUTETABLE_ENTRY_ID: - return InfoNetworkLayerTdiSetEx - ( ID->toi_class, - ID->toi_type, - ID->toi_id, - NULL, - &ID->toi_entity, - Buffer, - BufferSize ); - } - } - break; - } - - return TDI_INVALID_PARAMETER; -} + switch (ID->toi_class) + { + case INFO_CLASS_PROTOCOL: + switch (ID->toi_id) + { + case IP_MIB_ARPTABLE_ENTRY_ID: + if (ID->toi_id != INFO_TYPE_PROVIDER) + return TDI_INVALID_PARAMETER; + + if (ID->toi_entity.tei_entity != CL_NL_ENTITY && + ID->toi_entity.tei_entity != CO_NL_ENTITY) + return TDI_INVALID_PARAMETER; + + return InfoTdiSetRoute((PIPROUTE_ENTRY)Buffer); + + default: + return TDI_INVALID_REQUEST; + } + + default: + return TDI_INVALID_REQUEST; + } +} 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 [iso-8859-1] (original) +++ trunk/reactos/drivers/network/tcpip/tcpip/main.c [iso-8859-1] Fri Oct 9 04:40:38 2009 @@ -699,17 +699,7 @@ return STATUS_INSUFFICIENT_RESOURCES; } - EntityList[0].tei_entity = CL_NL_ENTITY; - EntityList[0].tei_instance = 0; - EntityList[0].context = 0; - EntityList[0].info_req = InfoNetworkLayerTdiQueryEx; - EntityList[0].info_set = InfoNetworkLayerTdiSetEx; - EntityList[1].tei_entity = CL_TL_ENTITY; - EntityList[1].tei_instance = 0; - EntityList[1].context = 0; - EntityList[1].info_req = InfoTransportLayerTdiQueryEx; - EntityList[1].info_set = InfoTransportLayerTdiSetEx; - EntityCount = 2; + EntityCount = 0; EntityMax = MAX_TDI_ENTITIES; /* Allocate NDIS packet descriptors */ Modified: trunk/reactos/drivers/network/tcpip/tcpip/ninfo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/tcpip/tcpi… ============================================================================== --- trunk/reactos/drivers/network/tcpip/tcpip/ninfo.c [iso-8859-1] (original) +++ trunk/reactos/drivers/network/tcpip/tcpip/ninfo.c [iso-8859-1] Fri Oct 9 04:40:38 2009 @@ -13,57 +13,7 @@ #define IP_ROUTE_TYPE_ADD 3 #define IP_ROUTE_TYPE_DEL 2 -TDI_STATUS InfoTdiQueryGetAddrTable( PNDIS_BUFFER Buffer, - PUINT BufferSize ) { - - TDI_STATUS Status = TDI_INVALID_REQUEST; - KIRQL OldIrql; - UINT Count = 0; - UINT IfCount = CountInterfaces(); - PIPADDR_ENTRY IpAddress = - exAllocatePool( NonPagedPool, sizeof( IPADDR_ENTRY ) * IfCount ); - PIPADDR_ENTRY IpCurrent = IpAddress; - IF_LIST_ITER(CurrentIF); - - TI_DbgPrint(DEBUG_INFO, ("Called.\n")); - - if (!IpAddress) - return TDI_NO_RESOURCES; - - TcpipAcquireSpinLock(&InterfaceListLock, &OldIrql); - - ForEachInterface(CurrentIF) { - IpCurrent->Index = CurrentIF->Index; - IpCurrent->Addr = 0; - IpCurrent->BcastAddr = 0; - IpCurrent->Mask = 0; - - /* Locate the diffrent addresses and put them the right place */ - GetInterfaceIPv4Address( CurrentIF, - ADE_UNICAST, - &IpCurrent->Addr ); - GetInterfaceIPv4Address( CurrentIF, - ADE_BROADCAST, - &IpCurrent->BcastAddr ); - GetInterfaceIPv4Address( CurrentIF, - ADE_ADDRMASK, - &IpCurrent->Mask ); - IpCurrent++; - Count++; - } EndFor(CurrentIF); - ASSERT( Count == IfCount ); - - TcpipReleaseSpinLock(&InterfaceListLock, OldIrql); - - Status = InfoCopyOut( (PCHAR)IpAddress, sizeof(*IpAddress) * IfCount, - Buffer, BufferSize ); - - exFreePool( IpAddress ); - - TI_DbgPrint(DEBUG_INFO, ("Returning %08x\n", Status)); - - return Status; -} + /* Get IPRouteEntry s for each of the routes in the system */ TDI_STATUS InfoTdiQueryGetRouteTable( PNDIS_BUFFER Buffer, PUINT BufferSize ) { @@ -138,7 +88,65 @@ return Status; } -TDI_STATUS InfoTdiQueryGetIPSnmpInfo( PNDIS_BUFFER Buffer, +TDI_STATUS InfoTdiQueryGetAddrTable(TDIEntityID ID, + PNDIS_BUFFER Buffer, + PUINT BufferSize) +{ + KIRQL OldIrql; + PIPADDR_ENTRY IPEntry; + PIP_INTERFACE CurrentIF; + UINT i; + + TI_DbgPrint(DEBUG_INFO, ("Called.\n")); + + + TcpipAcquireSpinLock(&EntityListLock, &OldIrql); + + for (i = 0; i < EntityCount; i++) + { + if (EntityList[i].tei_entity == ID.tei_entity && + EntityList[i].tei_instance == ID.tei_instance) + break; + } + + if (i == EntityCount) + { + TcpipReleaseSpinLock(&EntityListLock, OldIrql); + return TDI_INVALID_PARAMETER; + } + + IPEntry = exAllocatePool(NonPagedPool, sizeof(IPADDR_ENTRY)); + if (!IPEntry) + { + TcpipReleaseSpinLock(&EntityListLock, OldIrql); + return TDI_NO_RESOURCES; + } + + CurrentIF = EntityList[i].context; + + IPEntry->Index = CurrentIF->Index; + GetInterfaceIPv4Address(CurrentIF, + ADE_UNICAST, + &IPEntry->Addr); + GetInterfaceIPv4Address(CurrentIF, + ADE_ADDRMASK, + &IPEntry->Mask); + GetInterfaceIPv4Address(CurrentIF, + ADE_BROADCAST, + &IPEntry->BcastAddr); + + TcpipReleaseSpinLock(&EntityListLock, OldIrql); + + InfoCopyOut((PCHAR)IPEntry, sizeof(IPADDR_ENTRY), + Buffer, BufferSize); + + exFreePool(IPEntry); + + return TDI_SUCCESS; +} + +TDI_STATUS InfoTdiQueryGetIPSnmpInfo( TDIEntityID ID, + PNDIS_BUFFER Buffer, PUINT BufferSize ) { IPSNMP_INFO SnmpInfo; UINT IfCount = CountInterfaces(); @@ -161,95 +169,35 @@ return Status; } -TDI_STATUS InfoNetworkLayerTdiQueryEx( UINT InfoClass, - UINT InfoType, - UINT InfoId, - PVOID Context, - TDIEntityID *id, - PNDIS_BUFFER Buffer, - PUINT BufferSize ) { - TDI_STATUS Status = TDI_INVALID_REQUEST; - - TI_DbgPrint(DEBUG_INFO, ("Called.\n")); - - switch( InfoClass ) { - case INFO_CLASS_GENERIC: - if( InfoType == INFO_TYPE_PROVIDER && InfoId == ENTITY_TYPE_ID ) { - ULONG Return = CL_NL_IP; - Status = InfoCopyOut( (PCHAR)&Return, sizeof(Return), - Buffer, BufferSize ); - } - break; - - case INFO_CLASS_PROTOCOL: - switch( InfoType ) { - case INFO_TYPE_PROVIDER: - switch( InfoId ) { - case IP_MIB_ADDRTABLE_ENTRY_ID: - Status = InfoTdiQueryGetAddrTable( Buffer, BufferSize ); - break; - - case IP_MIB_ROUTETABLE_ENTRY_ID: - Status = InfoTdiQueryGetRouteTable( Buffer, BufferSize ); - break; - - case IP_MIB_STATS_ID: - Status = InfoTdiQueryGetIPSnmpInfo( Buffer, BufferSize ); - break; - } - break; - } - } - - TI_DbgPrint(DEBUG_INFO, ("Returning %08x\n", Status)); - - return Status; -} - -TDI_STATUS InfoNetworkLayerTdiSetEx( UINT InfoClass, - UINT InfoType, - UINT InfoId, - PVOID Context, - TDIEntityID *id, - PCHAR Buffer, - UINT BufferSize ) { - NTSTATUS Status = TDI_INVALID_REQUEST; - IP_ADDRESS Address; - IP_ADDRESS Netmask; - IP_ADDRESS Router; +TDI_STATUS InfoTdiSetRoute(PIPROUTE_ENTRY Route) +{ + IP_ADDRESS Address, Netmask, Router; PNEIGHBOR_CACHE_ENTRY NCE; - TI_DbgPrint(DEBUG_INFO,("Called\n")); - - //OskitDumpBuffer( (OSK_PCHAR)Buffer, BufferSize ); - - if( InfoClass == INFO_CLASS_PROTOCOL && - InfoType == INFO_TYPE_PROVIDER && - InfoId == IP_MIB_ROUTETABLE_ENTRY_ID && - id->tei_entity == CL_NL_ENTITY ) { /* Add or delete a route */ - PIPROUTE_ENTRY Route = (PIPROUTE_ENTRY)Buffer; - AddrInitIPv4( &Address, Route->Dest ); - AddrInitIPv4( &Netmask, Route->Mask ); - AddrInitIPv4( &Router, Route->Gw ); - - if( Route->Type == IP_ROUTE_TYPE_ADD ) { /* Add the route */ - TI_DbgPrint(DEBUG_INFO,("Adding route (%s)\n", A2S(&Address))); - /* Find the existing route this belongs to */ - NCE = RouterGetRoute( &Router ); - /* Really add the route */ - if( NCE && - RouterCreateRoute( &Address, &Netmask, &Router, - NCE->Interface, Route->Metric1 ) ) - Status = STATUS_SUCCESS; - else - Status = STATUS_UNSUCCESSFUL; - } else if( Route->Type == IP_ROUTE_TYPE_DEL ) { - TI_DbgPrint(DEBUG_INFO,("Removing route (%s)\n", A2S(&Address))); - Status = RouterRemoveRoute( &Address, &Router ); - } else Status = TDI_INVALID_REQUEST; - } - - TI_DbgPrint(DEBUG_INFO,("Returning %x\n", Status)); - - return Status; -} + AddrInitIPv4( &Address, Route->Dest ); + AddrInitIPv4( &Netmask, Route->Mask ); + AddrInitIPv4( &Router, Route->Gw ); + + if( Route->Type == IP_ROUTE_TYPE_ADD ) { /* Add the route */ + TI_DbgPrint(DEBUG_INFO,("Adding route (%s)\n", A2S(&Address))); + /* Find the existing route this belongs to */ + NCE = RouterGetRoute( &Router ); + if (!NCE) return TDI_INVALID_PARAMETER; + + /* Really add the route */ + if (!RouterCreateRoute( &Address, &Netmask, &Router, + NCE->Interface, Route->Metric1)) + return TDI_NO_RESOURCES; + + return TDI_SUCCESS; + } else if( Route->Type == IP_ROUTE_TYPE_DEL ) { + TI_DbgPrint(DEBUG_INFO,("Removing route (%s)\n", A2S(&Address))); + if (NT_SUCCESS(RouterRemoveRoute( &Address, &Router ))) + return TDI_SUCCESS; + else + return TDI_INVALID_PARAMETER; + } + + return TDI_INVALID_REQUEST; +} +

1 0

[sginsberg] 43337: - Stub out ExpSystemErrorHandler more properly.
by sginsberg＠svn.reactos.org 08 Oct '09

08 Oct '09

Author: sginsberg Date: Thu Oct 8 22:06:37 2009 New Revision: 43337 URL: http://svn.reactos.org/svn/reactos?rev=43337&view=rev Log: - Stub out ExpSystemErrorHandler more properly. Modified: trunk/reactos/ntoskrnl/ex/harderr.c Modified: trunk/reactos/ntoskrnl/ex/harderr.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ex/harderr.c?rev=… ============================================================================== --- trunk/reactos/ntoskrnl/ex/harderr.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/ex/harderr.c [iso-8859-1] Thu Oct 8 22:06:37 2009 @@ -58,12 +58,26 @@ IN PULONG_PTR Parameters, IN BOOLEAN Shutdown) { - ULONG_PTR Dummy[4] = {0, 0, 0, 0}; + ULONG_PTR BugCheckParameters[MAXIMUM_HARDERROR_PARAMETERS] = {0, 0, 0, 0}; + ULONG i; + + /* Sanity check */ + ASSERT(NumberOfParameters <= MAXIMUM_HARDERROR_PARAMETERS); + + /* + * KeBugCheck expects MAXIMUM_HARDERROR_PARAMETERS parameters, + * but we might get called with less, so use a local buffer here. + */ + for (i = 0; i < NumberOfParameters; i++) + { + /* Copy them over */ + BugCheckParameters[i] = Parameters[i]; + } /* FIXME: STUB */ KeBugCheckEx(FATAL_UNHANDLED_HARD_ERROR, ErrorStatus, - (ULONG_PTR)Dummy, + (ULONG_PTR)BugCheckParameters, 0, 0); return STATUS_SUCCESS; @@ -507,7 +521,7 @@ * Optional string parameter (can be only one per error code) * * @param Parameters - * Array of ULONG parameters for use in error message string + * Array of ULONG_PTR parameters for use in error message string * * @param ValidResponseOptions * See HARDERROR_RESPONSE_OPTION for possible values description

1 0

[dchapyshev] 43336: - Partially revert r42993 for imagelist See issue #4304 for more details.
by dchapyshev＠svn.reactos.org 08 Oct '09

08 Oct '09

Author: dchapyshev Date: Thu Oct 8 13:18:34 2009 New Revision: 43336 URL: http://svn.reactos.org/svn/reactos?rev=43336&view=rev Log: - Partially revert r42993 for imagelist See issue #4304 for more details. Modified: trunk/reactos/dll/win32/comctl32/imagelist.c Modified: trunk/reactos/dll/win32/comctl32/imagelist.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/imageli… ============================================================================== --- trunk/reactos/dll/win32/comctl32/imagelist.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/imagelist.c [iso-8859-1] Thu Oct 8 13:18:34 2009 @@ -156,7 +156,7 @@ * This function CANNOT be used to reduce the number of images. */ static void -IMAGELIST_InternalExpandBitmaps (HIMAGELIST himl, INT nImageCount, INT cx, INT cy) +IMAGELIST_InternalExpandBitmaps (HIMAGELIST himl, INT nImageCount, INT cy) { HDC hdcBitmap; HBITMAP hbmNewBitmap, hbmNull; @@ -167,15 +167,14 @@ && (himl->cy >= cy)) return; - if (cx == 0) cx = himl->cx; nNewCount = himl->cCurImage + nImageCount + himl->cGrow; - imagelist_get_bitmap_size(himl, nNewCount, cx, &sz); + imagelist_get_bitmap_size(himl, nNewCount, himl->cx, &sz); TRACE("Create expanded bitmaps : himl=%p x=%d y=%d count=%d\n", himl, sz.cx, cy, nNewCount); hdcBitmap = CreateCompatibleDC (0); - hbmNewBitmap = ImageList_CreateImage(hdcBitmap, himl, nNewCount, cx); + hbmNewBitmap = ImageList_CreateImage(hdcBitmap, himl, nNewCount, himl->cx); if (hbmNewBitmap == 0) ERR("creating new image bitmap (x=%d y=%d)!\n", sz.cx, cy); @@ -249,7 +248,7 @@ nImageCount = bmp.bmWidth / himl->cx; - IMAGELIST_InternalExpandBitmaps (himl, nImageCount, bmp.bmWidth, bmp.bmHeight); + IMAGELIST_InternalExpandBitmaps (himl, nImageCount, bmp.bmHeight); hdcBitmap = CreateCompatibleDC(0); @@ -351,7 +350,7 @@ else nImageCount = 0; - IMAGELIST_InternalExpandBitmaps (himl, nImageCount, bmp.bmWidth, bmp.bmHeight); + IMAGELIST_InternalExpandBitmaps (himl, nImageCount, bmp.bmHeight); nIndex = himl->cCurImage; himl->cCurImage += nImageCount; @@ -2305,7 +2304,7 @@ if (nIndex == -1) { if (himl->cCurImage + 1 > himl->cMaxImage) - IMAGELIST_InternalExpandBitmaps (himl, 1, 0, 0); + IMAGELIST_InternalExpandBitmaps (himl, 1, 0); nIndex = himl->cCurImage; himl->cCurImage++;

1 0

[janderwald] 43335: - When enumerating controls, cControls *tells* the number of controls not *cConnections*. Fix a crash in XP sndvol32 - Clear the entire struct, not just part of it - WdmAudMixerCapabilities just provides the index of a device, not its handle. Fixes displaying of duplicate mixer name in sndvol32 - Verify that a mixer line exists
by janderwald＠svn.reactos.org 08 Oct '09

08 Oct '09

Author: janderwald Date: Thu Oct 8 09:25:26 2009 New Revision: 43335 URL: http://svn.reactos.org/svn/reactos?rev=43335&view=rev Log: - When enumerating controls, cControls *tells* the number of controls not *cConnections*. Fix a crash in XP sndvol32 - Clear the entire struct, not just part of it - WdmAudMixerCapabilities just provides the index of a device, not its handle. Fixes displaying of duplicate mixer name in sndvol32 - Verify that a mixer line exists Modified: trunk/reactos/drivers/wdm/audio/legacy/wdmaud/mixer.c Modified: trunk/reactos/drivers/wdm/audio/legacy/wdmaud/mixer.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/legacy/w… ============================================================================== --- trunk/reactos/drivers/wdm/audio/legacy/wdmaud/mixer.c [iso-8859-1] (original) +++ trunk/reactos/drivers/wdm/audio/legacy/wdmaud/mixer.c [iso-8859-1] Thu Oct 8 09:25:26 2009 @@ -92,7 +92,7 @@ { MixerLineSrc = (LPMIXERLINE_EXT)CONTAINING_RECORD(Entry, MIXERLINE_EXT, Entry); - for(Index = 0; Index < MixerLineSrc->Line.cConnections; Index++) + for(Index = 0; Index < MixerLineSrc->Line.cControls; Index++) { if (MixerLineSrc->LineControls[Index].dwControlID == dwControlID) { @@ -1530,7 +1530,7 @@ } /* initialize mixer destination line */ - RtlZeroMemory(DestinationLine, sizeof(MIXERLINEW)); + RtlZeroMemory(DestinationLine, sizeof(MIXERLINE_EXT)); DestinationLine->Line.cbStruct = sizeof(MIXERLINEW); DestinationLine->Line.dwSource = MAXULONG; DestinationLine->Line.dwLineID = DESTINATION_LINE; @@ -1702,14 +1702,14 @@ IN PWDMAUD_CLIENT ClientInfo, IN PWDMAUD_DEVICE_EXTENSION DeviceExtension) { - if ((ULONG)DeviceInfo->hDevice >= DeviceExtension->MixerInfoCount) + if ((ULONG)DeviceInfo->DeviceIndex >= DeviceExtension->MixerInfoCount) { /* invalid parameter */ return STATUS_INVALID_PARAMETER; } /* copy cached mixer caps */ - RtlMoveMemory(&DeviceInfo->u.MixCaps, &DeviceExtension->MixerInfo[(ULONG)DeviceInfo->hDevice].MixCaps, sizeof(MIXERCAPSW)); + RtlMoveMemory(&DeviceInfo->u.MixCaps, &DeviceExtension->MixerInfo[(ULONG)DeviceInfo->DeviceIndex].MixCaps, sizeof(MIXERCAPSW)); return STATUS_SUCCESS; } @@ -1842,7 +1842,11 @@ } MixerLineSrc = GetSourceMixerLineByLineId(&DeviceExtension->MixerInfo[(ULONG)DeviceInfo->hDevice], DeviceInfo->u.MixLine.dwLineID); - ASSERT(MixerLineSrc); + if (!MixerLineSrc) + { + DPRINT1("Failed to find Line with id %u\n", DeviceInfo->u.MixLine.dwLineID); + return SetIrpIoStatus(Irp, STATUS_INVALID_PARAMETER, 0); + } /* copy cached data */ RtlCopyMemory(&DeviceInfo->u.MixLine, &MixerLineSrc->Line, sizeof(MIXERLINEW)); @@ -1938,7 +1942,7 @@ /* invalid parameter */ return SetIrpIoStatus(Irp, STATUS_INVALID_PARAMETER, 0); } - + DPRINT1("MixerId %u ControlId %u\n",(ULONG)DeviceInfo->hDevice, DeviceInfo->u.MixControls.dwControlID); Status = GetMixerControlById(&DeviceExtension->MixerInfo[(ULONG)DeviceInfo->hDevice], DeviceInfo->u.MixControls.dwControlID, NULL, &MixerControl, NULL); if (NT_SUCCESS(Status)) { @@ -1946,7 +1950,6 @@ } return SetIrpIoStatus(Irp, Status, sizeof(WDMAUD_DEVICE_INFO)); } - UNIMPLEMENTED; //DbgBreakPoint();

1 0

[sginsberg] 43334: MiReserveSystemPtes: Don't assert that allocation succeeded, just warn if we failed. All callers of the function handle allocation failure properly.
by sginsberg＠svn.reactos.org 07 Oct '09

07 Oct '09

Author: sginsberg Date: Wed Oct 7 22:14:01 2009 New Revision: 43334 URL: http://svn.reactos.org/svn/reactos?rev=43334&view=rev Log: MiReserveSystemPtes: Don't assert that allocation succeeded, just warn if we failed. All callers of the function handle allocation failure properly. Modified: trunk/reactos/ntoskrnl/mm/ARM3/syspte.c Modified: trunk/reactos/ntoskrnl/mm/ARM3/syspte.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/syspte.c?… ============================================================================== --- trunk/reactos/ntoskrnl/mm/ARM3/syspte.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/mm/ARM3/syspte.c [iso-8859-1] Wed Oct 7 22:14:01 2009 @@ -170,12 +170,26 @@ IN MMSYSTEM_PTE_POOL_TYPE SystemPtePoolType) { PMMPTE PointerPte; - + // // Use the extended function // PointerPte = MiReserveAlignedSystemPtes(NumberOfPtes, SystemPtePoolType, 0); - ASSERT(PointerPte != NULL); + + // + // Check if allocation failed + // + if (!PointerPte) + { + // + // Warn that we are out of memory + // + DPRINT1("MiReserveSystemPtes: Failed to reserve %lu PTE(s)!\n", NumberOfPtes); + } + + // + // Return the PTE Pointer + // return PointerPte; }

1 0

[dgorbachev] 43333: Fix GCC 4.1.3 warning.
by dgorbachev＠svn.reactos.org 07 Oct '09

07 Oct '09

Author: dgorbachev Date: Wed Oct 7 22:04:17 2009 New Revision: 43333 URL: http://svn.reactos.org/svn/reactos?rev=43333&view=rev Log: Fix GCC 4.1.3 warning. Modified: trunk/reactos/ntoskrnl/kd/kdio.c Modified: trunk/reactos/ntoskrnl/kd/kdio.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd/kdio.c?rev=433… ============================================================================== --- trunk/reactos/ntoskrnl/kd/kdio.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kd/kdio.c [iso-8859-1] Wed Oct 7 22:04:17 2009 @@ -68,7 +68,7 @@ KdpDebugBuffer, end, NULL, NULL); } - InterlockedExchangeAddUL(&KdpFreeBytes, num); + (VOID)InterlockedExchangeAddUL(&KdpFreeBytes, num); } }

1 0

[dgorbachev] 43332: Happy testing.
by dgorbachev＠svn.reactos.org 07 Oct '09

07 Oct '09

Author: dgorbachev Date: Wed Oct 7 21:58:15 2009 New Revision: 43332 URL: http://svn.reactos.org/svn/reactos?rev=43332&view=rev Log: Happy testing. Modified: trunk/reactos/base/setup/usetup/bootsup.c Modified: trunk/reactos/base/setup/usetup/bootsup.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/setup/usetup/bootsup.… ============================================================================== --- trunk/reactos/base/setup/usetup/bootsup.c [iso-8859-1] (original) +++ trunk/reactos/base/setup/usetup/bootsup.c [iso-8859-1] Wed Oct 7 21:58:15 2009 @@ -12,9 +12,9 @@ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * GNU General Public License for more details. * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. */ /* * COPYRIGHT: See COPYING in the top level directory @@ -305,7 +305,7 @@ L"SystemPath", ArcPath); - /* Options=/DEBUG /DEBUGPORT=COM1 /BAUDRATE=115200 /SOS*/ + /* Options=/DEBUG /DEBUGPORT=COM1 /BAUDRATE=115200 /SOS */ IniCacheInsertKey(IniSection, NULL, INSERT_LAST, @@ -430,6 +430,12 @@ L"ReactOS_KdSerial", L"\"ReactOS (RosDbg)\"", L"ReactOS", ArcPath, L"/DEBUG /DEBUGPORT=COM1 /BAUDRATE=115200 /SOS /KDSERIAL"); + + /* ReactOS_LogFile */ + CreateFreeLoaderEntry(IniCache, IniSection, + L"ReactOS_LogFile", L"\"ReactOS (Log file)\"", + L"Windows2003", ArcPath, + L"/DEBUG /DEBUGPORT=FILE /SOS"); /* ReactOS_Ram */ CreateFreeLoaderEntry(IniCache, IniSection,

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