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

Account: New delete method

This method should be used insted of intrinsic DeleteInstace.

Signed-off-by: Roman Rakus <rrakus@redhat.com>

Diffs

  • mof/60_LMI_Account.mof (0a79f59b222b75ca8e0e3ce8f1e4249e50052251)
  • src/account/LMI_AccountProvider.c (97279234f7da4866d48bf8effee8edb5b59cfa1c)

View Diff