From: Tony S. <Ton...@na...> - 2011-03-01 14:26:05
|
Hello, I seem to have found a memory leak in the Umfpack module, more specifically in the function factorize (umfpackmodule.c). This calls the function newUMFPackObject, in which (as far as I understand it) memory is allocated for the variable colidx via SpMatrix_LLMatBuildColIndex(&colidx, matrix, 1). This memory is never freed. Calling SpMatrix_LLMatDestroyColIndex(&colidx) towards the end of newUMFPackObject seems to resolve the issue, but I am uncertain where it is safe to make this call. I would guess that one should call it just before the two switch statements that check for problems with the matrix and return NULL if something is awry. colidx does not seem to be in use at this point, but someone with more insight into the code than me should confirm this. Kind regards, Tony Stillfjord |