diff options
author | Mark Wielaard <mark@klomp.org> | 2013-07-17 12:11:26 +0200 |
---|---|---|
committer | David Tardon <dtardon@redhat.com> | 2013-07-17 10:57:21 +0000 |
commit | cae861f0b32e01cb775214d783c98255acb60af0 (patch) | |
tree | f94d5786080416ef396cc19a4826ab1e409b42ec /sal/qa/osl | |
parent | 4cf02127888589d66c90286e9994b5cf07ea3cab (diff) |
Fix memory leak with osl_getCurrentSecurity ().
The result of osl_getCurrentSecurity () should always be deleted again
with osl_freeSecurityHandle ().
Change-Id: If0991937fcb24207d1f78166f61c4be22d423629
Reviewed-on: https://gerrit.libreoffice.org/4947
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
Diffstat (limited to 'sal/qa/osl')
-rw-r--r-- | sal/qa/osl/process/osl_process.cxx | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/sal/qa/osl/process/osl_process.cxx b/sal/qa/osl/process/osl_process.cxx index f09567659552..5dcd55561884 100644 --- a/sal/qa/osl/process/osl_process.cxx +++ b/sal/qa/osl/process/osl_process.cxx @@ -138,16 +138,18 @@ public: void osl_joinProcessWithTimeout_timeout_failure() { oslProcess process; + oslSecurity security = osl_getCurrentSecurity(); oslProcessError osl_error = osl_executeProcess( suExecutableFileURL.pData, parameters_, parameters_count_, osl_Process_NORMAL, - osl_getCurrentSecurity(), + security, suCWD.pData, NULL, 0, &process); + osl_freeSecurityHandle(security); CPPUNIT_ASSERT_EQUAL_MESSAGE ( @@ -188,16 +190,18 @@ public: void osl_joinProcessWithTimeout_without_timeout_failure() { oslProcess process; + oslSecurity security = osl_getCurrentSecurity (); oslProcessError osl_error = osl_executeProcess( suExecutableFileURL.pData, parameters_, parameters_count_, osl_Process_NORMAL, - osl_getCurrentSecurity(), + security, suCWD.pData, NULL, 0, &process); + osl_freeSecurityHandle(security); CPPUNIT_ASSERT_EQUAL_MESSAGE ( @@ -228,16 +232,18 @@ public: void osl_joinProcessWithTimeout_infinite() { oslProcess process; + oslSecurity security = osl_getCurrentSecurity (); oslProcessError osl_error = osl_executeProcess( suExecutableFileURL.pData, parameters_, parameters_count_, osl_Process_NORMAL, - osl_getCurrentSecurity(), + security, suCWD.pData, NULL, 0, &process); + osl_freeSecurityHandle(security); CPPUNIT_ASSERT_EQUAL_MESSAGE ( @@ -264,16 +270,18 @@ public: void osl_joinProcess() { oslProcess process; + oslSecurity security = osl_getCurrentSecurity (); oslProcessError osl_error = osl_executeProcess( suExecutableFileURL.pData, parameters_, parameters_count_, osl_Process_NORMAL, - osl_getCurrentSecurity(), + security, suCWD.pData, NULL, 0, &process); + osl_freeSecurityHandle(security); CPPUNIT_ASSERT_EQUAL_MESSAGE ( |