Commit graph

31878 commits

Author SHA1 Message Date
Cameron Gutman b4b79146c7 - Revert my last commit
- Looks like this is meant to fail
 - I added a comment so no one does this again

svn path=/branches/aicom-network-fixes/; revision=35745
2008-08-29 04:23:10 +00:00
Cameron Gutman cf392f00f6 - Fix a code typo resulting in calls to MiQueryResources to return early
svn path=/branches/aicom-network-fixes/; revision=35744
2008-08-29 03:59:25 +00:00
Cameron Gutman 8fa497277d - Free the address we've previously allocated if needed
svn path=/branches/aicom-network-fixes/; revision=35743
2008-08-29 02:19:16 +00:00
Cameron Gutman 976fe9f031 - Set Handle to NULL so we don't try to close it later
svn path=/branches/aicom-network-fixes/; revision=35741
2008-08-29 01:08:55 +00:00
Cameron Gutman 906dfdd979 - Don't handle OSK_EINPROGRESS since TCPTranslateError already has done the conversion to STATUS_PENDING
svn path=/branches/aicom-network-fixes/; revision=35714
2008-08-27 21:45:45 +00:00
Cameron Gutman 92073dfb72 - Fix many memory leaks
- Complete with the correct status

svn path=/branches/aicom-network-fixes/; revision=35656
2008-08-26 02:57:12 +00:00
Cameron Gutman d4296ce6a5 - Fix a memory leak
svn path=/branches/aicom-network-fixes/; revision=35653
2008-08-26 00:47:22 +00:00
Cameron Gutman abd660f2dc - Revert my whole ASSERT mess
- This fixes Qemu

svn path=/branches/aicom-network-fixes/; revision=35611
2008-08-24 19:53:16 +00:00
Cameron Gutman 0a84b51976 - Apply the same change in my branch as I did to trunk in r35602
svn path=/branches/aicom-network-fixes/; revision=35610
2008-08-24 19:28:25 +00:00
Cameron Gutman 7a30e67498 - Add another missing PollReeval
svn path=/branches/aicom-network-fixes/; revision=35609
2008-08-24 19:21:41 +00:00
Cameron Gutman 63ba865297 - Revert r35601
- This should always be set if FCB->CurrentThread is non-NULL

svn path=/branches/aicom-network-fixes/; revision=35607
2008-08-24 18:32:19 +00:00
Cameron Gutman d70ad469a8 - Set FCB->Locked to TRUE after we increment the lock count
svn path=/branches/aicom-network-fixes/; revision=35601
2008-08-24 16:20:14 +00:00
Cameron Gutman a66142aff9 - Add some missing calls to PollReeval
- Move an existing PollReeval to always be called

svn path=/branches/aicom-network-fixes/; revision=35597
2008-08-24 13:56:35 +00:00
Cameron Gutman e4a04351f2 - Fail if DatagramRecv->Address is NULL
svn path=/branches/aicom-network-fixes/; revision=35596
2008-08-24 13:25:19 +00:00
Cameron Gutman 963205e8b8 - Set *ConnectionInfo to NULL if we can't allocate memory for ConnInfo
svn path=/branches/aicom-network-fixes/; revision=35589
2008-08-24 02:55:26 +00:00
Cameron Gutman 4f54a68190 - Skip ObReferenceObjectByHandle if HandleArray[i].Handle is NULL
- Fail if ObReferenceObjectByHandle didn't succeed
 - Remove a bad ASSERT I added in my last commit

svn path=/branches/aicom-network-fixes/; revision=35583
2008-08-24 01:34:17 +00:00
Cameron Gutman 98c2a1ea67 - Fail if MmProbeAndLockPages doesn't succeed
- Fail if IoAllocateMdl returns NULL
 - Add some ASSERTs

svn path=/branches/aicom-network-fixes/; revision=35582
2008-08-24 01:09:48 +00:00
Cameron Gutman e2c0738526 - Make sure IPAddr is non-NULL
- Change STATUS_UNSUCCESSFUL to STATUS_INVALID_PARAMETER

svn path=/branches/aicom-network-fixes/; revision=35579
2008-08-23 22:06:49 +00:00
Cameron Gutman 055139fc07 - Code cleanup
- Make sure FCB->RemoteAddress is non-NULL

svn path=/branches/aicom-network-fixes/; revision=35578
2008-08-23 21:53:37 +00:00
Cameron Gutman 2d966ffcfb - Make sure FCB->AddressFrom is non-NULL
svn path=/branches/aicom-network-fixes/; revision=35577
2008-08-23 21:41:10 +00:00
Cameron Gutman fb0a6601f9 - More STATUS_UNSUCCESSFUL to STATUS_INVALID_PARAMETER changes
svn path=/branches/aicom-network-fixes/; revision=35538
2008-08-22 19:35:20 +00:00
Cameron Gutman f337da0bfb - Handle STATUS_INVALID_PARAMETER
svn path=/branches/aicom-network-fixes/; revision=35537
2008-08-22 19:30:33 +00:00
Cameron Gutman e233f4f784 - Return a more accurate status (STATUS_INVALID_PARAMETER) than STATUS_UNSUCCESSFUL when a parameter is NULL
svn path=/branches/aicom-network-fixes/; revision=35536
2008-08-22 19:26:30 +00:00
Cameron Gutman 3b1ae0bb37 - Replace MmMapLockedPages with MmGetSystemAddressForMdlSafe
- Add a check to UnlockRequest
 - Enclose MmProbeAndLockPages in SEH
 - Fixes bug 3672

svn path=/branches/aicom-network-fixes/; revision=35507
2008-08-21 14:46:03 +00:00
Cameron Gutman 3b40e9b047 - Reapply r35481
svn path=/branches/aicom-network-fixes/; revision=35486
2008-08-20 19:06:43 +00:00
Cameron Gutman df19533df7 - Fix IRP cancellation issues
- Ping doesn't hang anymore

svn path=/branches/aicom-network-fixes/; revision=35485
2008-08-20 18:50:26 +00:00
Cameron Gutman 67fae61a27 - Revert 35477-35481
svn path=/branches/aicom-network-fixes/; revision=35484
2008-08-20 18:46:38 +00:00
Cameron Gutman 9b492699af - Fix handling of cancelled IRPs
svn path=/branches/aicom-network-fixes/; revision=35481
2008-08-20 17:29:01 +00:00
Cameron Gutman 6159e6d129 - Don't forget about ConnectIrp (This fix was lost in the merge)
- Don't destroy the same socket twice
 - Don't unlock the same socket twice
 - Don't destroy a socket with pending IRPs

svn path=/branches/aicom-network-fixes/; revision=35480
2008-08-20 14:56:12 +00:00
Cameron Gutman 3def13f731 - Fix build
svn path=/branches/aicom-network-fixes/; revision=35479
2008-08-20 14:48:42 +00:00
Cameron Gutman a796846871 - The rest of the merge
- This will most likely need fixing later

svn path=/branches/aicom-network-fixes/; revision=35478
2008-08-20 14:43:33 +00:00
Cameron Gutman 8b269440fd - Reapply arty's datagram cancellation work
- Part 1 of 3

svn path=/branches/aicom-network-fixes/; revision=35477
2008-08-20 14:21:08 +00:00
Cameron Gutman fcdbf0c9d5 - Move LockBuffers
- Remove a duplicate debug print

svn path=/branches/aicom-network-fixes/; revision=35469
2008-08-20 01:42:12 +00:00
Cameron Gutman 95efddbf54 - Use FCB->Connection.Object instead of FCB->AddressFile.Object
- Spotted by Arty

svn path=/branches/aicom-network-fixes/; revision=35468
2008-08-20 01:10:13 +00:00
Cameron Gutman 5c1039783c - Fix a bug that resulted in the status getting changed if TdiQueryInformation failed
svn path=/branches/aicom-network-fixes/; revision=35467
2008-08-19 23:50:41 +00:00
Cameron Gutman 0afe589112 - Use MmGetSystemAddressForMdlSafe instead of MmMapLockedPages
- Noticed by Alex

svn path=/branches/aicom-network-fixes/; revision=35466
2008-08-19 23:44:11 +00:00
Cameron Gutman 717ec21981 - Fix a memory leak
svn path=/branches/aicom-network-fixes/; revision=35463
2008-08-19 21:45:08 +00:00
Cameron Gutman 735fdd1782 - Make sure we have an mdl and a buffer
svn path=/branches/aicom-network-fixes/; revision=35457
2008-08-19 17:45:53 +00:00
Cameron Gutman 2a199d0cb2 - SEHify LockRequest
- Free everything and return if MmMapLockedPages fails

svn path=/branches/aicom-network-fixes/; revision=35454
2008-08-19 16:23:01 +00:00
Cameron Gutman ea2d464b14 - MmMapLockedPages takes KPROCESSOR_MODE not LOCK_OPERATION as the second parameter
svn path=/branches/aicom-network-fixes/; revision=35453
2008-08-19 16:06:36 +00:00
Cameron Gutman 1c3484cdd1 - Make sure we don't complete with STATUS_SUCCESS if MmMapLockedPages fails
- Move and enable MmUnlockPages and IoFreeMdl

svn path=/branches/aicom-network-fixes/; revision=35446
2008-08-19 04:36:05 +00:00
Cameron Gutman 34a1b59e0a - Release the spin lock
svn path=/branches/aicom-network-fixes/; revision=35441
2008-08-19 00:19:18 +00:00
Cameron Gutman 6b5a88dd52 - Use the correct ioctl
svn path=/branches/aicom-network-fixes/; revision=35439
2008-08-18 23:41:23 +00:00
Cameron Gutman e57331851c - Don't hardcode the Ioctl
svn path=/branches/aicom-network-fixes/; revision=35438
2008-08-18 23:35:32 +00:00
Cameron Gutman bd043fd1cf - Call MmProbeAndLockPages for SysMdl
svn path=/branches/aicom-network-fixes/; revision=35431
2008-08-18 14:00:27 +00:00
Cameron Gutman 6634c6a919 - Handle STATUS_FILE_CLOSED
svn path=/branches/aicom-network-fixes/; revision=35420
2008-08-18 00:05:05 +00:00
Cameron Gutman 8b67a0b644 - Change STATUS_UNSUCCESSFUL to STATUS_FILE_CLOSED
- Move the Irp->Cancel check after SocketAcquireStateLock to fix a possible crash
 - Change STATUS_INVALID_PARAMETER to STATUS_FILE_CLOSED
 - Move SocketAcquireStateLock in front of the check for a NULL return from LockRequest to prevent another possible crash

svn path=/branches/aicom-network-fixes/; revision=35419
2008-08-17 22:12:31 +00:00
Cameron Gutman ec037e4d9d - Call SocketStateUnlock before returning
- Don't complete the IRP again in the completion routine

svn path=/branches/aicom-network-fixes/; revision=35418
2008-08-17 21:48:09 +00:00
Cameron Gutman e070cbeb39 - Add debug prints for failure statuses
svn path=/branches/aicom-network-fixes/; revision=35404
2008-08-17 08:17:03 +00:00
Cameron Gutman 54a02c3ea8 - Make sure we successfully allocated ConnectionReturnInfo and ConnectionCallInfo
- Make sure ObReferenceObjectByHandle was successful

svn path=/branches/aicom-network-fixes/; revision=35398
2008-08-16 21:16:41 +00:00