From 49987bfa9f3f448e4a5ca031beeb1b1e0bd3be7d Mon Sep 17 00:00:00 2001 From: Jiajun Xin Date: Tue, 20 Nov 2018 20:29:59 +0800 Subject: [PATCH] remove the line ENGINE_free. In this function, the engine is not newed, so there should not be a free here. This line can cause nil pointer dereference when called. --- go/gmssl/engine.go | 1 - 1 file changed, 1 deletion(-) diff --git a/go/gmssl/engine.go b/go/gmssl/engine.go index ac88ed6c..21d198c2 100644 --- a/go/gmssl/engine.go +++ b/go/gmssl/engine.go @@ -95,7 +95,6 @@ func NewEngineByName(name string) (*Engine, error) { ret := &Engine{eng} runtime.SetFinalizer(ret, func(ret *Engine) { C.ENGINE_finish(ret.engine) - C.ENGINE_free(ret.engine) }) if 1 != C.ENGINE_init(eng) { return nil, GetErrors()