X-Git-Url: http://git.annexia.org/?p=libguestfs.git;a=blobdiff_plain;f=ocaml%2Fguestfs_c.c;h=86fa29390d631426e4f233ef207af71c13dd14e3;hp=600440c92966ef54c2999eb2c4bf652644e3a226;hb=ab0397017cc26833c09946cca19d86b907822a94;hpb=13339826ea01f8dbd581b5d2544e7692171cf386

diff --git a/ocaml/guestfs_c.c b/ocaml/guestfs_c.c
index 600440c..86fa293 100644
--- a/ocaml/guestfs_c.c
+++ b/ocaml/guestfs_c.c
@@ -62,28 +62,19 @@ Val_guestfs (guestfs_h *g)
   CAMLreturn (rv);
 }
 
-/* Handle errors. */
-/* XXX Like the current Perl bindings, this is unsafe in a multi-
- * threaded environment.
- */
-static char *last_error = NULL;
-
-static void
-error_handler (guestfs_h *g,
-	       void *data,
-	       const char *msg)
-{
-  if (last_error != NULL) free (last_error);
-  last_error = strdup (msg);
-}
-
 void
 ocaml_guestfs_raise_error (guestfs_h *g, const char *func)
 {
   CAMLparam0 ();
   CAMLlocal1 (v);
+  const char *msg;
+
+  msg = guestfs_last_error (g);
 
-  v = caml_copy_string (last_error);
+  if (msg)
+    v = caml_copy_string (msg);
+  else
+    v = caml_copy_string (func);
   caml_raise_with_arg (*caml_named_value ("ocaml_guestfs_error"), v);
   CAMLnoreturn;
 }
@@ -100,7 +91,7 @@ ocaml_guestfs_create (void)
   if (g == NULL)
     caml_failwith ("failed to create guestfs handle");
 
-  guestfs_set_error_handler (g, error_handler, NULL);
+  guestfs_set_error_handler (g, NULL, NULL);
 
   gv = Val_guestfs (g);
   CAMLreturn (gv);
@@ -119,3 +110,33 @@ ocaml_guestfs_close (value gv)
 
   CAMLreturn (Val_unit);
 }
+
+/* Copy string array value.
+ * The return value is only 'safe' provided we don't allocate anything
+ * further on the OCaml heap (ie. cannot trigger the OCaml GC) because
+ * that could move the strings around.
+ */
+char **
+ocaml_guestfs_strings_val (guestfs_h *g, value sv)
+{
+  CAMLparam1 (sv);
+  char **r;
+  int i;
+
+  r = guestfs_safe_malloc (g, sizeof (char *) * (Wosize_val (sv) + 1));
+  for (i = 0; i < Wosize_val (sv); ++i)
+    r[i] = String_val (Field (sv, i));
+  r[i] = NULL;
+
+  CAMLreturnT (char **, r);
+}
+
+/* Free array of strings. */
+void
+ocaml_guestfs_free_strings (char **argv)
+{
+  /* Don't free the actual strings - they are String_vals on
+   * the OCaml heap.
+   */
+  free (argv);
+}