Un-needed check for null pointer before free

[amgross]

At least twice I see checking of null pointer before free it:

erpc/erpcgen/src/templates/c_common_functions.template

Lines 380 to 383 in 340dd88

	if ({$info.freeName})
	{
	erpc_free({$info.freeName});
	}

erpc/erpcgen/src/templates/c_common_functions.template

Lines 403 to 406 in 340dd88

	if ({$info.name})
	{
	erpc_free({$info.name});
	}

Can't we assume that erpc_free know to handle properly null pointer like the c standard library free function?

It is not bug, just extra code and run-time.

[Hadatko]

Hi @amgross , you are correct i think we can get rid of these checks and add comment to erpc_free function that NULL pointer is valid and should be handled extra inside. Which in case of calling free is not necessary...

May i let you create PR?

[amgross]

OK :)