CORTX-33743: Analyze and fix memory leaks found in glibc when running…

… s3 IOs with valgrind

Problem statement:
Valgrind reported few leaks in the code, where strdup was used for allocating memory to the duplicate string.

Fix Description:
Analyzed the code statically and freed up the variables allocated memory dynamically during the execution.
Attaching the valrind report with the code changes, where no leak is found wrt the leaks described of the JIRA ticket.

ha/entrypoint.c

@@ -600,6 +600,7 @@ static int ha_entrypoint_client_fom_tick(struct m0_fom *fom)
 		fop->f_opaque = ecl;
 		next_state = M0_HEC_SEND_WAIT;
 		rc = m0_rpc_post(item) == 0 ? M0_FSO_WAIT : M0_FSO_AGAIN;
+		m0_free(req_fop_data->erf_git_rev_id.b_addr);
 		break;
 
 	case M0_HEC_SEND_WAIT:

motr/setup.c

@@ -1099,8 +1099,10 @@ static int cs_storage_init(const char *stob_type,
 /**
    Finalises storage for a request handler in a motr context.
  */
-static void cs_storage_fini(struct cs_stobs *stob)
+static void cs_storage_fini(struct m0_reqh_context *rctx)
 {
+	struct cs_stobs *stob = &rctx->rc_stob;
+	m0_free((char*)rctx->rc_addb_stlocation);
 	cs_storage_devs_fini();
 	if (stob->s_sdom != NULL)
 		m0_stob_domain_fini(stob->s_sdom);
@@ -1687,7 +1689,7 @@ static int cs_storage_setup(struct m0_motr *cctx)
 cleanup_addb2:
 	m0_reqh_addb2_fini(&rctx->rc_reqh);
 cleanup_stob:
-	cs_storage_fini(&rctx->rc_stob);
+	cs_storage_fini(rctx);
 reqh_be_fini:
 	m0_reqh_be_fini(&rctx->rc_reqh);
 be_fini:
@@ -2874,7 +2876,7 @@ static void cs_level_leave(struct m0_module *module)
 		break;
 	case CS_LEVEL_STORAGE_SETUP:
 		if (rctx->rc_state == RC_INITIALISED) {
-			cs_storage_fini(&rctx->rc_stob);
+			cs_storage_fini(rctx);
 			rctx->rc_reqh.rh_pools = NULL;
 		}
 		break;