git.annexia.org
/
libguestfs.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Fix 'file(1)' command to work on /dev devices.
[libguestfs.git]
/
daemon
/
file.c
diff --git
a/daemon/file.c
b/daemon/file.c
index
d8759f0
..
c09de71
100644
(file)
--- a/
daemon/file.c
+++ b/
daemon/file.c
@@
-320,19
+320,24
@@
char *
do_file (const char *path)
{
char *out, *err;
do_file (const char *path)
{
char *out, *err;
- int r, len;
+ int r, len
, freeit = 0
;
char *buf;
char *buf;
- NEED_ROOT (NULL);
+ /*NEED_ROOT (NULL); - no: we allow people to run this on /dev devices. */
ABS_PATH (path, NULL);
ABS_PATH (path, NULL);
- len = strlen (path) + 9;
- buf = malloc (len);
- if (!buf) {
- reply_with_perror ("malloc");
- return NULL;
+ if (strncmp (path, "/dev/", 5) == 0)
+ buf = (char *) path;
+ else {
+ len = strlen (path) + 9;
+ buf = malloc (len);
+ if (!buf) {
+ reply_with_perror ("malloc");
+ return NULL;
+ }
+ snprintf (buf, len, "/sysroot%s", path);
+ freeit = 1;
}
}
- snprintf (buf, len, "/sysroot%s", path);
/* file(1) manpage claims "file returns 0 on success, and non-zero on
* error", but this is evidently not true. It always returns 0, in
/* file(1) manpage claims "file returns 0 on success, and non-zero on
* error", but this is evidently not true. It always returns 0, in
@@
-340,13
+345,13
@@
do_file (const char *path)
* first.
*/
if (access (buf, R_OK) == -1) {
* first.
*/
if (access (buf, R_OK) == -1) {
- free (buf);
+
if (freeit)
free (buf);
reply_with_perror ("access: %s", path);
return NULL;
}
r = command (&out, &err, "file", "-bsL", buf, NULL);
reply_with_perror ("access: %s", path);
return NULL;
}
r = command (&out, &err, "file", "-bsL", buf, NULL);
- free (buf);
+
if (freeit)
free (buf);
if (r == -1) {
free (out);
if (r == -1) {
free (out);