forked from auracaster/openocd
target: openrisc: jsp_server: check for out of memory, fix memory leak
During JSP service start, check that memory is properly allocated and if add_service() fails release the allocated memory. Change-Id: I7eb2ea8a10dbeccd27fd39485080552c5d0736d3 Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/9375 Tested-by: jenkins Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
This commit is contained in:
@@ -195,10 +195,19 @@ static const struct service_driver jsp_service_driver = {
|
||||
int jsp_init(struct or1k_jtag *jtag_info, char *banner)
|
||||
{
|
||||
struct jsp_service *jsp_service = malloc(sizeof(struct jsp_service));
|
||||
if (!jsp_service) {
|
||||
LOG_ERROR("Out of memory");
|
||||
return ERROR_FAIL;
|
||||
}
|
||||
|
||||
jsp_service->banner = banner;
|
||||
jsp_service->jtag_info = jtag_info;
|
||||
|
||||
return add_service(&jsp_service_driver, jsp_port, 1, jsp_service);
|
||||
int retval = add_service(&jsp_service_driver, jsp_port, 1, jsp_service);
|
||||
if (retval != ERROR_OK)
|
||||
free(jsp_service);
|
||||
|
||||
return retval;
|
||||
}
|
||||
|
||||
COMMAND_HANDLER(handle_jsp_port_command)
|
||||
|
||||
Reference in New Issue
Block a user