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

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!