Remove double mutex lock()

Fix XML generation for Xen USB devices

Arrange for "make syntax-check" to pass.

* .x-sc_avoid_if_before_free: Exempt ChangeLog.
* tests/statstest.c: Include <config.h>, not "config.h".

Fix device name -> number conversion for block stats

Also detect and remove unnecessary if-before-xmlXPathFreeContext.

* build-aux/find-unnecessary-if-before-free: Update regexp.
* src/openvz_conf.c: Remove unnecessary "if (P)"-before xmlXPathFreeContext.
* src/qemu_conf.c: Likewise.
* src/virsh.c: Likewise.
* src/xm_internal.c: Likewise.
* src/xml.c: Likewise.
* tests/xmlrpctest.c: Likewise.

Enable two more tests.

* Makefile.cfg (local-checks-to-skip)
[sc_cast_of_x_alloc_return_value, sc_cast_of_argument_to_free]: Enable.

Enable the <assert.h>-checking test; fix violations.

* Makefile.cfg (local-checks-to-skip)
[sc_prohibit_assert_without_use]: Enable.
* qemud/mdns.c: Don't include <assert.h>; no uses of assert here.
* qemud/qemud.c: Likewise.
* qemud/remote.c: Likewise.

Enable the po-check test; fix violations.

* Makefile.cfg (local-checks-to-skip) [po-check]: Enable.
* po/POTFILES.in: Add three file names.  Sort.

Update from gnulib
* lib/poll.c: Update.
* lib/string.in.h: Update.
* m4/string_h.m4: Update.
* tests/test-getaddrinfo.c: Update.
* lib/Makefile.am: Regenerate.

Enable the <config.h>-requiring test; fix violations

Use <config.h>, not "config.h", per autoconf documentation.
* Makefile.cfg (local-checks-to-skip) [sc_require_config_h]: Enable.
* .x-sc_require_config_h: New file, to list exempted files.
* Makefile.am (EXTRA_DIST): Add .x-sc_require_config_h.

Enable the duplicate-"the" test; fix violations

* Makefile.cfg (local-checks-to-skip) [sc_the_the]: Enable.
* docs/virsh.pod: Remove a duplicate "the".
* libvirt.spec.in: Likewise.
* virsh.1: Likewise.

Add framework for code style- and syntax-checking rules.

Almost all tests are initially disabled via the list in Makefile.cfg.
* Makefile.am (EXTRA_DIST): Add .x-sc_avoid_if_before_free.
Omit names of files that automake includes automatically.
* .x-sc_avoid_if_before_free: New file.
* build-aux/vc-list-files: Likewise.
* build-aux/find-unnecessary-if-before-free: Likewise.
* GNUmakefile, Makefile.cfg, Makefile.maint: New files.

Given code like if (foo) free (foo); remove the useless "if (foo) " part.

Likewise, given if (foo != NULL) free (foo); remove the useless "if" test.

* proxy/libvirt_proxy.c: Remove unnecessary "if" test before free.
* python/generator.py: Likewise.
* qemud/qemud.c: Likewise.
* src/buf.c: Likewise.
* src/conf.c: Likewise.
* src/hash.c: Likewise.
* src/iptables.c: Likewise.
* src/libvirt.c: Likewise.
* src/openvz_conf.c: Likewise.
* src/qemu_conf.c: Likewise.
* src/qemu_driver.c: Likewise.
* src/remote_internal.c: Likewise.
* src/test.c: Likewise.
* src/virsh.c: Likewise.
* src/virterror.c: Likewise.
* src/xen_internal.c: Likewise.
* src/xen_unified.c: Likewise.
* src/xend_internal.c: Likewise.
* src/xm_internal.c: Likewise.
* src/xml.c: Likewise.
* src/xmlrpc.c: Likewise.
* src/xs_internal.c: Likewise.
* tests/testutils.c: Likewise.
* tests/xencapstest.c: Likewise.
* tests/xmconfigtest.c: Likewise.

Clarify virsh documentation.
        * docs/virsh.pod: Clarify virsh documentation on when you can
          set memory and vCPUs.

Fix VIR_DOMAIN_NONE into VIR_DOMAIN_NOSTATE
* src/xen_internal.c: fix an erronous use of VIR_DOMAIN_NONE instead
  of VIR_DOMAIN_NOSTATE (both defined as 0, no regression)
Daniel

* src/libvirt.c: Jay Gagnon pointed out that virDomainGetXMLDesc
  was failing with a non-zero flag value
Daniel

Correct typos in the documentation (Atsushi SAKAI)
        * README, libvirt.spec.in, docs/devhelp/html.xsl,
          docs/devhelp/index.html: Correct typos in the documentation.

Fix enablement of SASL

Portability improvements for Mac OS X.
        * configure.in: Use PortableXDR if available.
        * qemud/internal.h: Use "socketcompat.h".
        * qemud/qemud.c: Use <signal.h>.
        * src/sexpr.c: Use <stdlib.h> instead of <malloc.h>.
        * src/socketcompat.h: Include <sys/un.h> for Mac OS X.

Remove DOS line-endings.
        * src/socketcompat.h: Remove DOS line-endings in this file.

Fix compat for old pkg-config and gnutls

Avoid "may be used uninitialized" warning.

* tests/xmconfigtest.c (testCompareParseXML): Initialize "old_priv".

Fix & refactor ref counting cleanup code

Make python generator fail build on any missing APIs

Add missing vcpu/schedular APIs to python binding

Use virFileReadAll in virsh.c

Remove no-op networking APIs from openvz driver

Adjust sexpr-related interfaces to be const-correct.

* src/sexpr.c (sexpr_cons, append, sexpr_append, sexpr2string)
(sexpr_lookup_key, sexpr_lookup, sexpr_node, sexpr_fmt_node):
Add "const" attribute where appropriate.
* src/xend_internal.c (sexpr_int, sexpr_float, sexpr_u64)
(sexpr_uuid, sexpr_to_xend_domain_info, sexpr_to_xend_node_info)
(sexpr_to_xend_topology_xml, sexpr_to_domain): Likewise.
* src/sexpr.h: Adjust prototypes.

Don't access line[-1] for a zero-length "line" from fgets.

A NUL byte at beginning of input, or just after a newline
would provoke an invalid buf[-1] access (possible segfault).
* src/libvirt.c (virConnectAuthCallbackDefault):

Preserve vif order
* src/xml-internal.c: apply patch from Hiroyuki Kaguchi to
  preserve the vif list order.
Daniel

Remove docs/API*.html
* docs/API* docs/api.xsl docs/site.xsl docs/Makefile.am: remove the
  generation of the API*.html files as it's not really useful here
Daniel

Added xstrtol_ll and xstrtol_ull convenience functions

Fix nodeinfo compat for Xen 3.2.0

Enable debug by default, but only if LIBVIRT_DEBUG=1. Use generic macro for debug output

Handle PyTuple_New's malloc failure.

* python/libvir.c (libvirt_virDomainBlockStats): Handle a NULL
return from PyTuple_New.
(libvirt_virDomainInterfaceStats, libvirt_virGetLastError): Likewise.
(libvirt_virConnGetLastError): Likewise.

Factor out some duplication.

* python/libvir.c (VIR_PY_NONE): New macro, to encapsulate
a common two-statement sequence.
Replace all such 2-stmt sequences.

Avoid format string warnings.

* src/virsh.c: Add "%s" where needed.
* src/proxy_internal.c: Likewise.

docs/examples/examples.xml: Regenerate, now that *.c file names are sorted.

Support VNC keymap option with QEMU (Daniel Hokka Zakrisson)

Fix off-by-one in SSH argv checking. Fix format string warning

Update from gnulib.

Clean up the global name space.

* src/qemu_conf.c: Add "static" and "const" attributes to some globals.
* src/qemu_conf.h: Update a declaration.
* src/qemu_driver.c (qemud_driver): Declare static.

Author: Jim Meyering <meyering@redhat.com>

Clean up global name space in examples and tests.

* docs/examples/suspend.c: Declare global "conn" to be static.
* tests/qemuxml2argvtest.c: Declare global "driver" to be static.
* tests/qemuxml2xmltest.c: Likewise.

Author: Jim Meyering <meyering@redhat.com>

Re-factor auth code to make clearer (Jim Meyering)

Fix crash when no auth callback

fix an unused variable warning

add iptablesSaveRules(), and don't save the rules to disk
and run lokkit each time a new rule is added.

now that we only use built-in iptables chains, we don't
need to add or delete them

Like --with-iptables-prefix, --with-iptables-dir is no
longer useful, so let's remove it.

--with-iptables-prefix was added to integrate with
a proposed system for letting iptables know how to
reload our rules. The proposed system wasn't accepted
so, although there might be some other theoretical
use for this, let's just remove it.

Add support for integrating our iptables support with Fedora's
iptables configuration using the lokkit --custom-rules command.

Basically, we write out our rules to /var/lib/libvirt/iptables
and run lokkit --custom-rules so that if e.g. iptables is
restarted or the user edits their firewall configuration, then
libvirt's rules get reloaded.

Include the iptables command and chain name in the saved rules files

Re-name the "flipflop" variable to "command_idx"

make use of virFileMakePath(), virFileBuildPath() and virRun().

Fix compile error in --with-iptables-dir code

Change virExec() such that with --enable-debug,
stdout and stderr no longer go to /dev/null.

Add virRun() helper function (Dan Berrange)

Fix vcpupin hypercall map data to be 8-bytes

Added graceful shutdown support to QEMU/KVM (patch from Guido Guenther)

Fix warning in debug output

Fix buffer passed to free()

Fix config file reading to not truncate large files

Miscellaneous fixes for building on Windows (MinGW).
* configure.in: xdr functions may require -lxdr.
* configure.in, src/Makefile.am: Extra linking flags for
  building libvirt DLL.
* src/socketcompat.h: Hide Windows Winsock incompatibilities
  in a separate header file.
* src/remote_internal.c: Switch to using socketcompat.h /
  socket_errno().
* src/test.c: Switch to using socketcompat.h.
* gnulib/lib/arpa/.cvsignore: Ignore another generated file.
* src/qparams.c: Include <config.h>.

Fix crash when no auth callback is provided

Ignore a few more files

* src/openvz.c: patch from Mikhail Pokidko to fix the OpenVZ
  compilation error #426385
Daniel

Fix docs, daniel

Added changelog for 0.4.0
Daniel

* configure.in docs/libvir.html docs/news.html po/*:
  prepare release of libvirt-0.4.0
Daniel

Added dep on 'nc' RPM

Fix nodeinfo compat with Xen 3.2.0

Fix remote_internal.c error handling of qparam_get_query
        * src/remote_internal.c: Fixed reversed check of error handling
          of qparam_get_query call (Dan Berrange).

Add separate qparams module for handling query parameters.
* src/qparams.c, src/qparams.h, src/Makefile.am: Added a
  separate 'qparams' module for handling query parameters.
* src/remote_internal.c: Factor out query parameter code so
  it uses the 'qparams' module.

Add extra utility functions to buf.c
* src/buf.c, src/buf.h: Added utility functions
  virBufferURIEncodeString and virBufferAddChar to the
  buffer module.

Don't invoke callback with unsupported credential types

fix setting CPU parameters in the config files
* src/xm_internal.c: patch from Saori Fukuta to fix setting
  CPU parameters in the config files, the value was overwritten
Daniel

fix the detach-disk command when using Xen tap devices
* src/xs_internal.[ch]: patch from Saori Fukuta to fix the
  detach-disk command when using Xen tap devices.
* docs/libvir.html: typo fix from Jim Meyering
Daniel

Fix virBufferEscapeString escaping of '<'
        * src/buf.c (virBufferEscapeString): '<' escaped as "&lt;"

Always build in python/ subdir.

* configure.in (PYTHON_SUBDIR): Don't set.  Remove all uses.
* Makefile.am (SUBDIRS): Hard-code "python" here.
(tests): Test equivalent "$(pythondir)", not @PYTHON_SUBDIR@.

Accommodate automake-1.9.

* tests/Makefile.am (TESTS_ENVIRONMENT): Adjust PATH setting
to work also with automake-1.9.  Can't use $(abs_top_builddir).

Author: Jim Meyering <meyering@redhat.com>

Use a variable name as sizeof argument, not a type name.

Given code like: T *var = calloc (n, sizeof (T));
Convert to this: T *var = calloc (n, sizeof (*var));
This first-cut change adjusts all malloc, calloc, and
realloc statements.

The only binary differences are in remote_internal.c
(due to the bug fix) and in xmlrpc.c (due to factorization).

* python/libvir.c: As above.
* qemud/event.c: Likewise.
* qemud/mdns.c: Likewise.
* qemud/qemud.c: Likewise.
* qemud/remote.c: Likewise.
* src/bridge.c: Likewise.
* src/buf.c: Likewise.
* src/conf.c: Likewise.
* src/hash.c: Likewise.
* src/iptables.c: Likewise.
* src/openvz_conf.c: Likewise.
* src/qemu_conf.c: Likewise.
* src/qemu_driver.c: Likewise.
* src/test.c: Likewise.
* src/xen_internal.c: Likewise.
* src/xen_unified.c: Likewise.
* src/xm_internal.c: Likewise.
* src/xml.c: Likewise.
* tests/qemuxml2argvtest.c: Likewise.
* src/xmlrpc.c (xmlRpcValuePtr): Likewise, and minor factorization.
* src/remote_internal.c (remoteAuthMakeCredentials): Use the right
type when allocating space for an array of cred _pointers_.

Test libvirtd's config-processing code.

And remove a minor diagnostic inconsistency.
* tests/daemon-conf: New test.
* tests/Makefile.am (TESTS_ENVIRONMENT): Prepend qemud/ to PATH,
so we can invoke libvirtd without an absolute name.
(test_scripts): Add daemon-conf.
* qemud/qemud.c (remoteConfigGetAuth): Use checkType,
rather than open-coding it with a different diagnostic.

Add gnulib-tool unit tests.
* bootstrap: Re-add --with-tests, now that gnulib-tool
arranges to use separate libraries for lib/ and tests/.
* configure.in (AC_OUTPUT): Add gnulib/tests/Makefile.in.
* Makefile.am (SUBDIRS): Add gnulib/tests.
* gnulib/tests/Makefile.am: New file.
* gnulib/lib/.cvsignore: Sort.
* gnulib/lib/Makefile.am, gnulib/m4/getdelim.m4, gnulib/m4/getline.m4:
* gnulib/m4/gnulib-cache.m4, gnulib/m4/gnulib-comp.m4: Regenerate.
* gnulib/tests/test-alloca-opt.c: New file, from gnulib.
* gnulib/tests/test-arpa_inet.c: Likewise.
* gnulib/tests/test-fseeko.c: Likewise.
* gnulib/tests/test-fseeko.sh: Likewise.
* gnulib/tests/test-getaddrinfo.c: Likewise.
* gnulib/tests/test-getdelim.c: Likewise.
* gnulib/tests/test-getline.c: Likewise.
* gnulib/tests/test-lseek.c: Likewise.
* gnulib/tests/test-lseek.sh: Likewise.
* gnulib/tests/test-netinet_in.c: Likewise.
* gnulib/tests/test-snprintf.c: Likewise.
* gnulib/tests/test-stdbool.c: Likewise.
* gnulib/tests/test-stdint.c: Likewise.
* gnulib/tests/test-stdio.c: Likewise.
* gnulib/tests/test-stdlib.c: Likewise.
* gnulib/tests/test-string.c: Likewise.
* gnulib/tests/test-sys_select.c: Likewise.
* gnulib/tests/test-sys_socket.c: Likewise.
* gnulib/tests/test-sys_stat.c: Likewise.
* gnulib/tests/test-sys_time.c: Likewise.
* gnulib/tests/test-unistd.c: Likewise.
* gnulib/tests/test-vasnprintf.c: Likewise.
* gnulib/tests/test-vasprintf.c: Likewise.
* gnulib/tests/test-wchar.c: Likewise.
* gnulib/tests/dummy.c: Likewise.
* gnulib/tests/intprops.h: Likewise.
* gnulib/tests/verify.h: Likewise.

Remove gnulib/lib/arpa/inet.h autogenerated file

Remove bogus autoconf 2.60 prereq

Fix minor quoting issue

Fri Dec  7 14:55:00 UTC 2007 Richard W.M. Jones <rjones@redhat.com>

        * proxy/Makefile.am: Include gnulib code when building the
          proxy.

Fri Dec  7 14:49:00 UTC 2007 Richard W.M. Jones <rjones@redhat.com>

* src/console.c, src/console.h, src/virsh.c: Disable
  text console on Windows.
* src/libvirt.c: Use replacement getpass from Gnulib.
* src/libvirt.c: Initialize Winsock before use.
* src/remote_internal.c: Header file fixes for Windows.
  Don't fail if AI_ADDRCONFIG isn't defined.  Disable
  unsupported stuff under Windows.
* src/uuid.c: ENODATA unavailable on Windows, use EIO instead.
* src/virsh.c: No uid_t / getuid on Windows.
* src/virsh.c: No O_SYNC on Windows.

Fri Dec  7 14:47:00 UTC 2007 Richard W.M. Jones <rjones@redhat.com>

* src/internal.h: Use gnulib gettext library.  Define
  INET_ADDRSTRLEN unless already defined.

Fri Dec  7 14:36:00 UTC 2007 Richard W.M. Jones <rjones@redhat.com>

* src/.cvsignore: Ignore *.loT files (generated under Windows).
* proxy/libvirt_proxy.c: Bail out earlier --without-xen.
* src/proxy_internal.c: Don't build proxy client side if
  configured --without-xen.
* src/iptables.c, src/iptables.h: Disable this code if
  configured --without-qemu.
* src/nodeinfo.c: If no 'uname' function, set model name to
  empty string (for Windows).
* src/xen_unified.h, src/util.c, src/test.c: Include <winsock2.h>
  on Windows.
* src/util.c: Disable virExec* and virFileLinkPointsTo on
  MinGW.

Fri Dec  7 14:34:00 UTC 2007 Richard W.M. Jones <rjones@redhat.com>

* src/sexpr.c: Cosmetic rearrangement of headers.

Insert cosmetic linebreak

Fri Dec  7 14:27:00 UTC 2007 Richard W.M. Jones <rjones@redhat.com>

* configure.in: Extra checks for uname and some header files
  (for MinGW Windows port).

Fri Dec  7 14:27:00 UTC 2007 Richard W.M. Jones <rjones@redhat.com>

* bootstrap, Makefile.am, gnulib/*: Import more gnulib
  modules for use by MinGW Windows port.

Include "config.h" in remaining non-generated files.

* proxy/libvirt_proxy.c: Likewise.
* python/libvir.c: Likewise.
* python/types.c: Likewise.
* src/event.c: Likewise.
* src/xm_internal.c: Likewise.
* tests/reconnect.c: Likewise.
* tests/testutils.c: Likewise.

* python/generator.py python/libvir.c python/libvirt-python-api.xml:
  add a python binding for virNodeGetCellsFreeMemory
Daniel

qemud/qemud.c (qemudInitPaths): Remove unused code.

Thu Dec  6 16:32:00 UTC 2007 Richard W.M. Jones <rjones@redhat.com>

* src/virsh.c: The prompt should depend on readonly status,
  not on the UID.

Thu Dec  6 16:30:00 UTC 2007 Richard W.M. Jones <rjones@redhat.com>

* configure.in, qemud/Makefile.am: Added a --without-libvirtd
  configure option.

Really fix macro for RHEL this time