aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel P. Berrange <berrange@redhat.com>2008-05-29 18:45:36 +0000
committerDaniel P. Berrange <berrange@redhat.com>2008-05-29 18:45:36 +0000
commit46f52f32b2d30222020b9ce3d25bc7ee3caf7f2f (patch)
treeb3c6c8e53ffa71a8ed9c35fd4110040e097bc8c2
parentFix missing check for memory allocation failure (diff)
downloadlibvirt-46f52f32b2d30222020b9ce3d25bc7ee3caf7f2f.tar.gz
libvirt-46f52f32b2d30222020b9ce3d25bc7ee3caf7f2f.tar.bz2
libvirt-46f52f32b2d30222020b9ce3d25bc7ee3caf7f2f.zip
Convert to use new memory allocation APIs
-rw-r--r--ChangeLog4
-rw-r--r--src/buf.c16
2 files changed, 10 insertions, 10 deletions
diff --git a/ChangeLog b/ChangeLog
index 14cd4bc93..21f0cb057 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+Thu May 29 14:45:00 EST 2008 Daniel P. Berrange <berrange@redhat.com>
+
+ * src/buf.c: Convert to use new memory allocation APIs
+
Thu May 29 14:44:00 EST 2008 Daniel P. Berrange <berrange@redhat.com>
* src/xend_internal.c: Fix missing check for failure
diff --git a/src/buf.c b/src/buf.c
index 687dce1af..c81de27a1 100644
--- a/src/buf.c
+++ b/src/buf.c
@@ -19,6 +19,7 @@
#define __VIR_BUFFER_C__
#include "buf.h"
+#include "memory.h"
/* If adding more fields, ensure to edit buf.h to match
@@ -40,8 +41,7 @@ struct _virBuffer {
static void
virBufferNoMemory(const virBufferPtr buf)
{
- free(buf->content);
- buf->content = NULL;
+ VIR_FREE(buf->content);
buf->size = 0;
buf->use = 0;
buf->error = 1;
@@ -60,7 +60,6 @@ static int
virBufferGrow(virBufferPtr buf, unsigned int len)
{
int size;
- char *newbuf;
if (buf->error)
return -1;
@@ -70,12 +69,10 @@ virBufferGrow(virBufferPtr buf, unsigned int len)
size = buf->use + len + 1000;
- newbuf = realloc(buf->content, size);
- if (newbuf == NULL) {
+ if (VIR_REALLOC_N(buf->content, size) < 0) {
virBufferNoMemory(buf);
return -1;
}
- buf->content = newbuf;
buf->size = size;
return 0;
}
@@ -269,8 +266,7 @@ virBufferEscapeString(const virBufferPtr buf, const char *format, const char *st
return;
len = strlen(str);
- escaped = malloc(5 * len + 1);
- if (escaped == NULL) {
+ if (VIR_ALLOC_N(escaped, 5 * len + 1) < 0) {
virBufferNoMemory(buf);
return;
}
@@ -314,14 +310,14 @@ virBufferEscapeString(const virBufferPtr buf, const char *format, const char *st
buf->content[buf->use] = 0;
grow_size = (count > 1000) ? count : 1000;
if (virBufferGrow(buf, grow_size) < 0) {
- free(escaped);
+ VIR_FREE(escaped);
return;
}
size = buf->size - buf->use - 1;
}
buf->use += count;
buf->content[buf->use] = '\0';
- free(escaped);
+ VIR_FREE(escaped);
}
/**