Use Marshal.Copy on the buffer

Susko3 · Apr 18, 2024 · a6f3944 · a6f3944
1 parent c80ec2d
commit a6f3944
Showing 1 changed file with 4 additions and 9 deletions.
diff --git a/osu.Framework/Platform/Windows/WindowsClipboard.cs b/osu.Framework/Platform/Windows/WindowsClipboard.cs
@@ -94,16 +94,11 @@ public override bool SetImage(Image image)
             {
                 var encoder = image.Configuration.ImageFormatsManager.GetEncoder(BmpFormat.Instance);
                 image.Save(stream, encoder);
-                var span = new ReadOnlySpan<byte>(stream.GetBuffer(), bitmap_file_header_length, (int)stream.Length - bitmap_file_header_length);
-                IntPtr unmanagedPointer = Marshal.AllocHGlobal(span.Length);
 
-                unsafe
-                {
-                    fixed (void* source = span)
-                        Buffer.MemoryCopy(source, unmanagedPointer.ToPointer(), span.Length, span.Length);
-                }
-
-                return setClipboard(unmanagedPointer, span.Length, cf_dib);
+                int bitmapDataLength = (int)stream.Length - bitmap_file_header_length;
+                IntPtr unmanagedPointer = Marshal.AllocHGlobal(bitmapDataLength);
+                Marshal.Copy(stream.GetBuffer(), bitmap_file_header_length, unmanagedPointer, bitmapDataLength);
+                return setClipboard(unmanagedPointer, bitmapDataLength, cf_dib);
             }
         }