Merge pull request #264 from ACRIOS-Systems/feature/fix_codec_nullptr_dereferencing

MichalPrincNXP · web-flow · commit bbcb60f3f093 · 2022-03-31T23:03:05.000+02:00
Fixed codec nullptr dereferencing.
diff --git a/erpc_c/infra/erpc_client_manager.cpp b/erpc_c/infra/erpc_client_manager.cpp
@@ -212,8 +212,11 @@ Codec *ClientManager::createBufferAndCodec(void)
 
 void ClientManager::releaseRequest(RequestContext &request)
 {
-    m_messageFactory->dispose(request.getCodec()->getBuffer());
-    m_codecFactory->dispose(request.getCodec());
+    if (request.getCodec() != NULL)
+    {
+        m_messageFactory->dispose(request.getCodec()->getBuffer());
+        m_codecFactory->dispose(request.getCodec());
+    }
 }
 
 void ClientManager::callErrorHandler(erpc_status_t err, uint32_t functionID)

Original file line number	Diff line number	Diff line change
`@@ -212,8 +212,11 @@ Codec *ClientManager::createBufferAndCodec(void)`
`212`	`212`
`213`	`213`	`void ClientManager::releaseRequest(RequestContext &request)`
`214`	`214`	`{`
`215`		`- m_messageFactory->dispose(request.getCodec()->getBuffer());`
`216`		`- m_codecFactory->dispose(request.getCodec());`
	`215`	`+ if (request.getCodec() != NULL)`
	`216`	`+ {`
	`217`	`+ m_messageFactory->dispose(request.getCodec()->getBuffer());`
	`218`	`+ m_codecFactory->dispose(request.getCodec());`
	`219`	`+ }`
`217`	`220`	`}`
`218`	`221`
`219`	`222`	`void ClientManager::callErrorHandler(erpc_status_t err, uint32_t functionID)`