[lvm-devel] LVM2/daemons/dmeventd libdevmapper-event.c
zkabelac at sourceware.org
zkabelac at sourceware.org
Thu Jan 14 10:11:27 UTC 2010
CVSROOT: /cvs/lvm2
Module name: LVM2
Changes by: zkabelac at sourceware.org 2010-01-14 10:11:26
Modified files:
daemons/dmeventd: libdevmapper-event.c
Log message:
Cleanup gcc warning: null argument where non-null required (arg.2)
As the const declaration of execvp is a bit weird, using local
dmeventdpath string.
Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/daemons/dmeventd/libdevmapper-event.c.diff?cvsroot=lvm2&r1=1.29&r2=1.30
--- LVM2/daemons/dmeventd/libdevmapper-event.c 2009/07/13 21:26:41 1.29
+++ LVM2/daemons/dmeventd/libdevmapper-event.c 2010/01/14 10:11:26 1.30
@@ -398,6 +398,8 @@
int pid, ret = 0;
int status;
struct stat statbuf;
+ char dmeventdpath[] = DMEVENTD_PATH; /* const type for execvp */
+ char * const args[] = { dmeventdpath, NULL };
if (stat(fifos->client_path, &statbuf))
goto start_server;
@@ -437,7 +439,7 @@
log_error("Unable to fork.");
else if (!pid) {
- execvp(DMEVENTD_PATH, NULL);
+ execvp(args[0], args);
_exit(EXIT_FAILURE);
} else {
if (waitpid(pid, &status, 0) < 0)
More information about the lvm-devel
mailing list