Fix #131: Return error on pageNumber = 0

unidoc · Dec 12, 2017 · 6613014 · 6613014
1 parent 978d94d
commit 6613014
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/pdf/model/reader.go b/pdf/model/reader.go
@@ -724,7 +724,11 @@ func (this *PdfReader) GetPage(pageNumber int) (*PdfPage, error) {
 	if len(this.pageList) < pageNumber {
 		return nil, errors.New("Invalid page number (page count too short)")
 	}
-	page := this.PageList[pageNumber-1]
+	idx := pageNumber - 1
+	if idx < 0 {
+		return nil, fmt.Errorf("Page numbering must start at 1")
+	}
+	page := this.PageList[idx]
 
 	return page, nil
 }