Use safe_malloc and/or check returns from malloc (Jim Meyering).

[libguestfs.git] / perl / Guestfs.xs

diff --git a/perl/Guestfs.xs b/perl/Guestfs.xs
index 63dce74..9d8a49c 100644 (file)
--- a/perl/Guestfs.xs
+++ b/perl/Guestfs.xs
@@ -64,12 +64,13 @@ XS_unpack_charPtrPtr (SV *arg) {
   AV *av;
   I32 i;
 
-  if (!arg || !SvOK (arg) || !SvROK (arg) || SvTYPE (SvRV (arg)) != SVt_PVAV) {
+  if (!arg || !SvOK (arg) || !SvROK (arg) || SvTYPE (SvRV (arg)) != SVt_PVAV)
     croak ("array reference expected");
-  }
 
   av = (AV *)SvRV (arg);
-  ret = (char **)malloc (av_len (av) + 1 + 1);
+  ret = malloc (av_len (av) + 1 + 1);
+  if (!ret)
+    croak ("malloc failed");
 
   for (i = 0; i <= av_len (av); i++) {
     SV **elem = av_fetch (av, i, 0);
@@ -168,6 +169,30 @@ PREINIT:
         croak ("config: %s", guestfs_last_error (g));
 
 void
+set_qemu (g, qemu)
+      guestfs_h *g;
+      char *qemu;
+PREINIT:
+      int r;
+ PPCODE:
+      r = guestfs_set_qemu (g, qemu);
+      if (r == -1)
+        croak ("set_qemu: %s", guestfs_last_error (g));
+
+SV *
+get_qemu (g)
+      guestfs_h *g;
+PREINIT:
+      const char *qemu;
+   CODE:
+      qemu = guestfs_get_qemu (g);
+      if (qemu == NULL)
+        croak ("get_qemu: %s", guestfs_last_error (g));
+      RETVAL = newSVpv (qemu, 0);
+ OUTPUT:
+      RETVAL
+
+void
 set_path (g, path)
       guestfs_h *g;
       char *path;
@@ -1405,3 +1430,146 @@ PREINIT:
       if (r == -1)
         croak ("tgz_out: %s", guestfs_last_error (g));
 
+void
+mount_ro (g, device, mountpoint)
+      guestfs_h *g;
+      char *device;
+      char *mountpoint;
+PREINIT:
+      int r;
+ PPCODE:
+      r = guestfs_mount_ro (g, device, mountpoint);
+      if (r == -1)
+        croak ("mount_ro: %s", guestfs_last_error (g));
+
+void
+mount_options (g, options, device, mountpoint)
+      guestfs_h *g;
+      char *options;
+      char *device;
+      char *mountpoint;
+PREINIT:
+      int r;
+ PPCODE:
+      r = guestfs_mount_options (g, options, device, mountpoint);
+      if (r == -1)
+        croak ("mount_options: %s", guestfs_last_error (g));
+
+void
+mount_vfs (g, options, vfstype, device, mountpoint)
+      guestfs_h *g;
+      char *options;
+      char *vfstype;
+      char *device;
+      char *mountpoint;
+PREINIT:
+      int r;
+ PPCODE:
+      r = guestfs_mount_vfs (g, options, vfstype, device, mountpoint);
+      if (r == -1)
+        croak ("mount_vfs: %s", guestfs_last_error (g));
+
+SV *
+debug (g, subcmd, extraargs)
+      guestfs_h *g;
+      char *subcmd;
+      char **extraargs;
+PREINIT:
+      char *result;
+   CODE:
+      result = guestfs_debug (g, subcmd, extraargs);
+      free (extraargs);
+      if (result == NULL)
+        croak ("debug: %s", guestfs_last_error (g));
+      RETVAL = newSVpv (result, 0);
+      free (result);
+ OUTPUT:
+      RETVAL
+
+void
+lvremove (g, device)
+      guestfs_h *g;
+      char *device;
+PREINIT:
+      int r;
+ PPCODE:
+      r = guestfs_lvremove (g, device);
+      if (r == -1)
+        croak ("lvremove: %s", guestfs_last_error (g));
+
+void
+vgremove (g, vgname)
+      guestfs_h *g;
+      char *vgname;
+PREINIT:
+      int r;
+ PPCODE:
+      r = guestfs_vgremove (g, vgname);
+      if (r == -1)
+        croak ("vgremove: %s", guestfs_last_error (g));
+
+void
+pvremove (g, device)
+      guestfs_h *g;
+      char *device;
+PREINIT:
+      int r;
+ PPCODE:
+      r = guestfs_pvremove (g, device);
+      if (r == -1)
+        croak ("pvremove: %s", guestfs_last_error (g));
+
+void
+set_e2label (g, device, label)
+      guestfs_h *g;
+      char *device;
+      char *label;
+PREINIT:
+      int r;
+ PPCODE:
+      r = guestfs_set_e2label (g, device, label);
+      if (r == -1)
+        croak ("set_e2label: %s", guestfs_last_error (g));
+
+SV *
+get_e2label (g, device)
+      guestfs_h *g;
+      char *device;
+PREINIT:
+      char *label;
+   CODE:
+      label = guestfs_get_e2label (g, device);
+      if (label == NULL)
+        croak ("get_e2label: %s", guestfs_last_error (g));
+      RETVAL = newSVpv (label, 0);
+      free (label);
+ OUTPUT:
+      RETVAL
+
+void
+set_e2uuid (g, device, uuid)
+      guestfs_h *g;
+      char *device;
+      char *uuid;
+PREINIT:
+      int r;
+ PPCODE:
+      r = guestfs_set_e2uuid (g, device, uuid);
+      if (r == -1)
+        croak ("set_e2uuid: %s", guestfs_last_error (g));
+
+SV *
+get_e2uuid (g, device)
+      guestfs_h *g;
+      char *device;
+PREINIT:
+      char *uuid;
+   CODE:
+      uuid = guestfs_get_e2uuid (g, device);
+      if (uuid == NULL)
+        croak ("get_e2uuid: %s", guestfs_last_error (g));
+      RETVAL = newSVpv (uuid, 0);
+      free (uuid);
+ OUTPUT:
+      RETVAL
+