[libvirt] [PATCH 1/2] daemon: convert virRun to virCommand

Michal Privoznik mprivozn at redhat.com
Thu Jan 26 08:56:06 UTC 2012


On 26.01.2012 01:51, Eric Blake wrote:
> Using snprintf to build up argv seems archaic.
> 
> * daemon/remote.c (remoteDispatchAuthPolkit): Modernize command call.
> ---
>  daemon/remote.c |   42 +++++++++++++++++-------------------------
>  1 files changed, 17 insertions(+), 25 deletions(-)
> 

ACK. I wonder if we should force developers to use virAsnprintf()
instead of snprintf() like we already do for malloc(), realloc(), etc.

Michal




More information about the libvir-list mailing list