]> granicus.if.org Git - graphviz/commitdiff
cgraph: abbreviate 'agxbdisown' string duplication to 'strdup'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 30 Apr 2022 23:57:37 +0000 (16:57 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 11 May 2022 05:24:16 +0000 (22:24 -0700)
lib/cgraph/agxbuf.c

index eef13d6321902a912757365a845ab2311f766adf..061cf05dbab37e532f95593ca0f127db3d055218 100644 (file)
@@ -150,26 +150,21 @@ int agxbpop(agxbuf * xb)
 
 char *agxbdisown(agxbuf * xb) {
 
-  size_t size;
   char *buf;
 
   /* terminate the existing string */
   agxbputc(xb, '\0');
 
-  size = (size_t)(xb->ptr - xb->buf);
-
   if (!xb->dyna) {
     /* the buffer is not dynamically allocated, so we need to copy its contents
      * to heap memory
      */
 
-    buf = malloc(size);
+    buf = strdup((char*)xb->buf);
     if (buf == NULL) {
       return NULL;
     }
 
-    memcpy(buf, xb->buf, size);
-
   } else {
     /* the buffer is already dynamically allocated, so take it as-is */
     buf = (char*)xb->buf;