CopyFileEx - Just want a code-review to make sure I'm doing this correctly

[RFBomb]

I'm trying to work with CopyFileEx, and it does work, but since it is using void* i was a bit unsure of proper best-practices.

So I have CopyFileEx in my NativeMethods, and here is my actual code used to invoke it. Any recommendations?

public static bool CopyFile(string source, string destination, CopyFileExOptions flags = CopyFileExOptions.NONE, CopyProgressCallback progressCallback = null)
        {
            if (!File.Exists(source)) throw new FileNotFoundException("Source File Not Found.", source);
            _ = Directory.CreateDirectory(Path.GetDirectoryName(destination));
            return InvokeCopyFileEx(source, destination, progressCallback, flags);
        }

internal static unsafe bool InvokeCopyFileEx(
            string source,
            string destination,
            CopyProgressCallback progressCallback,
            CopyFileExOptions options)
        {
            VersionManager.ThrowIfNotWindowsPlatform("P/Invoke.CopyFileEx is only available on Windows");
            LPPROGRESS_ROUTINE callback = null;
            if (progressCallback != null) callback = CopyCallback;

            var returnValue = Win32.PInvoke.CopyFileEx(source, destination, callback, lpData: null, pbCancel: null, dwCopyFlags: (uint)options);
            if (returnValue.Value == 0) Win32Error.ThrowLastError(source, destination);
            return File.Exists(destination);

            unsafe uint CopyCallback(long totalFileSize, long totalBytesTransferred, long streamSize, long streamBytesTransferred, uint dwStreamNumber, LPPROGRESS_ROUTINE_CALLBACK_REASON dwCallbackReason, Win32.Foundation.HANDLE hSourceFile, Win32.Foundation.HANDLE hDestinationFile, void* lpData)
            {
                return (uint)progressCallback(totalFileSize, totalBytesTransferred, streamSize, streamBytesTransferred, dwStreamNumber, (CopyProgressCallbackReason)dwCallbackReason);
            }
        }

public delegate CopyProgressCallbackResult CopyProgressCallback(
        long totalFileSize,
        long totalBytesTransferred,
        long streamSize,
        long streamBytesTransferred,
        uint streamID,
        CopyProgressCallbackReason reason
        //,IntPtr hSourceFile
        //,IntPtr hDestinationFile
        //,IntPtr data
        );

 public enum CopyProgressCallbackReason : uint
    {
        CALLBACK_CHUNK_FINISHED = LPPROGRESS_ROUTINE_CALLBACK_REASON.CALLBACK_CHUNK_FINISHED,
        CALLBACK_STREAM_SWITCH = LPPROGRESS_ROUTINE_CALLBACK_REASON.CALLBACK_STREAM_SWITCH
    }

public enum CopyProgressCallbackResult : uint
    {
        CONTINUE = 0U,
        CANCEL = 1U,
        STOP = 2U,
        QUIET = 3U
    }

[AArnott]

I'm not familiar with the API itself, but what you have here looks reasonable to me.

[RFBomb]

Appreciate the feedback. I figured it would be fine, as I have no intentions to expose the pointers to any callers, just using it for progress data really. Just wanted to ensure passing null there wasn't going to break anything