Fix build warnings and incorrect deprecation message (#88)

Author: SergiusTheBest

include/kf/FltCommunicationPort.h

@@ -120,7 +120,7 @@ namespace kf
                     return status;
                 }
 
-                const ULONG saclSize = sizeof(ACL) + lowMandatorySidLength + sizeof SYSTEM_MANDATORY_LABEL_ACE;
+                const ULONG saclSize = sizeof(ACL) + lowMandatorySidLength + sizeof(SYSTEM_MANDATORY_LABEL_ACE);
                 status = sacl.emplace(saclSize);
                 if (!NT_SUCCESS(status))
                 {

include/kf/FltResourceSharedLock.h

@@ -5,7 +5,7 @@ namespace kf
     //////////////////////////////////////////////////////////////////////////
     // FltResourceSharedLock
 
-    class [[deprecated("Use std::unique_lock instead")]] FltResourceSharedLock
+    class [[deprecated("Use std::shared_lock instead")]] FltResourceSharedLock
     {
     public:
         FltResourceSharedLock(_In_ PERESOURCE resource) : m_resource(FltResource::fromEResource(resource))

include/kf/GenericTableAvl.h

@@ -152,6 +152,9 @@ namespace kf
         __drv_allocatesMem(Mem)
         static void* NTAPI allocateRoutine(_In_ RTL_AVL_TABLE*, _In_ CLONG byteSize)
         {
+// 28160: Must succeed pool allocations are forbidden. Allocation failures cause a system crash.
+// 4996: ExAllocatePoolWithTag is deprecated, use ExAllocatePool2
+#pragma warning(suppress: 28160 4996) 
             return ::ExAllocatePoolWithTag(poolType, byteSize, PoolTag);
         }