fix: add guards for MagickImage.BilateralBlur (#1545)

Author: Gounlaf

src/Magick.NET.Core/IMagickImage.cs

@@ -559,7 +559,8 @@ Interlace Interlace
     /// Applies a non-linear, edge-preserving, and noise-reducing smoothing filter.
     /// </summary>
     /// <param name="width">The width of the neighborhood in pixels.</param>
-    /// <param name="height">The height of the neighborhood in pixels.</param>\
+    /// <param name="height">The height of the neighborhood in pixels.</param>
+    /// <exception cref="MagickException">Thrown when an error is raised by ImageMagick.</exception>
     void BilateralBlur(int width, int height);
 
     /// <summary>
@@ -569,6 +570,7 @@ Interlace Interlace
     /// <param name="height">The height of the neighborhood in pixels.</param>
     /// <param name="intensitySigma">The sigma in the intensity space.</param>
     /// <param name="spatialSigma">The sigma in the coordinate space.</param>
+    /// <exception cref="MagickException">Thrown when an error is raised by ImageMagick.</exception>
     void BilateralBlur(int width, int height, double intensitySigma, double spatialSigma);
 
     /// <summary>

src/Magick.NET/MagickImage.cs

@@ -1314,8 +1314,12 @@ public void AutoThreshold(AutoThresholdMethod method)
     /// </summary>
     /// <param name="width">The width of the neighborhood in pixels.</param>
     /// <param name="height">The height of the neighborhood in pixels.</param>
+    /// <exception cref="MagickException">Thrown when an error is raised by ImageMagick.</exception>
     public void BilateralBlur(int width, int height)
     {
+        Throw.IfNegative(nameof(width), width);
+        Throw.IfNegative(nameof(height), height);
+
         var intensitySigma = Math.Sqrt((width * width) + (height * height));
         BilateralBlur(width, height, intensitySigma, intensitySigma * 0.25);
     }
@@ -1327,8 +1331,14 @@ public void BilateralBlur(int width, int height)
     /// <param name="height">The height of the neighborhood in pixels.</param>
     /// <param name="intensitySigma">The sigma in the intensity space.</param>
     /// <param name="spatialSigma">The sigma in the coordinate space.</param>
+    /// <exception cref="MagickException">Thrown when an error is raised by ImageMagick.</exception>
     public void BilateralBlur(int width, int height, double intensitySigma, double spatialSigma)
-       => _nativeInstance.BilateralBlur(width, height, intensitySigma, spatialSigma);
+    {
+        Throw.IfNegative(nameof(width), width);
+        Throw.IfNegative(nameof(height), height);
+
+        _nativeInstance.BilateralBlur(width, height, intensitySigma, spatialSigma);
+    }
 
     /// <summary>
     /// Forces all pixels below the threshold into black while leaving all pixels at or above

tests/Magick.NET.Tests/MagickImageTests/TheBilateralBlurMethod.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,34 @@ public partial class MagickImageTests
 {
     public class TheBilateralBlurMethod
     {
+        [Fact]
+        public void ShouldThrowExceptionWhenWidthIsNegative()
+        {
+            using var image = new MagickImage(Files.NoisePNG);
+            Assert.Throws<ArgumentException>("width", () => image.BilateralBlur(-1, 2));
+        }
+
+        [Fact]
+        public void ShouldThrowExceptionWhenWidthIsNegativeThanOneWithLowAndHigh()
+        {
+            using var image = new MagickImage(Files.NoisePNG);
+            Assert.Throws<ArgumentException>("width", () => image.BilateralBlur(-1, 2, 0.1, 0.1));
+        }
+
+        [Fact]
+        public void ShouldThrowExceptionWhenHeightIsNegative()
+        {
+            using var image = new MagickImage(Files.NoisePNG);
+            Assert.Throws<ArgumentException>("height", () => image.BilateralBlur(2, -1));
+        }
+
+        [Fact]
+        public void ShouldThrowExceptionWhenHeightIsNegativeWithLowAndHigh()
+        {
+            using var image = new MagickImage(Files.NoisePNG);
+            Assert.Throws<ArgumentException>("height", () => image.BilateralBlur(2, -1, 0.1, 0.1));
+        }
+
         [Fact]
         public void ShouldApplyTheFilter()
         {