[libvirt] [PATCH 1/3] Support for vrdp/sdl/gui while defining a machine

Pritesh Kothari Pritesh.Kothari at Sun.COM
Tue May 12 15:15:18 UTC 2009


> This wasn't exactly what I anticipated. It intended that it should report
> the error *and* return as an error to the caller, rather than carrying on.

hmm.. I thought it more like handling NULL returns from strdup.

> That said, I notice that none of the existing code we have committed for
> VirtualBox driver deals with OOM on strdup() calls, so this patch is not
> making things any worse. Thus I think we can commit these patches and
> fix all the strdup calls in one go, later.


oops, Sorry this seems to be pretty big slip, yes will try to fix all of them 
but help is definitely appreciated.

Also if strdup() is going to be handle like alloc/VIR_ALLOC way, then i would 
wait for that and then do the changes.

Regards,
Pritesh.




More information about the libvir-list mailing list