From 874f89dba47d254fa82f0efd7c5da88e309a5431 Mon Sep 17 00:00:00 2001
From: Adam Thomason <ad@mthomason.net>
Date: Tue, 15 Nov 2016 15:39:27 -0800
Subject: [PATCH] Don't resize inputs with zero width or height

Inputs having zero as both width and height were caught, but either one
alone caused a panic. This adds a test showing the old failing behavior
and adds a check for non-positive width/height. Since Resize doesn't
have an error return value, the behavior in this case is to return the
original image.

Fixes https://github.com/nfnt/resize/issues/52
---
 resize.go      |  6 ++++++
 resize_test.go | 14 ++++++++++++++
 2 files changed, 20 insertions(+)

diff --git a/resize.go b/resize.go
index 57bd1fc..0d7fbf6 100644
--- a/resize.go
+++ b/resize.go
@@ -78,6 +78,7 @@ var blur = 1.0
 // If one of the parameters width or height is set to 0, its size will be calculated so that
 // the aspect ratio is that of the originating image.
 // The resizing algorithm uses channels for parallel computation.
+// If the input image has width or height of 0, it is returned unchanged.
 func Resize(width, height uint, img image.Image, interp InterpolationFunction) image.Image {
 	scaleX, scaleY := calcFactors(width, height, float64(img.Bounds().Dx()), float64(img.Bounds().Dy()))
 	if width == 0 {
@@ -92,6 +93,11 @@ func Resize(width, height uint, img image.Image, interp InterpolationFunction) i
 		return img
 	}
 
+	// Input image has no pixels
+	if img.Bounds().Dx() <= 0 || img.Bounds().Dy() <= 0 {
+		return img
+	}
+
 	if interp == NearestNeighbor {
 		return resizeNearest(width, height, scaleX, scaleY, img, interp)
 	}
diff --git a/resize_test.go b/resize_test.go
index d4b80be..178740e 100644
--- a/resize_test.go
+++ b/resize_test.go
@@ -37,6 +37,20 @@ func Test_ZeroImg(t *testing.T) {
 	}
 }
 
+func Test_HalfZeroImg(t *testing.T) {
+	zeroImg := image.NewGray16(image.Rect(0, 0, 0, 100))
+
+	m := Resize(0, 1, zeroImg, NearestNeighbor)
+	if m.Bounds() != zeroImg.Bounds() {
+		t.Fail()
+	}
+
+	m = Resize(1, 0, zeroImg, NearestNeighbor)
+	if m.Bounds() != zeroImg.Bounds() {
+		t.Fail()
+	}
+}
+
 func Test_CorrectResize(t *testing.T) {
 	zeroImg := image.NewGray16(image.Rect(0, 0, 256, 256))