On 01/15/2013 07:12 PM, John Ferlan wrote:
--- src/util/virbuffer.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/util/virbuffer.c b/src/util/virbuffer.c index 969dcbf..693e4b2 100644 --- a/src/util/virbuffer.c +++ b/src/util/virbuffer.c @@ -153,10 +153,9 @@ virBufferAdd(virBufferPtr buf, const char *str, int len) if (!str || !buf || (len == 0 && buf->indent == 0)) return;
- if (buf->error) - return; - indent = virBufferGetIndent(buf, true); + if (indent < 0) + return;
if (len < 0) len = strlen(str);
ACK && Pushed with a little explanation taken from 0/4: Since virBufferGetIndent() will check and fail on buf->error, I removed that check from virBufferAdd() and used the -1 return as the way to exit. Hope that's OK, Martin