
12 Nov
2012
12 Nov
'12
8:48 a.m.
On Mon, Nov 12, 2012 at 03:02:27PM +0800, Gao feng wrote:
+ len = strlen(new_line); + + if (copied + len > size) + len = size - copied; + + memcpy(buf + copied, new_line, len); + copied += len; + memset(line, 0, sizeof(line)); + if (new_line != line) + VIR_FREE(new_line);
How about using the virBuffer API here? Also you forgot to free 'line'. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones libguestfs lets you edit virtual machines. Supports shell scripting, bindings from many languages. http://libguestfs.org