netkvm: exclude the entire NBL history list if not defined

ybendito · ybendito · commit 87b7cb7a52b0 · 2025-05-18T14:21:49.000+03:00
This does not matter for functionality but in the NBL
destructor the verifier should not check the spinlock
of history list, which is not used in any other place.

Signed-off-by: Yuri Benditovich &lt;yuri.benditovich@daynix.com&gt;
diff --git a/NetKVM/Common/ParaNdis-TX.h b/NetKVM/Common/ParaNdis-TX.h
@@ -423,7 +423,9 @@ class CNBL : public CNdisAllocatableViaHelper<CNBL>, public CRefCountingObject,
 #endif
     CAllocationHelper<CNB> *m_NBAllocator;
 
+#if NBL_MAINTAIN_HISTORY
     CHistoryList m_History;
+#endif
 
     CChainOfNbls m_Chain = {};
 
diff --git a/NetKVM/Common/ParaNdis_TX.cpp b/NetKVM/Common/ParaNdis_TX.cpp
@@ -51,7 +51,9 @@ CNBL::~CNBL()
         }
     }
 
+#if NBL_MAINTAIN_HISTORY
     m_History.ForEachDetached([this](NBLHistory *Entry) { NBLHistory::Destroy(Entry, m_Context->MiniportHandle); });
+#endif
 }
 
 bool CNBL::ParsePriority()

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,9 @@ CNBL::~CNBL()`
`51`	`51`	`}`
`52`	`52`	`}`
`53`	`53`
	`54`	`+#if NBL_MAINTAIN_HISTORY`
`54`	`55`	`m_History.ForEachDetached([this](NBLHistory *Entry) { NBLHistory::Destroy(Entry, m_Context->MiniportHandle); });`
	`56`	`+#endif`
`55`	`57`	`}`
`56`	`58`
`57`	`59`	`bool CNBL::ParsePriority()`