plugin: do not leak cRelName on error path

Fixes #17683

Change-Id: I46f45c63796b58e8a8f14e37592231cbe7cd6934
Reviewed-on: https://go-review.googlesource.com/32438
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>

diff --git a/src/plugin/plugin_dlopen.go b/src/plugin/plugin_dlopen.go
index 4a3eb3d8618fc6de88298535806111be3fb22906..f4addde74ce453107b3c47ee8b812c7ba4d579a5 100644 (file)
--- a/src/plugin/plugin_dlopen.go
+++ b/src/plugin/plugin_dlopen.go
@@ -44,10 +44,10 @@ func open(name string) (*Plugin, error) {
        defer C.free(unsafe.Pointer(cPath))
 
        cRelName := C.CString(name)
+       defer C.free(unsafe.Pointer(cRelName))
        if C.realpath(cRelName, cPath) == nil {
                return nil, errors.New("plugin.Open(" + name + "): realpath failed")
        }
-       C.free(unsafe.Pointer(cRelName))
 
        filepath := C.GoString(cPath)