summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* ocaml: Don't install libguestfsocaml.a.Richard W.M. Jones2012-01-291-0/+1
| | | | (cherry picked from commit b1ace6738177481276dce05bf1d2597bb67f2a80)
* gobject: Add a TODO listMatthew Booth2012-01-291-0/+9
| | | | (cherry picked from commit dd25638fc14ea63cf8e8eb0940173f1c6952447f)
* gobject: Add an explicit close callMatthew Booth2012-01-294-13/+90
| | | | | | This change binds guestfs_close(). It consequently results in RConstOptString being able to throw an error. (cherry picked from commit 8bad4d1c4765fd997511a505cdb2a860148ae16d)
* gobject: Allow RConstOptString to return an errorMatthew Booth2012-01-291-18/+11
| | | | | | | | | | | | RConstOptString cannot return an error in the C api. This makes it a special case for the GObject api, as all other return types have a corresponding GError **err argument to return an error. This change removes this special case, and includes the possibility of an error return in the API. An error is indicated by setting *err to a non-NULL value. This change is in preparation for adding a close api. An attempt to call any api, even RConstOptString, on a closed handle must return an error. (cherry picked from commit 6568dd85f4aef705e7cf5c4301bc3b8f7119efd2)
* Version 1.16.1.1.16.1Richard W.M. Jones2012-01-2420-439/+439
|
* daemon: Fix use-after-free in case-insensitive-path (found by valgrind).Richard W.M. Jones2012-01-241-38/+81
| | | | | | | | | | This commit tidies up the code by splitting out the path element-searching code into a separate function. Valgrind found that 'closedir' frees the 'struct dirent *', which wasn't immediately obvious. So now we do the 'closedir' after all operations which touch 'd->d_name'. (cherry picked from commit a05ddcd2a719f97ba036e9d6ca4e6491ed8b1fd0)
* daemon: Fix leaking error message (found by valgrind).Richard W.M. Jones2012-01-241-0/+1
| | | | (cherry picked from commit 9700708a19a46a61ffe53c6e648206336781477e)
* daemon: Close inotify handle on exit.Richard W.M. Jones2012-01-241-0/+11
| | | | (cherry picked from commit 86f64c37b914645afb345f96af13ad843ec641b1)
* daemon: Close augeas handle on exit.Richard W.M. Jones2012-01-241-0/+11
| | | | (cherry picked from commit 94a7fb9fc5d04615f23f94c34eb90c35d20ba117)
* daemon: Fix leak of strings in md-detail (found by valgrind).Richard W.M. Jones2012-01-241-2/+2
| | | | (cherry picked from commit ff2e685828ce6160de34756696069b80e35472e8)
* daemon: md: Whitespace changes, and use size_t for i instead of pointer.Richard W.M. Jones2012-01-241-20/+23
| | | | | This is just a code clean-up with no functional change. (cherry picked from commit 4c2101362edf5f2df5d018c3674b423de2a3538a)
* daemon: Fix leak of strings in blkid (found by valgrind).Richard W.M. Jones2012-01-241-2/+2
| | | | (cherry picked from commit aa978b75ae1df0d3111953114fafe0854c3f222e)
* daemon: blkid: Whitespace changes, and use size_t for i instead of pointer.Richard W.M. Jones2012-01-241-18/+21
| | | | | This is just a code clean-up with no functional change. (cherry picked from commit 43a6974cb35759d4772f610f3c92b0d31f0503ac)
* daemon: Return value from write syscall is ssize_t, not int.Richard W.M. Jones2012-01-241-1/+1
| | | | (cherry picked from commit f86c3a45ae19f5bb558ab9ead255ecaf004b24b0)
* daemon: Don't leak 'cmdline' (found by valgrind).Richard W.M. Jones2012-01-241-0/+2
| | | | (cherry picked from commit 9f8f1fd8560f2c3e7b625cf6d65a052cd061fd2e)
* daemon: Length of message is a 32 bit unsigned quantity.Richard W.M. Jones2012-01-241-2/+2
| | | | (cherry picked from commit f65f653ab1541400afe2f796ab1b20cd721e769d)
* daemon: Fix use of uninitialized stack data (found by valgrind).Richard W.M. Jones2012-01-241-0/+2
| | | | | | This uninitialized data was also sent over the protocol, potentially being a serious information leak. (cherry picked from commit 9cf7ecbe4fea5162b86e2c78c1a40cca737bc1da)
* daemon: Fix memory leak in 'initrd-cat' (found by valgrind).Richard W.M. Jones2012-01-241-0/+2
| | | | (cherry picked from commit 84242a8d0bf9c37e700cd8a3436607dbc7908f1e)
* Enable running the daemon under valgrind.Richard W.M. Jones2012-01-247-3/+54
| | | | | | | | | | | | | | | | | | This commit allows you to run the daemon under valgrind. You have to enable it at configure time: ./configure --enable-valgrind-daemon This should *not* be done for production builds. When this feature is enabled, valgrind is added to the appliance and the daemon is run under valgrind. Log messages from valgrind are passed back over a virtio-serial channel into a file called 'valgrind.log.$PID' in the top build directory. Running 'make check', 'make extra-tests' etc causes many valgrind.log.* files to be created which must be examined by hand. (cherry picked from commit d08806adfc5d111b4b6c761540ff508d3f25b66e)
* resize, sparsify: find progress mini-library in out-of-tree builds (2)Hilko Bengen2012-01-242-3/+3
| | | | (cherry picked from commit b7122eaa58ccd153a8c39bc3b6f890755c5df089)
* Fix --{en,dis}able-install-daemonHilko Bengen2012-01-241-1/+1
| | | | | | | | | This fixes... configure: error: conditional "INSTALL_DAEMON" was never defined. ... when specifying --disable-daemon (cherry picked from commit 22224254fa4676735985818d2ff3c8feafe4457c)
* Version 1.16.0.1.16.0Richard W.M. Jones2012-01-2320-6755/+6986
|
* API support: Fix src/api-support/added file.Richard W.M. Jones2012-01-231-470/+470
| | | | | This fixes commit 37e07db1598b433c12ee643302712d81b2c5415d. (cherry picked from commit 734b1f662f4e563cc8baad3aca8cf751314e0e49)
* Update ROADMAP.Richard W.M. Jones2012-01-231-4/+8
|
* Finalize RELEASE NOTES.Richard W.M. Jones2012-01-231-6/+10
|
* Pull translations from Transifex.Richard W.M. Jones2012-01-2316-1998/+651
|
* debian: Debian 6 blkid has -p but not -i.Richard W.M. Jones2012-01-231-30/+62
| | | | | | Fix the existing test to work correctly in this case. Other cleanups.
* gobject: Require minimum gobject library version of 2.26.0Matthew Booth2012-01-231-1/+1
| | | | | The GObject bindings require G_DEFINE_BOXED_TYPE, which was not added until 2.25.15. There are probably other dependencies.
* maint: use $var notation rather than ${var} when possibleJim Meyering2012-01-2310-17/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I noticed some uses of ${srcdir} in shell scripts. That is almost always better written as $srcdir. The patch below converts most such variable references. Here are the few remaining candidates: $ git grep -i -E '\$\{[a-zA-Z_0-9]+\}'|grep -v Makefile.in.in configure.ac: JAR_INSTALL_DIR=\${prefix}/share/java configure.ac: JNI_INSTALL_DIR=\${libdir} debian/rules: for TEST in ${DEBIAN_SKIP_TEST}; do \ debian/rules:# mv $${mod} $$(dirname $${mod})/libguestfsmod.so; \ java/Makefile.am:libguestfs_jar_DATA = libguestfs-${VERSION}.jar java/Makefile.am:libguestfs-${VERSION}.jar: $(libguestfs_jar_class_files) perl/lib/Sys/Guestfs/Lib.pm: "-f", '${Package} ${Version} ${Architecture} ${Status}\n', perl/typemap: croak (\"${Package}::$func_name(): called on a closed handle\"); perl/typemap: croak (\"${Package}::$func_name(): $var is not a blessed HV reference\"); tests/data/Makefile.am: echo "$${i}abcdefghijklmnopqrstuvwxyz"; \ We could change all of those, too, except the ones in configure.ac and Makefile.am, since they refer to Make variables. Even those should be changed, but to use the preferred Makefile notation: $(prefix), $(libdir), $(VERSION). >From a86770ecd45666232a94d76c8725c8f9b1c76e3a Mon Sep 17 00:00:00 2001 From: Jim Meyering <meyering@redhat.com> Date: Mon, 23 Jan 2012 11:15:12 +0100 Subject: [PATCH libguestfs] maint: use $var notation rather than ${var} when possible The only case to avoid in a shell script is when the byte after the "}" is word-constituent, and concatenating it would thus change the name of the variable. These changes were induced by running this command: git grep -l -i -E '\$\{(srcdir|md)' \ |xargs perl -pi -e 's/\$\{(srcdir|md)\}($|\w)/\$$1$2/gi' The "g" was needed because there was one line with two instances. The "i" is to handle ${SRCDIR}. The ($|\w) ensures that concatenating whatever follows the "}" won't change semantics. * gobject/run-bindtests: Use "$srcdir", not "${srcdir}". * haskell/run-bindtests: Likewise. * java/run-bindtests: Likewise. * ocaml/run-bindtests: Likewise. * perl/run-bindtests: Likewise. * python/run-bindtests: Likewise. * ruby/run-bindtests: Likewise. * tests/guests/guest-aux/make-debian-img.sh: Likewise, but $SRCDIR. * tests/guests/guest-aux/make-ubuntu-img.sh: Likewise. * tests/guests/guest-aux/make-windows-img.sh: Likewise. * tests/md/test-mdadm.sh: Likewise, but $md.
* Update API support.Richard W.M. Jones2012-01-2324-474/+11234
|
* gobject: Run bindtests from srcdir.Hilko Bengen2012-01-231-2/+2
|
* build: don't use automake-internal variableJim Meyering2012-01-231-2/+2
| | | | | | | | | | | | | | | | Hi Rich, I realized a day or two late that my suggestion was not quite right. Here's the fix: >From 5294c21cf07c4ec2f094182ba9f32696f3de2751 Mon Sep 17 00:00:00 2001 From: Jim Meyering <meyering@redhat.com> Date: Mon, 23 Jan 2012 10:01:40 +0100 Subject: [PATCH libguestfs] build: don't use automake-internal variable * ocaml/Makefile.am (mlguestfs.cma): Use the documented variable form, $(libguestfsocaml_a_OBJECTS), not the $(am_libguestfsocaml_a_OBJECTS) that I suggested for commit 1.15.16-17-g8b9eaec.
* check for febootstrap etc. even if not building applianceHilko Bengen2012-01-231-31/+32
|
* fish: Filter out error messages about history file in test scriptHilko Bengen2012-01-231-1/+1
|
* Do not run appliance-related checks if not building applianceHilko Bengen2012-01-2315-13/+56
|
* Replace setting of environment variables with usage of local run scriptHilko Bengen2012-01-2319-63/+21
| | | | (Includes fix by RWMJ)
* php: function_entry -> zend_function_entry.Richard W.M. Jones2012-01-221-1/+1
| | | | | | function_entry was left over from PHP 3. See: http://news.php.net/php.pecl.dev/7123
* php: Enable 'make clean' in extension subdirectory.Richard W.M. Jones2012-01-221-4/+4
|
* Version 1.15.19.1.15.19Richard W.M. Jones2012-01-2220-2278/+2323
|
* gobject: Include bindtests-manual.js in EXTRA_DIST.Richard W.M. Jones2012-01-221-3/+7
| | | | This fixes commit 93b0769ec7929a909678411d75840f81bed55a81.
* gobject: Document these bindings in guestfs(3).Richard W.M. Jones2012-01-221-0/+7
|
* website: Rebuild the HTML manual pages from scratch.Richard W.M. Jones2012-01-221-0/+5
|
* daemon: Move internal-autosync function to new file internal.c.Richard W.M. Jones2012-01-214-18/+48
| | | | This is just code motion.
* gobject: Add bindtests for return valuesMatthew Booth2012-01-202-0/+111
|
* bindtests: Test optargs in test0Matthew Booth2012-01-205-33/+237
| | | | | | | | Note that this change disables compiling and running the haskell bindtests. The haskell bindings do not implement optargs, and adding optargs to test0 causes that method not to be bound in the haskell bindings. This prevents the haskell bindtests from compiling. These should be re-enabled when optargs are implemented.
* gobject: Add basic bindtestsMatthew Booth2012-01-206-2/+78
|
* run script: Add support for gobject introspectionMatthew Booth2012-01-201-3/+9
|
* gobject: Add GObject bindingsMatthew Booth2012-01-209-0/+1079
|
* generator: Add CamelName flagMatthew Booth2012-01-203-5/+15
| | | | | | | | | | We can make a good guess at camel case names for most APIs. For example, add_drive_opts can be automatically transformed to AddDriveOpts. However, other apis don't produce a satisfactory name when transformed automatically. For example, we would want md_create to produce MDCreate rather than MdCreate. This change adds a CamelName flag which allows a camel case name to be specified explicitly when the automatic transformation isn't satisfactory.
* resize, sparsify: find progress mini-library in out-of-tree buildsHilko Bengen2012-01-202-2/+2
|