check ReverseUrl arg index so templates dont panic

pbnjay · pbnjay · commit ea50b4cf0968 · 2015-02-23T23:59:09.000-05:00
addresses revel#811
diff --git a/template.go b/template.go
@@ -473,6 +473,11 @@ func ReverseUrl(args ...interface{}) (template.URL, error) {
 		return "", fmt.Errorf("reversing %s: %s", action, err)
 	}
 
+	if len(c.MethodType.Args) < len(args)-1 {
+		return "", fmt.Errorf("reversing %s: route defines %d args, but received %d",
+			action, len(c.MethodType.Args), len(args)-1)
+	}
+
 	// Unbind the arguments.
 	argsByName := make(map[string]string)
 	for i, argValue := range args[1:] {
