Re: [PATCH v8 0/3] MR414: fltmgr.sys: Implement FltBuildDefaultSecurityDescriptor

[Chip Davis (＠cdavis5e)]

Chip Davis (@cdavis5e) commented about dlls/fltmgr.sys/main.c:
> - if (ret == STATUS_SUCCESS)
> -
>    *descriptor = sec_desc;
> -
>
> +done:
> - if (ret != STATUS_SUCCESS)
> -
>    ExFreePool(sec_desc);
> -
> - ExFreePool(sid);
> - ExFreePool(sid_system);
> -
> - return ret;
>
> +}
> -
>
> +void WINAPI FltFreeSecurityDescriptor(PSECURITY_DESCRIPTOR descriptor)
> +{
> - RtlFreeHeap(GetProcessHeap(), 0, descriptor);

You allocated this above with `ExAllocatePool()`. Shouldn't this then use `ExFreePool()`?