Avoid VLAs with size depending on user input

Do not use var-length arrays with sizes from user input (like
parameters), since they may grow the stack too much.

Instead, allocate the memory on the heap.

Workaround lack of constness in custom_operations::identifier

Sadly the custom_operations struct of the OCaml C API uses char* for the
'identifier' member, even though it does not change it.

To avoid getting warnings when building with stricter CFLAGS, apply ugly
casts.

Add an helper CHECK_ERROR_CLEANUP macro

Extend the existing CHECK_ERROR macro to allow some code to be executed
right before raising Virterror.  Rework CHECK_ERROR to use
CHECK_ERROR_CLEANUP.

Add Domain.get_xml_desc_flags

Much like Domain.get_xml_desc with the possibility to specify flags for
the XML description.  The function is marked with a [`W] phantom because
some of the flags (e.g. XmlSecure) require a read/write connection.

Synchronize Virterror with libvirt 4.6.0

In particular, add all the various error codes, and error domains.
Also, add an older domain that was previously skipped.

Connect: add auth/credential handling for connect

Add OCaml versions of virConnectCredential & virConnectAuth structs,
using them to implement proper connect_auth & connect_auth_readonly.

This makes it possible to properly supply credentials when opening a
connection.

Add Val_opt_const & Val_const_ptr_t

Counterparts of Val_opt & Val_ptr_t, but with constness for the pointer.

Use -safe-string and fix the library.

Note this changes the type of the cpumap from string to bytes,
since (by the design of the API) it must be mutated.

Change binding of virConnectGetAllDomainStats to return dom UUID.

The virDomainPtr object returned by this binding isn't a reliable
virDomainPtr object.  The only thing we can safely do with it is to
get its UUID.  Modify the API correspondingly.

Updates commit 380f1e05b244ae4750ca5101b5b5a182dcd0d1fd.

examples: Print more stats in the get_all_domain_stats.ml example.

Updates commit 380f1e05b244ae4750ca5101b5b5a182dcd0d1fd.

Add a binding for virConnectGetAllDomainStats (RHBZ#1390171).

Update dependencies.

Use -g -warn-error.

Use -g for ocamlopt.  ocamlopt has supported generating DWARF
information for quite a long time.

Also use -warn-error with the same set of warnings as is used
by libguestfs.

Fix a warning in examples/get_cpu_stats.ml found by enabling
-warn-error.

Remove unused 'not_supported' function.

Found by compiling under GCC 6.  The warning was:

In file included from libvirt_c.c:2058:0:
libvirt_c_epilogue.c:89:1: error: 'not_supported' defined but not used [-Werror=unused-function]
 not_supported (const char *fn)
 ^~~~~~~~~~~~~

Don't bother checking return from virInitialize.

The Perl bindings don't do this, and it seems that the call can never
fail, or if it does we don't care.

Suppress errors to stderr and use thread-local virErrorPtr

This allows us to capture errors that happen during virConnectOpen,
without spewing error messages on stderr.

The patch is rather large, only because we don't need to pass the
virConnectPtr argument to CHECK_ERROR/_raise_virterror any longer.

Add a binding for virDomainCreateXML.

This is more modern than the ancient virDomainCreateLinux API,
and crucially allows you to pass flags such as AUTODESTROY.

Use C99 standard int64_t instead of OCaml defined (and soon to go) int64.

Kill broken NSIS support.

Version 0.6.1.4.

Update MANIFEST.

Version 0.6.1.3.

Update dependencies.

Add a simple example to show how to receive event callbacks

Signed-off-by: David Scott <dave.scott@eu.citrix.com>

Add event callback implementation based on virConnectDomainEventRegisterAny

A client may register a callback as follows:

    E.register_default_impl ();

    let conn = C.connect_readonly ?name () in

    let id = E.register_any conn (E.Lifecycle (fun dom e ->
        printd dom "Lifecycle %s" (E.Lifecycle.to_string e)
    )) in

Internally this will:
  1. generate a unique int64 used to identify the specific callback
  2. add the callback to an OCaml hashtable based on the signature
     (there is a distinct hashtable per callback signature)
  3. call virConnectDomainEventRegisterAny which registers a
     generic C callback in the stubs (one distinct callback per
     signature) and supply the int64 as the "opaque" data

The client must enter the event loop with:

    while true do
        E.run_default_impl ()
    done

When an event is triggered, the C callback will upcall into an OCaml
function (having re-acquired the heap lock) supplying the int64 value.
The OCaml function can then find the right callback in the Hashtbl
and call it.

The client can deregister the callback with:

  E.deregister_any conn id;

Signed-off-by: David Scott <dave.scott@eu.citrix.com>

Add binding for virConnectSetKeepAlive

This one is a 'one-off' but it ought to be possible to use the generator
to create the function (it has signature 'conn, int, int : int')

This function first appeared in libvirt version 0.9.8.

Signed-off-by: David Scott <dave.scott@eu.citrix.com>

Remove backwards compatability logic to simplify the bindings

Rather than compile against old versions of libvirt and resort
to throwing 'not supported' exceptions at runtime, we instead
require all symbols and definitions to be available at compile-time
i.e. we require a much more recent libvirt.

Signed-off-by: David Scott <dave.scott@eu.citrix.com>

Fix typo in ocaml_libvirt_storage_vol_get_info

The info.capacity was being overwritten with the info.allocation.

Signed-off-by: David Scott <dave.scott@eu.citrix.com>

Update dependencies.

Functions returning "unit" correspond to C fns which use -1 for failure

This affects the following functions:
  virStoragePoolBuild
  virStoragePoolDelete
  virStorageVolDelete

Previously a call to virStorageVolDelete would succeed returning 0, which
was interpreted as false, causing us to raise an exception with
VIR_ERR_NONE.

Signed-off-by: David Scott <dave.scott@eu.citrix.com>

Volume.delete takes a flags parameter

This corrects a mismatch between the generated C stubs and the .ml
interface.

Signed-off-by: David Scott <dave.scott@eu.citrix.com>

Correct typos in the storage interface's phantom types

A connection is either read/write (type rw = [`R|`W]) or read/only
(type ro = [`R]). A function which requires the ability to write
needs to take a parameter [> `W] rather than [`W] (which would
correspond to a write-only connection).

We can now use a read/write connection to call:
  Pool.set_autostart
  Volume.create_xml
  Volume.delete

Signed-off-by: David Scott <dave.scott@eu.citrix.com>

Add contrib directory, and non-upstream patch to add Domain.get_cpu_stats_total.

This patch needs more thorough review.

remove parameter nr_pcpus of Libvirt.Domain.get_cpu_stats

remove the value because we can get it from the libvirt API

Version 0.6.1.2.

Domain.get_cpu_stats: get number of params correctly (thanks Eric Blake).

See:
https://www.redhat.com/archives/libvir-list/2012-March/msg00306.html

This fixes commit c96c3a119b44d3321dddc5e189dcba991aaff677.

Version 0.6.1.1.

ocaml-libvirt: add D.get_cpu_stats() API to ocaml-libvirt

Changed from V3:
use new virDomainGetCPUStats() libvirt-API.
use C code to construct the typed_param list array

Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>

Remove virConnectListAllDomains.

This was never in a released libvirt.

Add header to Make.rules.in.

Remove obsolete: mlvirsh, gettext support, Windows installer.

Remove .hgignore.

Remove jobs API.

This was never in a released libvirt.

Add .gitignore file.

Correct to-do list.

Version 0.6.1.0 for release.

Correct mlvirsh for new read-write connection.

virDomainBlockPeek and virDomainMemoryPeek need a read/write connection
(CVE-2008-5086) (found by David Lutterkort).

D.get_id returns -1 for inactive domains instead of throwing an error.

Don't duplicate unnecessary call to D.get_info.

Implement version 3 of virConnectListAllDomains.

Add support for virConnectListAllDomains call.

Revert previous change.

Use CAMLreturn0 to avoid a warning.

Allow destdir options to be passed to 'make install' (Guillaume Rousse).

Split peek requests into maximum-sized chunks.

Add Domain.max_peek function to determine maximum size of
peek requests.

Don't pass -destdir parameter to ocamlfind.  Version 0.4.4.2.

Version 0.4.4.1 for release.

Changed INRIA to Red Hat (for Debian)

Files missed by 'make distclean' (for Debian)

Removed old ChangeLog file.

Clarify the license again for Debian.

Revert previous commit.

Reference the license from LGPL files (for Debian).

Clarify the license for Debian.

Split 'make install' into separate 'make install-opt' and 'make install-byte' (Sylvain Le Gall, via Debian)

Remove some old virt-top stuff.

Auto-generate ChangeLog.

Deprecate ChangeLog file.

Version 0.4.2.4 for release.

Missing dependencies.

List* functions throw exceptions if maxids = 0, so bypass this case.

Version 0.4.2.3.

Fix some bugs in the implementation of virDomainMemoryPeek

Updated MANIFEST

Add domblkpeek, dommempeek commands and
add per-argument help.

Add support for virDomainBlockPeek, virDomainMemoryPeek and
remove the deprecated conn/dom/net fields in virterror.

Remove Windows cruft from when it was lacking ocamlfind.

Brazilian Portuguese translation by Herli Joaquim de Menezes and revised by Igor Pires Soares.

Just the OCaml bindings (C library).

Initial import from old virt-top repository.