Version 2.5.0d4 - Fixes a problem which may cause the kext to refuse unload requests.

Author: Mieze

RealtekRTL8111.xcodeproj/project.pbxproj

@@ -3668,7 +3668,7 @@
 				CLANG_WARN_QUOTED_INCLUDE_IN_FRAMEWORK_HEADER = YES;
 				CLANG_WARN_UNGUARDED_AVAILABILITY = YES_AGGRESSIVE;
 				CODE_SIGN_STYLE = Manual;
-				CURRENT_PROJECT_VERSION = 2.5.0d3;
+				CURRENT_PROJECT_VERSION = 2.5.0d4;
 				DEBUG_INFORMATION_FORMAT = dwarf;
 				DEVELOPMENT_TEAM = FZX87E5QBD;
 				"DEVELOPMENT_TEAM[sdk=macosx*]" = "";
@@ -3682,7 +3682,7 @@
 				INFOPLIST_KEY_NSHumanReadableCopyright = "Copyright © 2013 Laura Müller. All rights reserved.";
 				LOCALIZATION_PREFERS_STRING_CATALOGS = YES;
 				MACOSX_DEPLOYMENT_TARGET = 10.13;
-				MARKETING_VERSION = 2.5.0d3;
+				MARKETING_VERSION = 2.5.0d4;
 				MODULE_NAME = com.insanelymac.RealtekRTL8111;
 				MODULE_START = 0;
 				MODULE_STOP = 0;
@@ -3715,7 +3715,7 @@
 				CLANG_WARN_UNGUARDED_AVAILABILITY = YES_AGGRESSIVE;
 				CODE_SIGN_STYLE = Manual;
 				COPY_PHASE_STRIP = NO;
-				CURRENT_PROJECT_VERSION = 2.5.0d3;
+				CURRENT_PROJECT_VERSION = 2.5.0d4;
 				DEVELOPMENT_TEAM = FZX87E5QBD;
 				"DEVELOPMENT_TEAM[sdk=macosx*]" = "";
 				ENABLE_NS_ASSERTIONS = NO;
@@ -3729,7 +3729,7 @@
 				INFOPLIST_KEY_NSHumanReadableCopyright = "Copyright © 2013 Laura Müller. All rights reserved.";
 				LOCALIZATION_PREFERS_STRING_CATALOGS = YES;
 				MACOSX_DEPLOYMENT_TARGET = 10.13;
-				MARKETING_VERSION = 2.5.0d3;
+				MARKETING_VERSION = 2.5.0d4;
 				MODULE_NAME = com.insanelymac.RealtekRTL8111;
 				MODULE_START = 0;
 				MODULE_STOP = 0;

RealtekRTL8111.xcodeproj/project.xcworkspace/xcuserdata/laura.xcuserdatad/UserInterfaceState.xcuserstate

@@ -135,9 +135,6 @@ void RTL8111::free()
     linuxData.mmio_addr = NULL;
 
     RELEASE(pciDevice);
-    freeTxResources();
-    freeRxResources();
-    freeStatResources();
 
     DebugLog("free() <===\n");
 

RealtekRTL8111/RealtekRTL8111.cpp

@@ -525,7 +525,13 @@ bool RTL8111::setupStatResources()
 void RTL8111::freeRxResources()
 {
     UInt32 i;
-        
+    
+    if (rxDescDmaCmd) {
+        rxDescDmaCmd->complete();
+        rxDescDmaCmd->clearMemoryDescriptor();
+        rxDescDmaCmd->release();
+        rxDescDmaCmd = NULL;
+    }
     if (rxBufDesc) {
         rxBufDesc->complete();
         rxBufDesc->release();
@@ -540,31 +546,27 @@ void RTL8111::freeRxResources()
             rxMbufArray[i] = NULL;
         }
     }
-    if (rxDescDmaCmd) {
-        rxDescDmaCmd->clearMemoryDescriptor();
-        rxDescDmaCmd->release();
-        rxDescDmaCmd = NULL;
-    }
     if (rxBufArrayMem) {
-        IOFree(txBufArrayMem, kRxBufArraySize);
+        IOFree(rxBufArrayMem, kRxBufArraySize);
         rxBufArrayMem = NULL;
         rxMbufArray = NULL;
     }
 }
 
 void RTL8111::freeTxResources()
 {
+    if (txDescDmaCmd) {
+        txDescDmaCmd->complete();
+        txDescDmaCmd->clearMemoryDescriptor();
+        txDescDmaCmd->release();
+        txDescDmaCmd = NULL;
+    }
     if (txBufDesc) {
         txBufDesc->complete();
         txBufDesc->release();
         txBufDesc = NULL;
         txPhyAddr = (IOPhysicalAddress64)NULL;
     }
-    if (txDescDmaCmd) {
-        txDescDmaCmd->clearMemoryDescriptor();
-        txDescDmaCmd->release();
-        txDescDmaCmd = NULL;
-    }
     if (txBufArrayMem) {
         IOFree(txBufArrayMem, kTxBufArraySize);
         txBufArrayMem = NULL;
@@ -575,17 +577,18 @@ void RTL8111::freeTxResources()
 
 void RTL8111::freeStatResources()
 {
+    if (statDescDmaCmd) {
+        statDescDmaCmd->complete();
+        statDescDmaCmd->clearMemoryDescriptor();
+        statDescDmaCmd->release();
+        statDescDmaCmd = NULL;
+    }
     if (statBufDesc) {
         statBufDesc->complete();
         statBufDesc->release();
         statBufDesc = NULL;
         statPhyAddr = (IOPhysicalAddress64)NULL;
     }
-    if (statDescDmaCmd) {
-        statDescDmaCmd->clearMemoryDescriptor();
-        statDescDmaCmd->release();
-        statDescDmaCmd = NULL;
-    }
 }