-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/825/#review1095
-----------------------------------------------------------
Ship it!
Ship It!
- Michal Minar
On Sept. 4, 2013, 2:51 p.m., Roman Rakus wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/825/
-----------------------------------------------------------
(Updated Sept. 4, 2013, 2:51 p.m.)
Review request for OpenLMI Developers.
Repository: openlmi-providers
Description
-------
Account: Fix memory corruption
There was false assumption that lu_end doesn't free error messages.
The code is restructured to copy error message to temporary string
holder, which is freed after and error message is correctly returned,
all in one place.
Signed-off-by: Roman Rakus <rrakus(a)redhat.com>
Diffs
-----
src/account/LMI_AccountProvider.c c58b001c52fb7e84425e403d02779efb172a5122
src/account/LMI_GroupProvider.c d3263f47fb8366703e334732dedc86c11b446974
src/account/LMI_IdentityProvider.c e008746e89ba7eab4e9a24c5647c048a88a51d8e
src/account/LMI_MemberOfGroupProvider.c b06cd437c6e80d759565d6746b9b402133793170
Diff:
http://reviewboard-openlmi.rhcloud.com/r/825/diff/
Testing
-------
Thanks,
Roman Rakus