Fix memory leak in error return path.

diff --git a/src/generator.ml b/src/generator.ml
index 1e6c0bc..5b6e1eb 100755 (executable)
--- a/src/generator.ml
+++ b/src/generator.ml
@@ -2883,6 +2883,7 @@ check_state (guestfs_h *g, const char *caller)
 
       pr "  if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {\n";
       pr "    error (g, \"%%s\", ctx.err.error_message);\n";
+      pr "    free (ctx.err.error_message);\n";
       pr "    guestfs_end_busy (g);\n";
       pr "    return %s;\n" error_code;
       pr "  }\n";

diff --git a/src/guestfs-actions.c b/src/guestfs-actions.c
index 2dc84e5..1296051 100644 (file)
--- a/src/guestfs-actions.c
+++ b/src/guestfs-actions.c
@@ -163,6 +163,7 @@ int guestfs_mount (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -245,6 +246,7 @@ int guestfs_sync (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -331,6 +333,7 @@ int guestfs_touch (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -422,6 +425,7 @@ char *guestfs_cat (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -513,6 +517,7 @@ char *guestfs_ll (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -604,6 +609,7 @@ char **guestfs_ls (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -696,6 +702,7 @@ char **guestfs_list_devices (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -788,6 +795,7 @@ char **guestfs_list_partitions (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -880,6 +888,7 @@ char **guestfs_pvs (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -972,6 +981,7 @@ char **guestfs_vgs (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -1064,6 +1074,7 @@ char **guestfs_lvs (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -1156,6 +1167,7 @@ struct guestfs_lvm_pv_list *guestfs_pvs_full (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -1244,6 +1256,7 @@ struct guestfs_lvm_vg_list *guestfs_vgs_full (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -1332,6 +1345,7 @@ struct guestfs_lvm_lv_list *guestfs_lvs_full (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -1424,6 +1438,7 @@ char **guestfs_read_lines (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -1517,6 +1532,7 @@ int guestfs_aug_init (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -1599,6 +1615,7 @@ int guestfs_aug_close (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -1692,6 +1709,7 @@ int guestfs_aug_defvar (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -1787,6 +1805,7 @@ struct guestfs_int_bool *guestfs_aug_defnode (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -1879,6 +1898,7 @@ char *guestfs_aug_get (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -1967,6 +1987,7 @@ int guestfs_aug_set (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -2057,6 +2078,7 @@ int guestfs_aug_insert (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -2148,6 +2170,7 @@ int guestfs_aug_rm (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -2236,6 +2259,7 @@ int guestfs_aug_mv (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -2327,6 +2351,7 @@ char **guestfs_aug_match (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -2414,6 +2439,7 @@ int guestfs_aug_save (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -2496,6 +2522,7 @@ int guestfs_aug_load (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -2587,6 +2614,7 @@ char **guestfs_aug_ls (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -2678,6 +2706,7 @@ int guestfs_rm (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -2764,6 +2793,7 @@ int guestfs_rmdir (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -2850,6 +2880,7 @@ int guestfs_rm_rf (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -2936,6 +2967,7 @@ int guestfs_mkdir (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -3022,6 +3054,7 @@ int guestfs_mkdir_p (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -3110,6 +3143,7 @@ int guestfs_chmod (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -3200,6 +3234,7 @@ int guestfs_chown (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -3291,6 +3326,7 @@ int guestfs_exists (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -3382,6 +3418,7 @@ int guestfs_is_file (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -3473,6 +3510,7 @@ int guestfs_is_dir (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -3559,6 +3597,7 @@ int guestfs_pvcreate (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -3648,6 +3687,7 @@ int guestfs_vgcreate (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -3738,6 +3778,7 @@ int guestfs_lvcreate (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -3826,6 +3867,7 @@ int guestfs_mkfs (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -3921,6 +3963,7 @@ int guestfs_sfdisk (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -4011,6 +4054,7 @@ int guestfs_write_file (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -4097,6 +4141,7 @@ int guestfs_umount (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -4184,6 +4229,7 @@ char **guestfs_mounts (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -4271,6 +4317,7 @@ int guestfs_umount_all (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -4353,6 +4400,7 @@ int guestfs_lvm_remove_all (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -4444,6 +4492,7 @@ char *guestfs_file (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -4536,6 +4585,7 @@ char *guestfs_command (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -4628,6 +4678,7 @@ char **guestfs_command_lines (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -4724,6 +4775,7 @@ struct guestfs_stat *guestfs_stat (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -4816,6 +4868,7 @@ struct guestfs_stat *guestfs_lstat (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -4908,6 +4961,7 @@ struct guestfs_statvfs *guestfs_statvfs (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -5000,6 +5054,7 @@ char **guestfs_tune2fs_l (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -5091,6 +5146,7 @@ int guestfs_blockdev_setro (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -5177,6 +5233,7 @@ int guestfs_blockdev_setrw (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -5268,6 +5325,7 @@ int guestfs_blockdev_getro (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -5359,6 +5417,7 @@ int guestfs_blockdev_getss (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -5450,6 +5509,7 @@ int guestfs_blockdev_getbsz (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -5538,6 +5598,7 @@ int guestfs_blockdev_setbsz (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -5629,6 +5690,7 @@ int64_t guestfs_blockdev_getsz (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -5720,6 +5782,7 @@ int64_t guestfs_blockdev_getsize64 (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -5806,6 +5869,7 @@ int guestfs_blockdev_flushbufs (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -5892,6 +5956,7 @@ int guestfs_blockdev_rereadpt (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -5992,6 +6057,7 @@ int guestfs_upload (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -6079,6 +6145,7 @@ int guestfs_download (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -6177,6 +6244,7 @@ char *guestfs_checksum (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -6277,6 +6345,7 @@ int guestfs_tar_in (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -6364,6 +6433,7 @@ int guestfs_tar_out (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -6469,6 +6539,7 @@ int guestfs_tgz_in (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -6556,6 +6627,7 @@ int guestfs_tgz_out (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -6649,6 +6721,7 @@ int guestfs_mount_ro (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -6739,6 +6812,7 @@ int guestfs_mount_options (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -6831,6 +6905,7 @@ int guestfs_mount_vfs (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -6925,6 +7000,7 @@ char *guestfs_debug (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -7011,6 +7087,7 @@ int guestfs_lvremove (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -7097,6 +7174,7 @@ int guestfs_vgremove (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -7183,6 +7261,7 @@ int guestfs_pvremove (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -7271,6 +7350,7 @@ int guestfs_set_e2label (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -7362,6 +7442,7 @@ char *guestfs_get_e2label (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -7450,6 +7531,7 @@ int guestfs_set_e2uuid (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -7541,6 +7623,7 @@ char *guestfs_get_e2uuid (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -7634,6 +7717,7 @@ int guestfs_fsck (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -7720,6 +7804,7 @@ int guestfs_zero (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -7808,6 +7893,7 @@ int guestfs_grub_install (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -7896,6 +7982,7 @@ int guestfs_cp (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -7984,6 +8071,7 @@ int guestfs_cp_a (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -8072,6 +8160,7 @@ int guestfs_mv (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -8158,6 +8247,7 @@ int guestfs_drop_caches (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -8245,6 +8335,7 @@ char *guestfs_dmesg (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -8327,6 +8418,7 @@ int guestfs_ping_daemon (guestfs_h *g)
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -8420,6 +8512,7 @@ int guestfs_equal (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return -1;
   }
@@ -8511,6 +8604,7 @@ char **guestfs_strings (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -8609,6 +8703,7 @@ char **guestfs_strings_e (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }
@@ -8705,6 +8800,7 @@ char *guestfs_hexdump (guestfs_h *g,
 
   if (ctx.hdr.status == GUESTFS_STATUS_ERROR) {
     error (g, "%s", ctx.err.error_message);
+    free (ctx.err.error_message);
     guestfs_end_busy (g);
     return NULL;
   }