[libvirt] [PATCH 2/2] Shorten domain name for automatic coredump
Pavel Hrdina
phrdina at redhat.com
Wed Apr 27 12:47:27 UTC 2016
On Tue, Apr 26, 2016 at 02:07:49PM +0200, Martin Kletzander wrote:
> If the domain name is long enough, the timestamp can prolong the
> filename for automatic coredump to more than the filesystem's limit.
> Simply shorten it like we do in other places. The timestamp helps with
> the unification, but having the ID in the name won't hurt.
>
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1289363
>
> Signed-off-by: Martin Kletzander <mkletzan at redhat.com>
> ---
> src/qemu/qemu_driver.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index 542d13c2ac70..2991bbc2ce4a 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -3965,13 +3965,14 @@ doCoreDumpToAutoDumpPath(virQEMUDriverPtr driver,
> char timestr[100];
> struct tm time_info;
> virQEMUDriverConfigPtr cfg = virQEMUDriverGetConfig(driver);
> + char *domname = virDomainObjGetShortName(vm);
>
if (!domname)
goto cleanup;
> localtime_r(&curtime, &time_info);
> strftime(timestr, sizeof(timestr), "%Y-%m-%d-%H:%M:%S", &time_info);
>
> if (virAsprintf(&dumpfile, "%s/%s-%s",
> cfg->autoDumpPath,
> - vm->def->name,
> + domname,
> timestr) < 0)
> goto cleanup;
>
> @@ -3984,6 +3985,7 @@ doCoreDumpToAutoDumpPath(virQEMUDriverPtr driver,
> "%s", _("Dump failed"));
> cleanup:
> VIR_FREE(dumpfile);
> + VIR_FREE(domname);
> virObjectUnref(cfg);
> return ret;
> }
ACK with the check for domname, safe for freeze.
More information about the libvir-list
mailing list