[Libvirt-cim] [PATCH 2 of 3] SDC: returned instances are not of type specific subclass

[Kaitlin Rupert]

Dan Smith wrote:
> I didn't notice this before, but shouldn't we initialize prop_list to
> NULL, make sure free_rasd_prop_list() can handle NULL, and then put
> the free call below the out: target?  There is another place in the
> function that looks like it jumps to out after prop_list could have
> been initialized.  Further, it would be better to have it below out,
> so that additions like this one don't have to add or utilize an err:
> target.
>
> Heidi, I know this isn't really a part of your patch, but I think it's
> worth looking at while we're here.
>
> Jay, can you comment?
>
> Thanks!
>
>   
This seems like a reasonable request to me.

-- 
Kaitlin Rupert
IBM Linux Technology Center
karupert at us.ibm.com