[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [PATCH] libxl: Compile with -Wformat-nonliteral
# HG changeset patch # User Ian Campbell <ian.campbell@xxxxxxxxxx> # Date 1323786705 0 # Node ID 1a48c8fbcae917bed86f861c26bcca8638905efe # Parent f28c55c5a90aacef252bd7100aa4ceb4c2077f0b libxl: Compile with -Wformat-nonliteral. At least one compiler (some Ubuntu version) uses this by default and it seems like a good idea anyway and the fixup required is trivial. One hunk is from a patch by Ian Jackson. Signed-off-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx> Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx> Reported-by: Adda Rathbone <addarathbone@xxxxxxxxxxxxxx> Tested-by: Adda Rathbone <addarathbone@xxxxxxxxxxxxxx> Tested-by: Andrew Pounce <andrew.pounce@xxxxxxxxxx> diff -r f28c55c5a90a -r 1a48c8fbcae9 tools/libxl/Makefile --- a/tools/libxl/Makefile Tue Dec 13 14:20:00 2011 +0000 +++ b/tools/libxl/Makefile Tue Dec 13 14:31:45 2011 +0000 @@ -12,7 +12,7 @@ XLUMAJOR = 1.0 XLUMINOR = 0 CFLAGS += -Werror -Wno-format-zero-length -Wmissing-declarations \ - -Wno-declaration-after-statement + -Wno-declaration-after-statement -Wformat-nonliteral CFLAGS += -I. -fPIC ifeq ($(CONFIG_Linux),y) diff -r f28c55c5a90a -r 1a48c8fbcae9 tools/libxl/libxl_create.c --- a/tools/libxl/libxl_create.c Tue Dec 13 14:20:00 2011 +0000 +++ b/tools/libxl/libxl_create.c Tue Dec 13 14:31:45 2011 +0000 @@ -459,8 +459,8 @@ static int store_libxl_entry(libxl__gc * path = libxl__xs_libxl_path(gc, domid); path = libxl__sprintf(gc, "%s/dm-version", path); - return libxl__xs_write(gc, XBT_NULL, path, "%s", libxl__strdup(gc, - libxl_device_model_version_to_string(dm_info->device_model_version))); + return libxl__xs_write(gc, XBT_NULL, path, "%s", + libxl_device_model_version_to_string(dm_info->device_model_version)); } static int do_domain_create(libxl__gc *gc, libxl_domain_config *d_config, diff -r f28c55c5a90a -r 1a48c8fbcae9 tools/libxl/libxl_device.c --- a/tools/libxl/libxl_device.c Tue Dec 13 14:20:00 2011 +0000 +++ b/tools/libxl/libxl_device.c Tue Dec 13 14:31:45 2011 +0000 @@ -515,7 +515,7 @@ int libxl__devices_destroy(libxl__gc *gc for (j = 0; j < num_devs; j++) { path = libxl__sprintf(gc, "/local/domain/%d/device/%s/%s/backend", domid, kinds[i], devs[j]); - path = libxl__xs_read(gc, XBT_NULL, libxl__sprintf(gc, "%s", path)); + path = libxl__xs_read(gc, XBT_NULL, path); if (path && libxl__parse_backend_path(gc, path, &dev) == 0) { dev.domid = domid; dev.kind = kind; _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxx http://lists.xensource.com/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |