This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/791/ |
On August 30th, 2013, 12:08 p.m. UTC, Radek Novacek wrote:
src/account/LMI_AccountProvider.c (Diff revision 2) static CMPIStatus LMI_AccountModifyInstance(464 exit(1);476 exit(1);Don't use exit(1) in the provider, IMHO it might cause the caller to hang. Rather return some error message.
Created ticket for it: https://fedorahosted.org/openlmi/ticket/146 Will do fix for all of this issues.
- Roman
On August 30th, 2013, 3:27 p.m. UTC, Roman Rakus wrote:
Review request for OpenLMI Developers.
By Roman Rakus.
Updated Aug. 30, 2013, 3:27 p.m.
Repository:
openlmi-providers
Description
Diffs
|