From: Rob Pike Date: Sat, 6 Oct 2012 22:26:59 +0000 (+1100) Subject: text/template: fix nil crash on Templates X-Git-Tag: go1.1rc2~2239 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=bcccad40202ba895d237d9d0a921b33bc2c5601f;p=gostls13.git text/template: fix nil crash on Templates Fixes #3872. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6612060 --- diff --git a/src/pkg/text/template/exec_test.go b/src/pkg/text/template/exec_test.go index d79365107d..683e9ac76b 100644 --- a/src/pkg/text/template/exec_test.go +++ b/src/pkg/text/template/exec_test.go @@ -811,3 +811,8 @@ func TestTree(t *testing.T) { t.Errorf("expected %q got %q", expect, result) } } + +func TestExecuteOnNewTemplate(t *testing.T) { + // This is issue 3872. + _ = New("Name").Templates() +} diff --git a/src/pkg/text/template/template.go b/src/pkg/text/template/template.go index 82fc9e5e39..a2b9062ad1 100644 --- a/src/pkg/text/template/template.go +++ b/src/pkg/text/template/template.go @@ -117,6 +117,9 @@ func (t *Template) AddParseTree(name string, tree *parse.Tree) (*Template, error // Templates returns a slice of the templates associated with t, including t // itself. func (t *Template) Templates() []*Template { + if t.common == nil { + return nil + } // Return a slice so we don't expose the map. m := make([]*Template, 0, len(t.tmpl)) for _, v := range t.tmpl {