On 8/27/2018 9:48 PM, Honggang LI wrote: > From: Honggang Li <honli@xxxxxxxxxx> > > Issue was found by Coverity. > > Error: RESOURCE_LEAK (CWE-772): [#def13] > opensm-3.3.20/osmtest/osmtest.c:6994: alloc_fn: Storage is returned from allocation function "path_new". > opensm-3.3.20/osmtest/include/osmtest_subnet.h:254:2: alloc_fn: Storage is returned from allocation function "malloc". > opensm-3.3.20/osmtest/include/osmtest_subnet.h:254:2: var_assign: Assigning: "p_obj" = "malloc(192UL)". > opensm-3.3.20/osmtest/include/osmtest_subnet.h:256:3: noescape: Resource "p_obj" is not freed or pointed-to in function "memset". [Note: The source code implementation of the function has been overridden by a builtin model.] > opensm-3.3.20/osmtest/include/osmtest_subnet.h:257:2: return_alloc: Returning allocated memory "p_obj". > opensm-3.3.20/osmtest/osmtest.c:6994: var_assign: Assigning: "p_path" = storage returned from "path_new()". > opensm-3.3.20/osmtest/osmtest.c:7136: leaked_storage: Variable "p_path" going out of scope leaks the storage it points to. > |# 7134| Exit: > |# 7135| OSM_LOG_EXIT(&p_osmt->log); > |# 7136|-> return (status); > |# 7137| } > |# 7138| > > Signed-off-by: Honggang Li <honli@xxxxxxxxxx> Thanks. Applied. -- Hal
