mirror of
https://github.com/libguestfs/libguestfs.git
synced 2026-03-21 22:53:37 +00:00
Add IS_DEVICE checks for all calls which take a device parameter.
This commit is contained in:
@@ -122,6 +122,8 @@ do_set_e2label (const char *device, const char *label)
|
||||
int r;
|
||||
char *err;
|
||||
|
||||
IS_DEVICE (device, -1);
|
||||
|
||||
r = command (NULL, &err, "/sbin/e2label", device, label, NULL);
|
||||
if (r == -1) {
|
||||
reply_with_error ("e2label: %s", err);
|
||||
@@ -139,6 +141,8 @@ do_get_e2label (const char *device)
|
||||
int r, len;
|
||||
char *out, *err;
|
||||
|
||||
IS_DEVICE (device, NULL);
|
||||
|
||||
r = command (&out, &err, "/sbin/e2label", device, NULL);
|
||||
if (r == -1) {
|
||||
reply_with_error ("e2label: %s", err);
|
||||
@@ -163,6 +167,8 @@ do_set_e2uuid (const char *device, const char *uuid)
|
||||
int r;
|
||||
char *err;
|
||||
|
||||
IS_DEVICE (device, -1);
|
||||
|
||||
r = command (NULL, &err, "/sbin/tune2fs", "-U", uuid, device, NULL);
|
||||
if (r == -1) {
|
||||
reply_with_error ("tune2fs -U: %s", err);
|
||||
@@ -180,6 +186,8 @@ do_get_e2uuid (const char *device)
|
||||
int r;
|
||||
char *out, *err, *p, *q;
|
||||
|
||||
IS_DEVICE (device, NULL);
|
||||
|
||||
/* It's not so straightforward to get the volume UUID. We have
|
||||
* to use tune2fs -l and then look for a particular string in
|
||||
* the output.
|
||||
|
||||
12
daemon/lvm.c
12
daemon/lvm.c
@@ -176,6 +176,8 @@ do_pvcreate (const char *device)
|
||||
char *err;
|
||||
int r;
|
||||
|
||||
IS_DEVICE (device, -1);
|
||||
|
||||
r = command (NULL, &err,
|
||||
"/sbin/lvm", "pvcreate", device, NULL);
|
||||
if (r == -1) {
|
||||
@@ -195,6 +197,8 @@ do_vgcreate (const char *volgroup, char * const* const physvols)
|
||||
int r, argc, i;
|
||||
const char **argv;
|
||||
|
||||
Xphysvols;
|
||||
|
||||
argc = count_strings (physvols) + 3;
|
||||
argv = malloc (sizeof (char *) * (argc + 1));
|
||||
if (argv == NULL) {
|
||||
@@ -335,6 +339,8 @@ do_lvremove (const char *device)
|
||||
char *err;
|
||||
int r;
|
||||
|
||||
IS_DEVICE (device, -1);
|
||||
|
||||
r = command (NULL, &err,
|
||||
"/sbin/lvm", "lvremove", "-f", device, NULL);
|
||||
if (r == -1) {
|
||||
@@ -353,6 +359,8 @@ do_vgremove (const char *device)
|
||||
char *err;
|
||||
int r;
|
||||
|
||||
IS_DEVICE (device, -1);
|
||||
|
||||
r = command (NULL, &err,
|
||||
"/sbin/lvm", "vgremove", "-f", device, NULL);
|
||||
if (r == -1) {
|
||||
@@ -371,6 +379,8 @@ do_pvremove (const char *device)
|
||||
char *err;
|
||||
int r;
|
||||
|
||||
IS_DEVICE (device, -1);
|
||||
|
||||
r = command (NULL, &err,
|
||||
"/sbin/lvm", "pvremove", "-ff", device, NULL);
|
||||
if (r == -1) {
|
||||
@@ -389,6 +399,8 @@ do_pvresize (const char *device)
|
||||
char *err;
|
||||
int r;
|
||||
|
||||
IS_DEVICE (device, -1);
|
||||
|
||||
r = command (NULL, &err,
|
||||
"/sbin/lvm", "pvresize", device, NULL);
|
||||
if (r == -1) {
|
||||
|
||||
@@ -46,6 +46,8 @@ do_mount_vfs (const char *options, const char *vfstype,
|
||||
char *mp;
|
||||
char *error;
|
||||
|
||||
IS_DEVICE (device, -1);
|
||||
|
||||
is_root = strcmp (mountpoint, "/") == 0;
|
||||
|
||||
if (!root_mounted && !is_root) {
|
||||
@@ -110,9 +112,10 @@ do_umount (const char *pathordevice)
|
||||
char *buf;
|
||||
char *err;
|
||||
|
||||
if (strncmp (pathordevice, "/dev/", 5) == 0)
|
||||
buf = (char *) pathordevice;
|
||||
else {
|
||||
if (strncmp (pathordevice, "/dev/", 5) == 0) {
|
||||
buf = pathordevice;
|
||||
IS_DEVICE (buf, -1);
|
||||
} else {
|
||||
len = strlen (pathordevice) + 9;
|
||||
freeit = 1;
|
||||
buf = malloc (len);
|
||||
|
||||
Reference in New Issue
Block a user