External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-07-06 09:55:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

leaseshelper: Report more errors

Browse Source 
Some functions or code paths that may fail don't report error
(e.g. when acquiring PID file fails) leading to a silent quit
of the leaseshelper. This makes it super hard for us and users
to debug what is happening. Fortunately, dnsmasq captures both
stdout and stderr so we can write an error message there.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
This commit is contained in:
Michal Privoznik 2020-06-15 12:53:48 +02:00
parent 876211ef4a
commit 9ed345ac1a
1 changed files with 14 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/network/leaseshelper.c
View File

@ -131,8 +131,10 @@ main(int argc, char **argv)
* events for expired leases. So, libvirtd sets another env var for this
* purpose */
if (!interface &&
!(interface = getenv("VIR_BRIDGE_NAME")))
goto cleanup;
!(interface = getenv("VIR_BRIDGE_NAME"))) {
fprintf(stderr, _("interface not set\n"));
exit(EXIT_FAILURE);
}
ip = argv[3];
mac = argv[2];
@ -160,13 +162,21 @@ main(int argc, char **argv)
pid_file = g_strdup(RUNSTATEDIR "/leaseshelper.pid");
/* Try to claim the pidfile, exiting if we can't */
if ((pid_file_fd = virPidFileAcquirePath(pid_file, true, getpid())) < 0)
if ((pid_file_fd = virPidFileAcquirePath(pid_file, true, getpid())) < 0) {
fprintf(stderr,
_("Unable to acquire PID file: %s\n errno=%d"),
pid_file, errno);
goto cleanup;
}
/* Since interfaces can be hot plugged, we need to make sure that the
* corresponding custom lease file exists. If not, 'touch' it */
if (virFileTouch(custom_lease_file, 0644) < 0)
if (virFileTouch(custom_lease_file, 0644) < 0) {
fprintf(stderr,
_("Unable to create: %s\n errno=%d"),
custom_lease_file, errno);
goto cleanup;
}
switch ((enum virLeaseActionFlags) action) {
case VIR_LEASE_ACTION_ADD: