[Libvirt-cim] [PATCH V3 09/11] CSI Discard libvirt event by default

[John Ferlan]

Since I don't have the original email to reply-to, here is the link:

https://www.redhat.com/archives/libvirt-cim/2012-December/msg00032.html

Seems this was "pulled back" from some other commit - might be nice to
reference the commit id where it was removed and now replaced - at least
for historical purposes.


 src/Virt_ComputerSystem.c

In trigger_mod_indication()
 * The "goto out" if/when "(ind == NULL)" will perhaps trigger a false
positive this succeeded since "s" is initialized to CMPI_RC_OK.



 src/Virt_ComputerSystemIndication.c

In doms_to_xml(),

 * the calloc() has no check for successful return of memory leading to
possible NULL pointer deref of *dom_xml_list.


In async_ind_native():
 * The CU_DEBUG() message "...indication diliv..." should be
"...indication delivery..."
 * The CU_DEBUG("Unrecognized indication type"); should report what
'ind_type' is (btw: it's unrecognised)

In lifecycle_thread_native():
 * Seems inefficient to traverse lists so many times.  In the for loop
running through prev_xml, it would seem that if 'dom_in_list()' returns
1, then we could call 'dom_changed()' and possibly 'async_ind_native()'
if it has changed.
 * Since this routine keeps lifecycle_mutex for the duration - I believe
it makes it even more important to separate it from 'dom_list' in your
previous change...

NOTE: the get_domain_list() call could be made better by getting all
"Active and Defined" domains in one virConnectListAllDomains() call, but
that's mostly unrelated to this code.  Getting the list of All domains
at one time is "preferred" since you won't run into odd timing issues by
making two calls.

In raise_indication()
 * The returns for the strdup()'s are not checked.

John