From a29619bb34b6dfdcbeda0c319c5870229550cd49 Mon Sep 17 00:00:00 2001
From: Florian Levis <levis.florian@gmail.com>
Date: Fri, 26 Apr 2024 17:42:05 +0200
Subject: [PATCH]  fix: add guards for MagickImage.MeanShift

---
 src/Magick.NET/MagickImage.cs                   |  7 ++++++-
 .../MagickImageTests/TheMeanShiftMethod.cs      | 17 +++++++++++++++++
 2 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/src/Magick.NET/MagickImage.cs b/src/Magick.NET/MagickImage.cs
index 6451a11b7d..e262688f60 100644
--- a/src/Magick.NET/MagickImage.cs
+++ b/src/Magick.NET/MagickImage.cs
@@ -4035,7 +4035,12 @@ public void MeanShift(int width, int height)
     /// <param name="height">The height of the pixels neighborhood.</param>
     /// <param name="colorDistance">The color distance.</param>
     public void MeanShift(int width, int height, Percentage colorDistance)
-        => _nativeInstance.MeanShift(width, height, PercentageHelper.ToQuantum(colorDistance));
+    {
+        Throw.IfNegative(nameof(width), width);
+        Throw.IfNegative(nameof(height), height);
+
+        _nativeInstance.MeanShift(width, height, PercentageHelper.ToQuantum(colorDistance));
+    }
 
     /// <summary>
     /// Filter image by replacing each pixel component with the median color in a circular neighborhood.
diff --git a/tests/Magick.NET.Tests/MagickImageTests/TheMeanShiftMethod.cs b/tests/Magick.NET.Tests/MagickImageTests/TheMeanShiftMethod.cs
index a44edad591..c21f7fd443 100644
--- a/tests/Magick.NET.Tests/MagickImageTests/TheMeanShiftMethod.cs
+++ b/tests/Magick.NET.Tests/MagickImageTests/TheMeanShiftMethod.cs
@@ -1,6 +1,7 @@
 // Copyright Dirk Lemstra https://github.com/dlemstra/Magick.NET.
 // Licensed under the Apache License, Version 2.0.
 
+using System;
 using ImageMagick;
 using Xunit;
 
@@ -10,6 +11,22 @@ public partial class MagickImageTests
 {
     public class TheMeanShiftMethod
     {
+        [Fact]
+        public void ShouldThrowExceptionWhenWidthIsNegative()
+        {
+            using var image = new MagickImage(Files.FujiFilmFinePixS1ProPNG);
+
+            Assert.Throws<ArgumentException>("width", () => image.MeanShift(-1, 1));
+        }
+
+        [Fact]
+        public void ShouldThrowExceptionWhenHeightIsNegative()
+        {
+            using var image = new MagickImage(Files.FujiFilmFinePixS1ProPNG);
+
+            Assert.Throws<ArgumentException>("height", () => image.MeanShift(1, -1));
+        }
+
         [Fact]
         public void ShouldNotChangeImageWhenSizeIsOne()
         {