diff options
author | Richard Jones <rjones@redhat.com> | 2010-02-12 14:06:25 +0000 |
---|---|---|
committer | Richard Jones <rjones@redhat.com> | 2010-02-12 16:49:00 +0000 |
commit | e9c37113104c1cfb234535adc9b52ad3880a41ce (patch) | |
tree | a3367b5e700b9b455340243d7c66c45a5adf6067 /daemon/lvm.c | |
parent | 9b3ef85a3d1b43faa98bb5dc922f8a273a5e1dde (diff) | |
download | libguestfs-e9c37113104c1cfb234535adc9b52ad3880a41ce.tar.gz libguestfs-e9c37113104c1cfb234535adc9b52ad3880a41ce.tar.xz libguestfs-e9c37113104c1cfb234535adc9b52ad3880a41ce.zip |
daemon: Don't need to prefix error messages with the command name.
The RPC stubs already prefix the command name to error messages.
The daemon doesn't have to do this. As a (small) benefit this also
makes the daemon slightly smaller.
Code in the daemon such as:
if (argv[0] == NULL) {
reply_with_error ("passed an empty list");
return NULL;
}
now results in error messages like this:
><fs> command ""
libguestfs: error: command: passed an empty list
(whereas previously you would have seen ..command: command:..)
Diffstat (limited to 'daemon/lvm.c')
-rw-r--r-- | daemon/lvm.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/daemon/lvm.c b/daemon/lvm.c index 7a05a540..b100cd39 100644 --- a/daemon/lvm.c +++ b/daemon/lvm.c @@ -270,7 +270,7 @@ do_lvresize (const char *logvol, int mbytes) "/sbin/lvm", "lvresize", "-L", size, logvol, NULL); if (r == -1) { - reply_with_error ("lvresize: %s", err); + reply_with_error ("%s", err); free (err); return -1; } @@ -418,7 +418,7 @@ do_pvresize (const char *device) r = command (NULL, &err, "/sbin/lvm", "pvresize", device, NULL); if (r == -1) { - reply_with_error ("pvresize: %s: %s", device, err); + reply_with_error ("%s: %s", device, err); free (err); return -1; } @@ -479,7 +479,7 @@ do_lvrename (const char *logvol, const char *newlogvol) "/sbin/lvm", "lvrename", logvol, newlogvol, NULL); if (r == -1) { - reply_with_error ("lvrename: %s -> %s: %s", logvol, newlogvol, err); + reply_with_error ("%s -> %s: %s", logvol, newlogvol, err); free (err); return -1; } @@ -501,7 +501,7 @@ do_vgrename (const char *volgroup, const char *newvolgroup) "/sbin/lvm", "vgrename", volgroup, newvolgroup, NULL); if (r == -1) { - reply_with_error ("vgrename: %s -> %s: %s", volgroup, newvolgroup, err); + reply_with_error ("%s -> %s: %s", volgroup, newvolgroup, err); free (err); return -1; } |